* objfiles.h (struct objfile): Remove obsolete comment.
authorTom Tromey <tromey@redhat.com>
Thu, 8 Dec 2011 18:08:12 +0000 (18:08 +0000)
committerTom Tromey <tromey@redhat.com>
Thu, 8 Dec 2011 18:08:12 +0000 (18:08 +0000)
* objfiles.c (build_objfile_section_table): Remove obsolete
comment.

gdb/ChangeLog
gdb/objfiles.c
gdb/objfiles.h

index 855bc97e94f430baeb849f337be8b796705eba9e..5b1de6407b924bbf7a569b9c38aac1eac4708900 100644 (file)
@@ -1,3 +1,9 @@
+2011-12-08  Tom Tromey  <tromey@redhat.com>
+
+       * objfiles.h (struct objfile): Remove obsolete comment.
+       * objfiles.c (build_objfile_section_table): Remove obsolete
+       comment.
+
 2011-12-07  Stan Shebs  <stan@codesourcery.com>
 
        * MAINTAINERS (Responsible Maintainers): Add Yao Qi as
index 34d64225c457d5511ed0780feb8d0ccb05d6abb6..fb264203388da5922f3292fb45a67b35c4277081 100644 (file)
@@ -162,12 +162,6 @@ add_to_objfile_sections (struct bfd *abfd, struct bfd_section *asect,
 int
 build_objfile_section_table (struct objfile *objfile)
 {
-  /* objfile->sections can be already set when reading a mapped symbol
-     file.  I believe that we do need to rebuild the section table in
-     this case (we rebuild other things derived from the bfd), but we
-     can't free the old one (it's in the objfile_obstack).  So we just
-     waste some memory.  */
-
   objfile->sections_end = 0;
   bfd_map_over_sections (objfile->obfd,
                         add_to_objfile_sections, (void *) objfile);
index 742da5d3c5163917adb61d733220ea546076e3a1..d29fb4d03b1148858d53ce116141a304da0aa15a 100644 (file)
@@ -173,19 +173,9 @@ struct objfile
   {
 
     /* All struct objfile's are chained together by their next pointers.
-       The global variable "object_files" points to the first link in this
-       chain.
-
-       FIXME:  There is a problem here if the objfile is reusable, and if
-       multiple users are to be supported.  The problem is that the objfile
-       list is linked through a member of the objfile struct itself, which
-       is only valid for one gdb process.  The list implementation needs to
-       be changed to something like:
-
-       struct list {struct list *next; struct objfile *objfile};
-
-       where the list structure is completely maintained separately within
-       each gdb process.  */
+       The program space field "objfiles"  (frequently referenced via
+       the macro "object_files") points to the first link in this
+       chain.  */
 
     struct objfile *next;