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.