PR symtab/12704
authorKeith Seitz <keiths@redhat.com>
Tue, 31 May 2011 21:54:07 +0000 (21:54 +0000)
committerKeith Seitz <keiths@redhat.com>
Tue, 31 May 2011 21:54:07 +0000 (21:54 +0000)
* cp-namespace.c (ANONYMOUS_NAMESPACE_LEN): Remove.
(cp_scan_for_anonymous_namespaces): Use CP_ANONYMOUS_NAMESPACE_STR
and CP_ANONYMOUS_NAMESPACE_LEN.
(cp_is_anonymous): Likewise.
* cp-support.h (CP_ANONYMOUS_NAMESPACE_STR): Define.
(CP_ANONYMOUS_NAMESPACE_LEN): Define.
* dwarf2read.c (namespace_name): Likewise.
(fixup_partial_die): Likewise.
* linespec.c (decode_compound): If CP_ANONYMOUS_NAMESPACE_STR is
seen in the input, keep it.

gdb/ChangeLog
gdb/cp-namespace.c
gdb/cp-support.h
gdb/dwarf2read.c
gdb/linespec.c

index 5a80e276baef339a12a73876feb118b9d180409b..23975346edf1efe7bcd7e1292d066c18f9584381 100644 (file)
@@ -1,3 +1,17 @@
+2010-05-31  Keith Seitz  <keiths@redhat.com>
+
+       PR symtab/12704
+       * cp-namespace.c (ANONYMOUS_NAMESPACE_LEN): Remove.
+       (cp_scan_for_anonymous_namespaces): Use CP_ANONYMOUS_NAMESPACE_STR
+       and CP_ANONYMOUS_NAMESPACE_LEN.
+       (cp_is_anonymous): Likewise.
+       * cp-support.h (CP_ANONYMOUS_NAMESPACE_STR): Define.
+       (CP_ANONYMOUS_NAMESPACE_LEN): Define.
+       * dwarf2read.c (namespace_name): Likewise.
+       (fixup_partial_die): Likewise.
+       * linespec.c (decode_compound): If CP_ANONYMOUS_NAMESPACE_STR is
+       seen in the input, keep it.
+
 2011-05-30  Pedro Alves  <pedro@codesourcery.com>
 
        * target.h (enum inferior_event_type): Delete INF_QUIT_REQ.
index 00c68b3bb283c4236d26b4f747b9c3584bdbeab9..7ff7bf76bc0e434fbf7e0197035d923d66da6cdc 100644 (file)
@@ -52,10 +52,6 @@ static struct type *cp_lookup_transparent_type_loop (const char *name,
 /* Check to see if SYMBOL refers to an object contained within an
    anonymous namespace; if so, add an appropriate using directive.  */
 
-/* Optimize away strlen ("(anonymous namespace)").  */
-
-#define ANONYMOUS_NAMESPACE_LEN 21
-
 void
 cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
 {
@@ -76,10 +72,11 @@ cp_scan_for_anonymous_namespaces (const struct symbol *symbol)
 
       while (name[next_component] == ':')
        {
-         if ((next_component - previous_component) == ANONYMOUS_NAMESPACE_LEN
+         if (((next_component - previous_component)
+              == CP_ANONYMOUS_NAMESPACE_LEN)
              && strncmp (name + previous_component,
-                         "(anonymous namespace)",
-                         ANONYMOUS_NAMESPACE_LEN) == 0)
+                         CP_ANONYMOUS_NAMESPACE_STR,
+                         CP_ANONYMOUS_NAMESPACE_LEN) == 0)
            {
              int dest_len = (previous_component == 0
                              ? 0 : previous_component - 2);
@@ -207,7 +204,7 @@ cp_set_block_scope (const struct symbol *symbol,
 int
 cp_is_anonymous (const char *namespace)
 {
-  return (strstr (namespace, "(anonymous namespace)")
+  return (strstr (namespace, CP_ANONYMOUS_NAMESPACE_STR)
          != NULL);
 }
 
index 57aa03aa523e64bc8ff35a7e4c0dcb68f9b2bad9..52d4cb788b54898c7f4d661460c1d5c5475e3cb3 100644 (file)
@@ -36,6 +36,14 @@ struct objfile;
 struct type;
 struct demangle_component;
 
+/* A string representing the name of the anonymous namespace used in GDB.  */
+
+#define CP_ANONYMOUS_NAMESPACE_STR "(anonymous namespace)"
+
+/* The length of the string representing the anonymous namespace.  */
+
+#define CP_ANONYMOUS_NAMESPACE_LEN 21
+
 /* This struct is designed to store data from using directives.  It
    says that names from namespace IMPORT_SRC should be visible within
    namespace IMPORT_DEST.  These form a linked list; NEXT is the next
index 01a94675ea97f3495fe328f01433e7bc8b9a131b..29ca473eebdfd4bdb1946479c53ca6ebd20d7ea9 100644 (file)
@@ -7877,7 +7877,7 @@ namespace_name (struct die_info *die, int *is_anonymous, struct dwarf2_cu *cu)
 
   *is_anonymous = (name == NULL);
   if (*is_anonymous)
-    name = "(anonymous namespace)";
+    name = CP_ANONYMOUS_NAMESPACE_STR;
 
   return name;
 }
@@ -9547,7 +9547,7 @@ fixup_partial_die (struct partial_die_info *part_die,
   /* Set default names for some unnamed DIEs.  */
 
   if (part_die->name == NULL && part_die->tag == DW_TAG_namespace)
-    part_die->name = "(anonymous namespace)";
+    part_die->name = CP_ANONYMOUS_NAMESPACE_STR;
 
   /* If there is no parent die to provide a namespace, and there are
      children, see if we can determine the namespace from their linkage
index dcb80ff3823ec8ec3e6c1dcae54b05d162eab284..8ee721b239948fbd6cca3984ec6fe19efe401217 100644 (file)
@@ -1430,8 +1430,15 @@ decode_compound (char **argptr, int funfirstline,
            {
              /* PASS2: We'll keep getting here, until P points to one of the
                 break characters, at which point we exit this loop.  */
-             if (*p && strchr (break_characters, *p) == NULL)
-               p++;
+             if (*p)
+               {
+                 if (p[1] == '('
+                     && strncmp (&p[1], CP_ANONYMOUS_NAMESPACE_STR,
+                                 CP_ANONYMOUS_NAMESPACE_LEN) == 0)
+                   p += CP_ANONYMOUS_NAMESPACE_LEN;
+                 else if (strchr (break_characters, *p) == NULL)
+                   ++p;
+               }
            }
        }