Hi,

On 10/27/2013 09:12 PM, Tim Shen wrote:
@@ -190,9 +224,9 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
            }
          break;
        case _S_opcode_subexpr_begin:
-         // Here's the critical part: if there's nothing changed since last
-         // visit, do NOT continue. This prevents the executor from get into
-         // infinite loop when use "()*" to match "".
+         // If there's nothing changed since last visit, do NOT continue.
+         // This prevents the executor from get into infinite loop when use
+         // "()*" to match "".
          //
          // Every change on _M_cur_results will be roll back after the
          // recursion step finished.
Should we move this comment too before the 'case', and aligned with it?

Either way, patch Ok of course.

Paolo.

Reply via email to