[Patch] Refractor Thompson matcher

Paolo Carlini paolo.carlini@oracle.com
Tue Jul 30 23:18:00 GMT 2013


Hi,

On 07/31/2013 01:11 AM, Tim Shen wrote:
> On Wed, Jul 31, 2013 at 7:03 AM, Paolo Carlini <paolo.carlini@oracle.com> wrote:
>> Did you actually bootstrap & test the patch? Because you didn't last time,
>> right?
> I compiled it and run the 28_regex testsuite, nothing wrong happened
> because there're all single-file testcases in it;
> but I ignored the duplicated definition problem. It'll never happen in
> the future. Sorry again.
Earlier, even without bootstrapping you should have noticed that the 
library didn't build. It's not about excuses, it's about following a 
policy, which, should be obvious now, we are following for a reason: all 
posted patches must be boostrapped & tested.
>> Please add FIXME comments to all the non-template functions in *.tcc files
>> (which will be moved to *.cc files). To repeat: the inlines in *.tcc files
>> must be all and only non-templates, as a temporary hack.
> Added.
Please post the complete patch you intend to commit. Part of the GCC 
policy is also that all the patches must be posted complete, exactly as 
would be committed upon approval.

Paolo.



More information about the Libstdc++ mailing list