Remove extra scopes from objfile_relocate1
authorTom Tromey <tom@tromey.com>
Tue, 7 Mar 2023 23:24:11 +0000 (16:24 -0700)
committerTom Tromey <tom@tromey.com>
Sat, 11 Mar 2023 15:48:10 +0000 (08:48 -0700)
objfile_relocate1 introduces new scopes that aren't necessary.  I
noticed this while working on an earlier patch in this series.  This
patch removes these.

Approved-By: Simon Marchi <simon.marchi@efficios.com>
gdb/objfiles.c

index 2c8f5d14444639972208b90f0aa1714f5990f9a7..21ec8c7ad2ea73accc795a888a727088309c4b26 100644 (file)
@@ -615,53 +615,43 @@ objfile_relocate1 (struct objfile *objfile,
     return 0;
 
   /* OK, get all the symtabs.  */
-  {
-    for (compunit_symtab *cust : objfile->compunits ())
-      {
-       struct blockvector *bv = cust->blockvector ();
-       int block_line_section = SECT_OFF_TEXT (objfile);
-
-       if (bv->map () != nullptr)
-         bv->map ()->relocate (delta[block_line_section]);
-
-       for (block *b : bv->blocks ())
-         {
-           struct symbol *sym;
-           struct mdict_iterator miter;
-
-           b->set_start (b->start () + delta[block_line_section]);
-           b->set_end (b->end () + delta[block_line_section]);
-
-           for (blockrange &r : b->ranges ())
-             {
-               r.set_start (r.start () + delta[block_line_section]);
-               r.set_end (r.end () + delta[block_line_section]);
-             }
-
-           /* We only want to iterate over the local symbols, not any
-              symbols in included symtabs.  */
-           ALL_DICT_SYMBOLS (b->multidict (), miter, sym)
-             {
-               relocate_one_symbol (sym, objfile, delta);
-             }
-         }
-      }
-  }
+  for (compunit_symtab *cust : objfile->compunits ())
+    {
+      struct blockvector *bv = cust->blockvector ();
+      int block_line_section = SECT_OFF_TEXT (objfile);
 
-  /* Relocate isolated symbols.  */
-  {
-    struct symbol *iter;
+      if (bv->map () != nullptr)
+       bv->map ()->relocate (delta[block_line_section]);
 
-    for (iter = objfile->template_symbols; iter; iter = iter->hash_next)
-      relocate_one_symbol (iter, objfile, delta);
-  }
+      for (block *b : bv->blocks ())
+       {
+         struct symbol *sym;
+         struct mdict_iterator miter;
 
-  {
-    int i;
+         b->set_start (b->start () + delta[block_line_section]);
+         b->set_end (b->end () + delta[block_line_section]);
 
-    for (i = 0; i < objfile->section_offsets.size (); ++i)
-      objfile->section_offsets[i] = new_offsets[i];
-  }
+         for (blockrange &r : b->ranges ())
+           {
+             r.set_start (r.start () + delta[block_line_section]);
+             r.set_end (r.end () + delta[block_line_section]);
+           }
+
+         /* We only want to iterate over the local symbols, not any
+            symbols in included symtabs.  */
+         ALL_DICT_SYMBOLS (b->multidict (), miter, sym)
+           {
+             relocate_one_symbol (sym, objfile, delta);
+           }
+       }
+    }
+
+  /* Relocate isolated symbols.  */
+  for (symbol *iter = objfile->template_symbols; iter; iter = iter->hash_next)
+    relocate_one_symbol (iter, objfile, delta);
+
+  for (int i = 0; i < objfile->section_offsets.size (); ++i)
+    objfile->section_offsets[i] = new_offsets[i];
 
   /* Rebuild section map next time we need it.  */
   get_objfile_pspace_data (objfile->pspace)->section_map_dirty = 1;