This is the mail archive of the gcc@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: Adding -Wshadow=local to gcc build rules

From: Segher Boessenkool <segher at kernel dot crashing dot org>
To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>
Cc: "gcc at gcc dot gnu dot org" <gcc at gcc dot gnu dot org>
Date: Sat, 21 Sep 2019 04:44:48 -0500
Subject: Re: Adding -Wshadow=local to gcc build rules
References: <AM6PR10MB256682838C777B5C8828C4BEE48E0@AM6PR10MB2566.EURPRD10.PROD.OUTLOOK.COM>

Hi!

On Wed, Sep 18, 2019 at 01:08:50PM +0000, Bernd Edlinger wrote:
> I'm currently trying to add -Wshadow=local to the gcc build rules.
> I started with -Wshadow, but gave up that idea immediately.
> 
> As you could expect the current code base has plenty of shadowed
> local variables.  Most are trivial to resolve, some are less trivial.
> I am not finished yet, but it is clear that it will be a rather big
> patch.

It shouldn't be a big patch.  It should be a biggish collection of small
patches, most of which are trivial and obvious and you can just commit
without further approval!


Segher

References:
- Adding -Wshadow=local to gcc build rules
  - From: Bernd Edlinger

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