[PATCH 034/238] [misc.] Fix -Wshadow warnings.
Pedro Alves
pedro@codesourcery.com
Tue Dec 6 18:25:00 GMT 2011
On Monday 05 December 2011 13:26:46, Joel Brobecker wrote:
> > > Looks OK to me. But Pedro might comment on the new variable name
> > > (sal->sal2).
> [...]
> > Initially it was sal->las which probably isn't better, so I'll wait
> > for some consensus to arise on this patch too.
>
> FWIW, I'm OK with sal2. I also agree with stan that it's best to
> avoid `las' (cute, though :-))
sal2 is fine with me too.
There might actually be a bug here, that I still haven't managed
to investigate:
http://sourceware.org/ml/gdb-patches/2011-03/msg00066.html
(but don't let this block your patch.)
Michael wanted sal2 too, so sal2 it is.
Curiously, we have very few instances of struct symtab_and_line
that aren't called "sal" or don't have "sal" in their name:
$ grep "struct symtab_and_line[ \t]\+[A-Za-z_]\+;" * -rn | grep -v sal
linespec.c:1910: struct symtab_and_line val;
linespec.c:2000: struct symtab_and_line val;
mi/mi-cmd-file.c:36: struct symtab_and_line st;
symtab.c:1923: struct symtab_and_line val;
But they're all close enough. Interestingly, sal2 is the first. :-)
$ grep sal2 * -rn
*empty*
Sals are funny creatures.
--
Pedro Alves
More information about the Gdb-patches
mailing list