From: Tristan Gingold <gingold@adacore.com>
Date: Tue, 9 Feb 2010 10:59:50 +0000 (+0000)
Subject: 2010-02-08  Tristan Gingold  <gingold@adacore.com>
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=399f313b12b98f268b6c95967d473dfd64c72df6;p=binutils-gdb.git

2010-02-08  Tristan Gingold  <gingold@adacore.com>

	* objfiles.c (objfile_separate_debug_iterate): Do not iterate on
	brothers of the parent.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0df0ae00a79..ceb22c0c946 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2010-02-09  Tristan Gingold  <gingold@adacore.com>
+
+	* objfiles.c (objfile_separate_debug_iterate): Do not iterate on
+	brothers of the parent.
+
 2010-02-08  Tom Tromey  <tromey@redhat.com>
 
 	PR c++/8017:
diff --git a/gdb/objfiles.c b/gdb/objfiles.c
index 0b07e370916..489b81289f0 100644
--- a/gdb/objfiles.c
+++ b/gdb/objfiles.c
@@ -392,18 +392,21 @@ objfile_separate_debug_iterate (const struct objfile *parent,
 {
   struct objfile *res;
 
+  /* If any, return the first child.  */
   res = objfile->separate_debug_objfile;
   if (res)
     return res;
 
-  res = objfile->separate_debug_objfile_link;
-  if (res)
-    return res;
-
   /* Common case where there is no separate debug objfile.  */
   if (objfile == parent)
     return NULL;
 
+  /* Return the brother if any.  Note that we don't iterate on brothers of
+     the parents.  */
+  res = objfile->separate_debug_objfile_link;
+  if (res)
+    return res;
+
   for (res = objfile->separate_debug_objfile_backlink;
        res != parent;
        res = res->separate_debug_objfile_backlink)