[C++ Patch] PR 37087 (take 2)

Paolo Carlini paolo.carlini@oracle.com
Tue Aug 12 17:02:00 GMT 2008


Hi Manuel,
> Would you mind using in testcases as much as the error output as
> possible to match? Otherwise we may issue a wrong message and not
> notice the regression. Moreover, automatic changes are easier if you
> can grep for the error messages.
>
> Also, in C++ testcases, you MUST use "error:", "warning:" or "note:"
> in the match because the g++ testsuite cannot distinguish those.
> That is PR 25241.
>   
Ok, no problem, I didn't know about PR 25241 and well, I suspect a few 
others contributors are "guilty" of the same crime... ;)

Anyway, many thanks for the suggestion, I tweaked the new testcase.

Paolo.

///////////
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch_37087_2_b
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20080812/75702df0/attachment.ksh>


More information about the Gcc-patches mailing list