[rfa+5.3] bug in my earlier DW_TAG_namespace patch

[David Carlton <carlton@math.stanford.edu>]

I just noticed that my earlier patch that allowed GDB to accept
DW_TAG_namespace entries had a bug in it.  The function
scan_partial_symbols in dwarf2read.c only descends into DIEs that have
a name; but anonymous namespaces can lead to DIEs without names which
have interesting children (where by "interesting" I mean that we want
to add partial symbols corresponding to them).  This could result in
us missing some partial symbols.

I've enclosed a patch below: for obvious reasons, it should go into
GDB 5.3 as well.  I've run GDB on a copy of GDB with this patch
applied and followed the control flow when GDB runs into various sorts
of DW_TAG_namespace entries, and the control flow seems to behave like
I expect it to.  (This is, admittedly, after only a limited amount of
testing.)  I'm in the middle of running the testsuite (I'll run it
twice, once on a compiler that doesn't generate DW_TAG_namespace
entries and once on a compiler that does); assuming that there are no
new regressions, is the patch okay?

David Carlton
carlton@math.stanford.edu

2002-11-21  David Carlton  <carlton@math.stanford.edu>

	* dwarf2read.c (scan_partial_symbols): Descend into namespace
	pdi's with no name.

Index: dwarf2read.c
===================================================================
RCS file: /cvs/src/src/gdb/dwarf2read.c,v
retrieving revision 1.75
diff -u -p -r1.75 dwarf2read.c
--- dwarf2read.c	11 Nov 2002 00:55:34 -0000	1.75
+++ dwarf2read.c	22 Nov 2002 00:11:49 -0000
@@ -1359,7 +1359,9 @@ scan_partial_symbols (char *info_ptr, st
     {
       info_ptr = read_partial_die (&pdi, abfd, info_ptr, cu_header);
 
-      if (pdi.name)
+      /* Anonymous namespaces have no name but are interesting.  */
+
+      if (pdi.name != NULL || pdi.tag == DW_TAG_namespace)
 	{
 	  switch (pdi.tag)
 	    {