Avoid shadowing in linux-tdep.c
authorTom Tromey <tom@tromey.com>
Sun, 22 Apr 2018 16:46:51 +0000 (10:46 -0600)
committerTom Tromey <tom@tromey.com>
Fri, 5 Oct 2018 04:51:47 +0000 (22:51 -0600)
This is one of the uglier changes to avoid local shadowing.  Because
obstack.h uses statement expressions, in some cases a nested obstack
call will result in shadowing.  Rather than try to fix obstack.h, this
patch simply works around the one instance of this problem.

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

* linux-tdep.c (linux_make_mappings_corefile_notes): Introduce new
variable "size".

gdb/ChangeLog
gdb/linux-tdep.c

index 0b5f35b2b7c9b2bf44c35927fbacb5193b46f8fb..c547ee96258fccf64bc5e549a1bec3ab52dc22a9 100644 (file)
@@ -1,3 +1,8 @@
+2018-10-04  Tom Tromey  <tom@tromey.com>
+
+       * linux-tdep.c (linux_make_mappings_corefile_notes): Introduce new
+       variable "size".
+
 2018-10-04  Tom Tromey  <tom@tromey.com>
 
        * mdebugread.c (parse_partial_symbols): Use std::string.
index 8c76ec316b84310e7e939a6917a71f50c9a53420..352114943fe807d4832a56cf7c1666945c1474c7 100644 (file)
@@ -1547,8 +1547,9 @@ linux_make_mappings_corefile_notes (struct gdbarch *gdbarch, bfd *obfd,
                 long_type, mapping_data.file_count);
 
       /* Copy the filenames to the data obstack.  */
+      int size = obstack_object_size (&filename_obstack);
       obstack_grow (&data_obstack, obstack_base (&filename_obstack),
-                   obstack_object_size (&filename_obstack));
+                   size);
 
       note_data = elfcore_write_note (obfd, note_data, note_size,
                                      "CORE", NT_FILE,