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: Implement -Wswitch-fallthrough


On Mon, Jul 11, 2016 at 01:01:05PM -0700, Andi Kleen wrote:
> Marek Polacek <polacek@redhat.com> writes:
> >
> > This warning is enabled by default for C/C++.
> 
> That's purely evil. So have to put your non standard builtin all over
> standards compliant code just to shut off a warning that is likely
> common. And you're not even supporting the classic lint style comment either.
 
I explained why supporting the classic lint style comment wouldn't fly.

> I think it's ok to make this a separate option, but even putting it in
> -Wall would annoy far too many people (including me) for good reasons.
> Making it default is just totally wrong.

Ok.  As I mentioned elsewhere, moving this to -Wextra (or even out of -Wall/-Wextra)
is a one liner change.

	Marek


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