This is the mail archive of the
mailing list for the GCC project.
Re: PR cpp/33907 Empty macro definitions not considered equal
Tom Tromey <email@example.com> writes:
| >>>>> "Manuel" == Manuel López-Ibáñez <firstname.lastname@example.org> writes:
| Manuel> This patch fixes PR33907. The conclusion is that warning about A being
| Manuel> redefined for this testcase is annoying:
| Manuel> #define A(a)
| Manuel> #define A(b)
| Is this a change from 4.2?
| It seems to me that if we can be pedantic without unduly affecting
| existing code, then we ought to be.
I strongly agree. My opinion is that, bu default, we should emit a
diagnostic on the above construct -- they can hide real bugs and that
did happen to combination of real codes I've seen (think of the macro
likely on some linux boxes).