Mercurial > hg > octave-terminal
changeset 12921:7820a12baadd
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 +++ b/src/pt-binop.cc @@ -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) {