Mercurial > octave-nkf

changeset 12921:7820a12baadd

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Style fixes on comments about profiling operators
author Jordi Gutiérrez Hermoso <jordigh@gmail.com>
date Thu, 04 Aug 2011 23:08:20 -0500
parents 5d18231eee00
children 1322308fa83a
files src/pt-binop.cc
diffstat 1 files changed, 9 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/pt-binop.cc	Fri Jul 29 17:51:39 2011 +0200
+++ b/src/pt-binop.cc	Thu Aug 04 23:08:20 2011 -0500
@@ -124,8 +124,11 @@
               profile_data_accumulator::enter pe (profiler,
                                                   "binary " + oper ());
 
-              /* Note: The profiler does not catch the braindead-short-circuit
-                 evaluation code above.  But that should be ok.  */
+              // Note: The profiler does not catch the braindead
+              // short-circuit evaluation code above, but that should be
+              // ok. The evaluation of operands and the operator itself
+              // is entangled and it's not clear where to start/stop
+              // timing the operator to make it reasonable.
 
               retval = ::do_binary_op (etype, a, b);
 
@@ -190,10 +193,10 @@
 
   bool result = false;
 
-  /* This evaluation is not caught by the profiler, since we can't find
-     a reasonable place where to time.  Note that we don't want to include
-     evaluation of LHS or RHS into the timing, but this is entangled
-     together with short-circuit evaluation here.  */
+  // This evaluation is not caught by the profiler, since we can't find
+  // a reasonable place where to time. Note that we don't want to
+  // include evaluation of LHS or RHS into the timing, but this is
+  // entangled together with short-circuit evaluation here.
 
   if (op_lhs)
     {