[Patch 3/3] Separate `repeating` node from `alternative` node in regex

2014-04-25 Thread Tim Shen
Before this patch both alternative operator "|" and star operator "*" generate _S_opcode_alternative node. Now they generate _S_opcode_alternative and _S_opcode_repeat respectively, because the alternative handing is simpler comparing to the repeat node handling introduced by last patch. The whol

Re: [Patch 3/3] Separate `repeating` node from `alternative` node in regex

2014-04-25 Thread Tim Shen
Patch attached. -- Regards, Tim Shen commit cd0683e3a3eae0c62d2867fb5456edb3735b38ae Author: tim Date: Fri Apr 25 10:45:26 2014 -0400 2014-04-25 Tim Shen * include/bits/regex_automaton.h (_NFA<>::_M_insert_repeat): Add _S_opcode_repeat support to distingush a loop

Re: [Patch 3/3] Separate `repeating` node from `alternative` node in regex

2014-04-27 Thread Jonathan Wakely
On 25/04/14 18:04 -0400, Tim Shen wrote: * include/bits/regex_automaton.h (_NFA<>::_M_insert_repeat): Add _S_opcode_repeat support to distingush a loop from _S_opcode_alternative. * include/bits/regex_automaton.tcc (_State_base::_M_print, _State_base::_M_do

Re: [Patch 3/3] Separate `repeating` node from `alternative` node in regex

2014-04-27 Thread Tim Shen
On Sun, Apr 27, 2014 at 8:31 AM, Jonathan Wakely wrote: > This one is OK to commit too - thanks. Committed. Thanks! -- Regards, Tim Shen