daily update
[deliverable/binutils-gdb.git] / gdb / objfiles.c
index 82ef45c934714499c8ac30ce2cb2587a5456f508..953bc88fdce51fc123a4e5feb300ac2fd8692a5b 100644 (file)
@@ -53,6 +53,7 @@
 #include "observer.h"
 #include "complaints.h"
 #include "psymtab.h"
+#include "solist.h"
 
 /* Prototypes for local functions */
 
@@ -688,6 +689,11 @@ void
 free_all_objfiles (void)
 {
   struct objfile *objfile, *temp;
+  struct so_list *so;
+
+  /* Any objfile referencewould become stale.  */
+  for (so = master_so_list (); so; so = so->next)
+    gdb_assert (so->objfile == NULL);
 
   ALL_OBJFILES_SAFE (objfile, temp)
   {
@@ -701,26 +707,26 @@ free_all_objfiles (void)
    Return non-zero iff any change happened.  */
 
 static int
-objfile_relocate1 (struct objfile *objfile, struct section_offsets *new_offsets)
+objfile_relocate1 (struct objfile *objfile, 
+                  struct section_offsets *new_offsets)
 {
   struct obj_section *s;
   struct section_offsets *delta =
     ((struct section_offsets *) 
      alloca (SIZEOF_N_SECTION_OFFSETS (objfile->num_sections)));
 
-  {
-    int i;
-    int something_changed = 0;
-    for (i = 0; i < objfile->num_sections; ++i)
-      {
-       delta->offsets[i] =
-         ANOFFSET (new_offsets, i) - ANOFFSET (objfile->section_offsets, i);
-       if (ANOFFSET (delta, i) != 0)
-         something_changed = 1;
-      }
-    if (!something_changed)
-      return 0;
-  }
+  int i;
+  int something_changed = 0;
+
+  for (i = 0; i < objfile->num_sections; ++i)
+    {
+      delta->offsets[i] =
+       ANOFFSET (new_offsets, i) - ANOFFSET (objfile->section_offsets, i);
+      if (ANOFFSET (delta, i) != 0)
+       something_changed = 1;
+    }
+  if (!something_changed)
+    return 0;
 
   /* OK, get all the symtabs.  */
   {
@@ -788,6 +794,7 @@ objfile_relocate1 (struct objfile *objfile, struct section_offsets *new_offsets)
 
   {
     struct minimal_symbol *msym;
+
     ALL_OBJFILE_MSYMBOLS (objfile, msym)
       if (SYMBOL_SECTION (msym) >= 0)
       SYMBOL_VALUE_ADDRESS (msym) += ANOFFSET (delta, SYMBOL_SECTION (msym));
@@ -810,6 +817,7 @@ objfile_relocate1 (struct objfile *objfile, struct section_offsets *new_offsets)
 
   {
     int i;
+
     for (i = 0; i < objfile->num_sections; ++i)
       (objfile->section_offsets)->offsets[i] = ANOFFSET (new_offsets, i);
   }
@@ -853,7 +861,6 @@ objfile_relocate (struct objfile *objfile, struct section_offsets *new_offsets)
     {
       struct section_addr_info *objfile_addrs;
       struct section_offsets *new_debug_offsets;
-      int new_debug_num_sections;
       struct cleanup *my_cleanups;
 
       objfile_addrs = build_section_addr_info_from_objfile (objfile);
@@ -866,8 +873,8 @@ objfile_relocate (struct objfile *objfile, struct section_offsets *new_offsets)
 
       gdb_assert (debug_objfile->num_sections
                  == bfd_count_sections (debug_objfile->obfd));
-      new_debug_offsets = xmalloc (SIZEOF_N_SECTION_OFFSETS
-                                                (debug_objfile->num_sections));
+      new_debug_offsets = 
+       xmalloc (SIZEOF_N_SECTION_OFFSETS (debug_objfile->num_sections));
       make_cleanup (xfree, new_debug_offsets);
       relative_addr_info_to_section_offsets (new_debug_offsets,
                                             debug_objfile->num_sections,
@@ -1004,65 +1011,64 @@ qsort_cmp (const void *a, const void *b)
   else if (sect1_addr > sect2_addr)
     return 1;
   else
-   {
-     /* Sections are at the same address.  This could happen if
-       A) we have an objfile and a separate debuginfo.
-       B) we are confused, and have added sections without proper relocation,
-       or something like that. */
-
-     const struct objfile *const objfile1 = sect1->objfile;
-     const struct objfile *const objfile2 = sect2->objfile;
-
-     if (objfile1->separate_debug_objfile == objfile2
-        || objfile2->separate_debug_objfile == objfile1)
-       {
-        /* Case A.  The ordering doesn't matter: separate debuginfo files
-           will be filtered out later.  */
-
-        return 0;
-       }
-
-     /* Case B.  Maintain stable sort order, so bugs in GDB are easier to
-       triage.  This section could be slow (since we iterate over all
-       objfiles in each call to qsort_cmp), but this shouldn't happen
-       very often (GDB is already in a confused state; one hopes this
-       doesn't happen at all).  If you discover that significant time is
-       spent in the loops below, do 'set complaints 100' and examine the
-       resulting complaints.  */
-
-     if (objfile1 == objfile2)
-       {
-        /* Both sections came from the same objfile.  We are really confused.
-           Sort on sequence order of sections within the objfile.  */
-
-        const struct obj_section *osect;
-
-        ALL_OBJFILE_OSECTIONS (objfile1, osect)
-          if (osect == sect1)
-            return -1;
-          else if (osect == sect2)
-            return 1;
-
-        /* We should have found one of the sections before getting here.  */
-        gdb_assert (0);
-       }
-     else
-       {
-        /* Sort on sequence number of the objfile in the chain.  */
-
-        const struct objfile *objfile;
-
-        ALL_OBJFILES (objfile)
-          if (objfile == objfile1)
-            return -1;
-          else if (objfile == objfile2)
-            return 1;
-
-        /* We should have found one of the objfiles before getting here.  */
-        gdb_assert (0);
-       }
-
-   }
+    {
+      /* Sections are at the same address.  This could happen if
+        A) we have an objfile and a separate debuginfo.
+        B) we are confused, and have added sections without proper relocation,
+        or something like that. */
+
+      const struct objfile *const objfile1 = sect1->objfile;
+      const struct objfile *const objfile2 = sect2->objfile;
+
+      if (objfile1->separate_debug_objfile == objfile2
+         || objfile2->separate_debug_objfile == objfile1)
+       {
+         /* Case A.  The ordering doesn't matter: separate debuginfo files
+            will be filtered out later.  */
+
+         return 0;
+       }
+
+      /* Case B.  Maintain stable sort order, so bugs in GDB are easier to
+        triage.  This section could be slow (since we iterate over all
+        objfiles in each call to qsort_cmp), but this shouldn't happen
+        very often (GDB is already in a confused state; one hopes this
+        doesn't happen at all).  If you discover that significant time is
+        spent in the loops below, do 'set complaints 100' and examine the
+        resulting complaints.  */
+
+      if (objfile1 == objfile2)
+       {
+         /* Both sections came from the same objfile.  We are really confused.
+            Sort on sequence order of sections within the objfile.  */
+
+         const struct obj_section *osect;
+
+         ALL_OBJFILE_OSECTIONS (objfile1, osect)
+           if (osect == sect1)
+             return -1;
+           else if (osect == sect2)
+             return 1;
+
+         /* We should have found one of the sections before getting here.  */
+         gdb_assert (0);
+       }
+      else
+       {
+         /* Sort on sequence number of the objfile in the chain.  */
+
+         const struct objfile *objfile;
+
+         ALL_OBJFILES (objfile)
+           if (objfile == objfile1)
+             return -1;
+           else if (objfile == objfile2)
+             return 1;
+
+         /* We should have found one of the objfiles before getting here.  */
+         gdb_assert (0);
+       }
+    }
 
   /* Unreachable.  */
   gdb_assert (0);
@@ -1471,6 +1477,23 @@ objfiles_changed (void)
   get_objfile_pspace_data (current_program_space)->objfiles_changed_p = 1;
 }
 
+/* Close ABFD, and warn if that fails.  */
+
+int
+gdb_bfd_close_or_warn (struct bfd *abfd)
+{
+  int ret;
+  char *name = bfd_get_filename (abfd);
+
+  ret = bfd_close (abfd);
+
+  if (!ret)
+    warning (_("cannot close \"%s\": %s"),
+            name, bfd_errmsg (bfd_get_error ()));
+
+  return ret;
+}
+
 /* Add reference to ABFD.  Returns ABFD.  */
 struct bfd *
 gdb_bfd_ref (struct bfd *abfd)
@@ -1519,9 +1542,7 @@ gdb_bfd_unref (struct bfd *abfd)
   bfd_usrdata (abfd) = NULL;  /* Paranoia.  */
 
   name = bfd_get_filename (abfd);
-  if (!bfd_close (abfd))
-    warning (_("cannot close \"%s\": %s"),
-            name, bfd_errmsg (bfd_get_error ()));
+  gdb_bfd_close_or_warn (abfd);
   xfree (name);
 }
 
This page took 0.026114 seconds and 4 git commands to generate.