* section.c (bfd_section_list_remove): Don't clear s->next.

	(bfd_section_list_append): Always init s->prev.
	(bfd_section_list_prepend): Define.
	(bfd_section_list_insert_after): Minor optimization.
	(bfd_section_removed_from_list): Rewrite.
	* elf.c (assign_section_numbers): Simplify list traversal now that
	bfd_section_list_remove doesn't destroy removed section next ptr.
	* sunos.c (sunos_add_dynamic_symbols): Likewise.
	* elfxx-ia64.c (elfNN_ia64_object_p): Use bfd_section_list_prepend.
	* xcofflink.c (_bfd_xcoff_bfd_final_link): Simplify list traversal.
	* bfd-in2.h: Regenerate.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index e99fd75da0adfc81babcc9f56bc2a88767fb015d..8f403c22af9ed9816fb92099bbde717be67b9886 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,17 @@
+2005-05-04  Alan Modra  <amodra@bigpond.net.au>
+
+       * section.c (bfd_section_list_remove): Don't clear s->next.
+       (bfd_section_list_append): Always init s->prev.
+       (bfd_section_list_prepend): Define.
+       (bfd_section_list_insert_after): Minor optimization.
+       (bfd_section_removed_from_list): Rewrite.
+       * elf.c (assign_section_numbers): Simplify list traversal now that
+       bfd_section_list_remove doesn't destroy removed section next ptr.
+       * sunos.c (sunos_add_dynamic_symbols): Likewise.
+       * elfxx-ia64.c (elfNN_ia64_object_p): Use bfd_section_list_prepend.
+       * xcofflink.c (_bfd_xcoff_bfd_final_link): Simplify list traversal.
+       * bfd-in2.h: Regenerate.
+
 2005-05-02  H.J. Lu  <hongjiu.lu@intel.com>
 
        * bfd.c (bfd): Remove section_tail and add section_last.

diff --git a/bfd/bfd-in2.h b/bfd/bfd-in2.h
index 545e442763469b426c30ae284002cc92bbdec60e..944a33a1a391846054ed8a7084dc020eee243141 100644 (file)
--- a/bfd/bfd-in2.h
+++ b/bfd/bfd-in2.h
@@ -1451,10 +1451,7 @@ extern const struct bfd_symbol * const bfd_ind_symbol;
       else                                             \
         (ABFD)->sections = _next;                      \
       if (_next)                                       \
-        {                                              \
-          _next->prev = _prev;                         \
-          _s->next = NULL;                             \
-        }                                              \
+        _next->prev = _prev;                           \
       else                                             \
         (ABFD)->section_last = _prev;                  \
     }                                                  \
@@ -1471,10 +1468,32 @@ extern const struct bfd_symbol * const bfd_ind_symbol;
           _abfd->section_last->next = _s;              \
         }                                              \
       else                                             \
-        _abfd->sections = _s;                          \
+        {                                              \
+          _s->prev = NULL;                             \
+          _abfd->sections = _s;                        \
+        }                                              \
       _abfd->section_last = _s;                        \
     }                                                  \
   while (0)
+#define bfd_section_list_prepend(ABFD, S) \
+  do                                                   \
+    {                                                  \
+      asection *_s = S;                                \
+      bfd *_abfd = ABFD;                               \
+      _s->prev = NULL;                                 \
+      if (_abfd->sections)                             \
+        {                                              \
+          _s->next = _abfd->sections;                  \
+          _abfd->sections->prev = _s;                  \
+        }                                              \
+      else                                             \
+        {                                              \
+          _s->next = NULL;                             \
+          _abfd->section_last = _s;                    \
+        }                                              \
+      _abfd->sections = _s;                            \
+    }                                                  \
+  while (0)
 #define bfd_section_list_insert_after(ABFD, A, S) \
   do                                                   \
     {                                                  \
@@ -1485,7 +1504,7 @@ extern const struct bfd_symbol * const bfd_ind_symbol;
       _s->prev = _a;                                   \
       _a->next = _s;                                   \
       if (_next)                                       \
-        _s->next->prev = _s;                           \
+        _next->prev = _s;                              \
       else                                             \
         (ABFD)->section_last = _s;                     \
     }                                                  \
@@ -1506,7 +1525,7 @@ extern const struct bfd_symbol * const bfd_ind_symbol;
     }                                                  \
   while (0)
 #define bfd_section_removed_from_list(ABFD, S) \
-  ((S)->next == NULL && (S) != (ABFD)->section_last)
+  ((S)->next == NULL ? (ABFD)->section_last != (S) : (S)->next->prev != (S))
 
 void bfd_section_list_clear (bfd *);
 

diff --git a/bfd/elf.c b/bfd/elf.c
index fac05e0d886f14e3cf12be00f9fc9e3eac4e01b7..a4699326d312585cf3513097c25fd280919d3c60 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -2762,14 +2762,11 @@ assign_section_numbers (bfd *abfd, struct bfd_link_info *link_info)
   /* SHT_GROUP sections are in relocatable files only.  */
   if (link_info == NULL || link_info->relocatable)
     {
-      asection *n;
-
       /* Put SHT_GROUP sections first.  */
-      for (sec = abfd->sections; sec; sec = n)
+      for (sec = abfd->sections; sec != NULL; sec = sec->next)
        {
          d = elf_section_data (sec);
 
-         n = sec->next;
          if (d->this_hdr.sh_type == SHT_GROUP)
            { 
              if (sec->flags & SEC_LINKER_CREATED)

diff --git a/bfd/elfxx-ia64.c b/bfd/elfxx-ia64.c
index a483e2d193c1002a131de477dd7c8b9172e3d262..5c3521c014ab65e0b58f57862451ccaf291f49ba 100644 (file)
--- a/bfd/elfxx-ia64.c
+++ b/bfd/elfxx-ia64.c
@@ -4934,7 +4934,7 @@ elfNN_ia64_object_p (bfd *abfd)
 
          /* Move the fake group section to the beginning.  */
          bfd_section_list_remove (abfd, group);
-         bfd_section_list_insert_before (abfd, abfd->sections, group);
+         bfd_section_list_prepend (abfd, group);
 
          elf_next_in_group (group) = sec;
 

diff --git a/bfd/section.c b/bfd/section.c
index 45ede06fc62e8ee1ec533c4e795e9f703e9c5b15..3f008ea1909761719e3fb09cf614164eed9e7e09 100644 (file)
--- a/bfd/section.c
+++ b/bfd/section.c
@@ -552,10 +552,7 @@ CODE_FRAGMENT
 .      else                                            \
 .        (ABFD)->sections = _next;                     \
 .      if (_next)                                      \
-.        {                                             \
-.          _next->prev = _prev;                                \
-.          _s->next = NULL;                            \
-.        }                                             \
+.        _next->prev = _prev;                          \
 .      else                                            \
 .        (ABFD)->section_last = _prev;                 \
 .    }                                                 \
@@ -572,10 +569,32 @@ CODE_FRAGMENT
 .          _abfd->section_last->next = _s;             \
 .        }                                             \
 .      else                                            \
-.        _abfd->sections = _s;                         \
+.        {                                             \
+.          _s->prev = NULL;                            \
+.          _abfd->sections = _s;                       \
+.        }                                             \
 .      _abfd->section_last = _s;                       \
 .    }                                                 \
 .  while (0)
+.#define bfd_section_list_prepend(ABFD, S) \
+.  do                                                  \
+.    {                                                 \
+.      asection *_s = S;                               \
+.      bfd *_abfd = ABFD;                              \
+.      _s->prev = NULL;                                        \
+.      if (_abfd->sections)                            \
+.        {                                             \
+.          _s->next = _abfd->sections;                 \
+.          _abfd->sections->prev = _s;                 \
+.        }                                             \
+.      else                                            \
+.        {                                             \
+.          _s->next = NULL;                            \
+.          _abfd->section_last = _s;                   \
+.        }                                             \
+.      _abfd->sections = _s;                           \
+.    }                                                 \
+.  while (0)
 .#define bfd_section_list_insert_after(ABFD, A, S) \
 .  do                                                  \
 .    {                                                 \
@@ -586,7 +605,7 @@ CODE_FRAGMENT
 .      _s->prev = _a;                                  \
 .      _a->next = _s;                                  \
 .      if (_next)                                      \
-.        _s->next->prev = _s;                          \
+.        _next->prev = _s;                             \
 .      else                                            \
 .        (ABFD)->section_last = _s;                    \
 .    }                                                 \
@@ -607,7 +626,7 @@ CODE_FRAGMENT
 .    }                                                 \
 .  while (0)
 .#define bfd_section_removed_from_list(ABFD, S) \
-.  ((S)->next == NULL && (S) != (ABFD)->section_last)
+.  ((S)->next == NULL ? (ABFD)->section_last != (S) : (S)->next->prev != (S))
 .
 */
 

diff --git a/bfd/sunos.c b/bfd/sunos.c
index 1b17d3eacdae78adac15fbf763d1815b9b68ca20..5995f162fef68fcf70c05c4bb42e69e274065698 100644 (file)
--- a/bfd/sunos.c
+++ b/bfd/sunos.c
@@ -859,11 +859,10 @@ sunos_add_dynamic_symbols (bfd *abfd,
     abfd->sections = NULL;
   else
     {
-      asection *s, *n;
+      asection *s;
 
-      for (s = abfd->sections; s != NULL; s = n)
+      for (s = abfd->sections; s != NULL; s = s->next)
        {
-         n = s->next;
          if ((s->flags & SEC_LINKER_CREATED) == 0)
            bfd_section_list_remove (abfd, s);
        }

diff --git a/bfd/xcofflink.c b/bfd/xcofflink.c
index 127ac3d67d525a4793bcedd41811f8e862d34ad1..c72507edfa9bd7c75fb32d1171e9619ab4b69caa 100644 (file)
--- a/bfd/xcofflink.c
+++ b/bfd/xcofflink.c
@@ -5436,19 +5436,18 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
        {
          bfd_boolean saw_contents;
          int indx;
-         asection **op;
          file_ptr sofar;
 
          /* Insert .pad sections before every section which has
             contents and is loaded, if it is preceded by some other
             section which has contents and is loaded.  */
          saw_contents = TRUE;
-         for (op = &abfd->sections; *op != NULL; op = &(*op)->next)
+         for (o = abfd->sections; o != NULL; o = o->next)
            {
-             if (strcmp ((*op)->name, ".pad") == 0)
+             if (strcmp (o->name, ".pad") == 0)
                saw_contents = FALSE;
-             else if (((*op)->flags & SEC_HAS_CONTENTS) != 0
-                      && ((*op)->flags & SEC_LOAD) != 0)
+             else if ((o->flags & SEC_HAS_CONTENTS) != 0
+                      && (o->flags & SEC_LOAD) != 0)
                {
                  if (! saw_contents)
                    saw_contents = TRUE;
@@ -5465,9 +5464,7 @@ _bfd_xcoff_bfd_final_link (bfd *abfd, struct bfd_link_info *info)
                      n->alignment_power = 0;
 
                      bfd_section_list_remove (abfd, n);
-                     bfd_section_list_insert_before (abfd, *op, n);
-
-                     op = &n->next;
+                     bfd_section_list_insert_before (abfd, o, n);
                      saw_contents = FALSE;
                    }
                }