[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