This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PING] [PATCH/C++/diagnostic] Be extra pedantic about stray semicolons


On 21/03/07, Mike Stump <mrs@apple.com> wrote:
On Mar 21, 2007, at 3:35 AM, Dirk Mueller wrote:
>         * g++.old-deja/g++.brendan/warnings2.C: Don't use -pedantic.
>       * g++.dg/other/static1.C: Remove stray semicolon.
[ ... ]
>         * g++.old-deja/g++.bob/inherit1.C: Likewise.

I reviewed the testsuite changes, and irrespective of the other
changes, I think they should be put in.

Given the largeness of the testsuite fixes, and given most of these
came from code, I do wonder if we'd be better off with a separate
switch to turn the error/warning off so people can migrate their code.


If it is in pedantic, it is because the warning is mandated by the standard, exactly what people should expect from pedantic. It is interesting to be able to enable warnings without enabling pedantic. But if you enable pedantic, I understand that you want to get the warnings mandated by the standard.

Just my opinion, which could be wrong.

Cheers,

Manuel.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]