On Fri, Oct 4, 2019 at 1:21 PM Bernd Edlinger <bernd.edlin...@hotmail.de> wrote: > > On 10/4/19 12:43 PM, Richard Biener wrote: > > On Thu, Oct 3, 2019 at 5:17 PM Bernd Edlinger <bernd.edlin...@hotmail.de> > > wrote: > >> > >> Hi, > >> > >> this adds -Wshadow=local to the GCC build rules. > >> > >> It is to be applied after all other patches in this series > >> including the trivial ones are applied. > >> > >> Bootstrapped and reg-tested on x86_64-pc-linux-gnu. > >> Is it OK for trunk? > > > > The -Wshadow=local hunk is obviously OK but there's > > a hunk in the patch adding -Wextra as well...?! > > > > Yes, that replaces -W with -Wextra, I couldn't resist > to doing that when I was already there, since > > gcc -v --help > prints: > -W This switch is deprecated; use -Wextra instead > > So the change log says that as well: > > > * configure.ac (ACX_PROG_CXX_WARNING_OPTS): Use -Wextra instead of > > -W. > > Add -Wshadow=local. > > Would you like me to split that patch for the -Wextra?
Oh, no need - just didin't look at the ChangeLog... Richard. > > Bernd.
