Shadowing fix in gdbscm_frame_read_var
authorTom Tromey <tom@tromey.com>
Sun, 22 Apr 2018 16:46:03 +0000 (10:46 -0600)
committerTom Tromey <tom@tromey.com>
Fri, 5 Oct 2018 04:51:48 +0000 (22:51 -0600)
-Wshadow=local pointed out that the shadowing in gdbscm_frame_read_var
means that the ultimate call to read_var_value will always be passed
block==NULL.  The fix is to remove the inner declaration.

gdb/ChangeLog
2018-10-04  Tom Tromey  <tom@tromey.com>

* guile/scm-frame.c (gdbscm_frame_read_var): Remove inner
declaration of "block".

gdb/ChangeLog
gdb/guile/scm-frame.c

index 42c55edf4c80c49cdd1267bc8a314dfb38de56cc..3bf08bcb4b0485512152054484ef8a399b6721a4 100644 (file)
@@ -1,3 +1,8 @@
+2018-10-04  Tom Tromey  <tom@tromey.com>
+
+       * guile/scm-frame.c (gdbscm_frame_read_var): Remove inner
+       declaration of "block".
+
 2018-10-04  Tom Tromey  <tom@tromey.com>
 
        * common/filestuff.c (fdwalk): Remove inner declaration of
index 3d27907e1935455b2d55f72e129ad96c675cc115..5a056de3328af43e0e368be93e3869737757a005 100644 (file)
@@ -878,7 +878,6 @@ gdbscm_frame_read_var (SCM self, SCM symbol_scm, SCM rest)
     }
   else if (scm_is_string (symbol_scm))
     {
-      const struct block *block = NULL;
       struct gdb_exception except = exception_none;
 
       if (! SCM_UNBNDP (block_scm))