------- Comment #10 from dfranke at gcc dot gnu dot org  2007-07-12 07:27 
-------
> BTW, if you revert a patch, you should add the testcase that is the reason
> it. To avoid regressing again if the patch gets re-applied. Similarly, you
> should xfail the other testcase that now fails.

Joost,

good point. I thought about XFAIL, but then again, probably nobody will notice
that there's something wrong since it's expected to fail. Hence I concluded
that it would be preferable to have people asking about a testsuite failure
twice a day then to hide the problem it in an xfail :)

However, if Paul does find the time today, I'm positive that he will set it
right -- otherwise I will implement your suggestion tomorrow.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32727

Reply via email to