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

[Manuel López-Ibáñez]

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.