Re: Adding -Wshadow=local to gcc build rules

2019-10-02 Thread Bernd Edlinger
On 9/18/19 3:08 PM, Bernd Edlinger wrote: > Hi, > > 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

Re: Adding -Wshadow=local to gcc build rules

2019-09-21 Thread Segher Boessenkool
Hi! On Wed, Sep 18, 2019 at 01:08:50PM +, 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

Re: Adding -Wshadow=local to gcc build rules

2019-09-20 Thread Jason Merrill
On Fri, Sep 20, 2019, 2:21 PM Bernd Edlinger wrote: > On 9/19/19 12:19 PM, Richard Biener wrote: > > On Wed, Sep 18, 2019 at 3:09 PM Bernd Edlinger > > wrote: > >> > >> Hi, > >> > >> I'm currently trying to add -Wshadow=local to the gcc build rules. > >> I started with -Wshadow, but gave up that

Re: Adding -Wshadow=local to gcc build rules

2019-09-20 Thread Bernd Edlinger
On 9/19/19 12:19 PM, Richard Biener wrote: > On Wed, Sep 18, 2019 at 3:09 PM Bernd Edlinger > wrote: >> >> Hi, >> >> 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

Re: Adding -Wshadow=local to gcc build rules

2019-09-19 Thread Richard Biener
On Wed, Sep 18, 2019 at 3:09 PM Bernd Edlinger wrote: > > Hi, > > 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 trivi

Re: Adding -Wshadow=local to gcc build rules

2019-09-18 Thread Tom Tromey
> "Bernd" == Bernd Edlinger writes: Bernd> I'm currently trying to add -Wshadow=local to the gcc build rules. Bernd> I started with -Wshadow, but gave up that idea immediately. Bernd> As you could expect the current code base has plenty of shadowed Bernd> local variables. Most are trivial t

Re: Adding -Wshadow=local to gcc build rules

2019-09-18 Thread Martin Liška
On 9/18/19 3:08 PM, Bernd Edlinger wrote: > Hi, > > 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

Adding -Wshadow=local to gcc build rules

2019-09-18 Thread Bernd Edlinger
Hi, 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 i