This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PING^3] Wconversion: fixes for C++ front-end
On 06 Feb 2007 13:20:54 -0600, Gabriel Dos Reis <gdr@cs.tamu.edu> wrote:
Paolo Carlini <pcarlini@suse.de> writes:
| Manuel López-Ibáñez wrote:
|
| > http://gcc.gnu.org/ml/gcc-patches/2006-12/msg00799.html
| >
| > Rediff, rebootstrapped and re-regression-tested against revision
| > 121356 on i686-pc-linux-gnu.
| >
| > Context and unified diffs follow.
|
| Is there something I'm missing or the new testcase
| Wconversion-integer.C doesn't reflect the consensus resulting from the
| discussions in the audit trail of libstdc++/30464?!? Shall we wait a
| bit and add the final version of it???
Waiting will not magically turn the consensus, without new data.
I was hoping the patch will be upgraded to reflect the consensus.
I am waiting for patches to get reviewed. I hope that at least this is
not my fault. If I can do something to get the patches reviewed
faster, please, let me know.
As I said before, this patch has nothing to do with the warning logic
for Wconversion, it just updates places in the C++ front-end where
convert_and_check should be called, add testcases for C++ and updates
the documentation. Yes, I could combine both things: upgrade the
warning logic and fix the C++ front-end, but why? I believed that a
patch should be as simple as possible. Having two patches is simpler.
Cheers,
Manuel.