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: Change default level for -Wimplicit-fallthrough

From: Jeff Law <law at redhat dot com>
To: Bernd Schmidt <bschmidt at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 2 Dec 2016 10:26:13 -0700
Subject: Re: Change default level for -Wimplicit-fallthrough
Authentication-results: sourceware.org; auth=none
References: <2fa103b3-9389-0a5e-a253-6b96dfe61d38@redhat.com>

On 11/03/2016 05:51 AM, Bernd Schmidt wrote:

I'm concerned about the number of false positives for this warning, and
judging by previous discussions, I'm not alone in this. This patch
limits it to level 1 (any comment before the case label disables the
warning) for cases where the user specified no explicit level. It'll
still generate enough noise that people will be aware of it and can
choose whether to use a higher level or not.

Bootstrapped and tested on x86_64-linux. Ok?

I think we should wait for the distro builds to fire up and see how thewarning impacts them before making a decision on this patch.


jeff

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