use remote-utils facilities for baud_rate
[deliverable/binutils-gdb.git] / gdb / symtab.c
index e4678b86b47600ac0b8e1807478aa8fd5da8adfb..d43cdf88698c97b01d51d65d574a3121582c9e5b 100644 (file)
@@ -45,17 +45,17 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 /* Prototypes for local functions */
 
-static char *
-expensive_mangler PARAMS ((const char *));
-
 extern int
-find_methods PARAMS ((struct type *, char *, char **, struct symbol **));
+find_methods PARAMS ((struct type *, char *, struct symbol **));
+
+static void
+completion_list_add_name PARAMS ((char *, char *, int, char *, char *));
 
 static void
-completion_list_add_symbol PARAMS ((char *, char *, int));
+build_canonical_line_spec PARAMS ((struct symtab_and_line *, char *, char ***));
 
 static struct symtabs_and_lines
-decode_line_2 PARAMS ((struct symbol *[], int, int));
+decode_line_2 PARAMS ((struct symbol *[], int, int, char ***));
 
 static void
 rbreak_command PARAMS ((char *, int));
@@ -81,20 +81,12 @@ output_source_filename PARAMS ((char *, int *));
 static char *
 operator_chars PARAMS ((char *, char **));
 
-static int
-find_line_common PARAMS ((struct linetable *, int, int *));
+static int find_line_common PARAMS ((struct linetable *, int, int *));
 
 static struct partial_symbol *
 lookup_partial_symbol PARAMS ((struct partial_symtab *, const char *,
                               int, enum namespace));
 
-static struct partial_symbol *
-lookup_demangled_partial_symbol PARAMS ((const struct partial_symtab *,
-                                        const char *));
-
-static struct symbol *
-lookup_demangled_block_symbol PARAMS ((const struct block *, const char *));
-
 static struct symtab *
 lookup_symtab_1 PARAMS ((char *));
 
@@ -134,63 +126,75 @@ lookup_symtab_1 (name)
   register struct symtab *s;
   register struct partial_symtab *ps;
   register char *slash;
-  register int len;
   register struct objfile *objfile;
 
-  ALL_SYMTABS (objfile, s)
-    {
-      if (strcmp (name, s->filename) == 0)
-       {
-         return (s);
-       }
-    }
+ got_symtab:
 
-  ALL_PSYMTABS (objfile, ps)
-    {
-      if (strcmp (name, ps -> filename) == 0)
-       {
-         if (ps -> readin)
-           {
-             error ("Internal: readin pst for `%s' found when no symtab found.", name);
-           }
-         return (PSYMTAB_TO_SYMTAB (ps));
-       }
-    }
+  /* First, search for an exact match */
+
+  ALL_SYMTABS (objfile, s)
+    if (STREQ (name, s->filename))
+      return s;
 
   slash = strchr (name, '/');
-  len = strlen (name);
+
+  /* Now, search for a matching tail (only if name doesn't have any dirs) */
 
   if (!slash)
-    {
-      ALL_SYMTABS (objfile, s)
-       {
-         int l = strlen (s->filename);
-         
-         if (l > len
-             && s->filename[l - len -1] == '/'
-             && (strcmp (s->filename + l - len, name) == 0))
-           {
-             return (s);
-           }
-       }
+    ALL_SYMTABS (objfile, s)
+      {
+       char *p = s -> filename;
+       char *tail = strrchr (p, '/');
 
-      ALL_PSYMTABS (objfile, ps)
-       {
-         int l = strlen (ps -> filename);
+       if (tail)
+         p = tail + 1;
+
+       if (STREQ (p, name))
+         return s;
+      }
+
+  /* Same search rules as above apply here, but now we look thru the
+     psymtabs.  */
+
+  ALL_PSYMTABS (objfile, ps)
+    if (STREQ (name, ps -> filename))
+      goto got_psymtab;
+
+  if (!slash)
+    ALL_PSYMTABS (objfile, ps)
+      {
+       char *p = ps -> filename;
+       char *tail = strrchr (p, '/');
+
+       if (tail)
+         p = tail + 1;
+
+       if (STREQ (p, name))
+         goto got_psymtab;
+      }
 
-         if (l > len
-             && ps -> filename[l - len - 1] == '/'
-             && (strcmp (ps->filename + l - len, name) == 0))
-           {
-             if (ps -> readin)
-               {
-                 error ("Internal: readin pst for `%s' found when no symtab found.", name);
-               }
-             return (PSYMTAB_TO_SYMTAB (ps));
-           }
-       }
-    }
   return (NULL);
+
+ got_psymtab:
+
+  if (ps -> readin)
+    error ("Internal: readin %s pst for `%s' found when no symtab found.",
+          ps -> filename, name);
+
+  s = PSYMTAB_TO_SYMTAB (ps);
+
+  if (s)
+    return s;
+
+  /* At this point, we have located the psymtab for this file, but
+     the conversion to a symtab has failed.  This usually happens
+     when we are looking up an include file.  In this case,
+     PSYMTAB_TO_SYMTAB doesn't return a symtab, even though one has
+     been created.  So, we need to run through the symtabs again in
+     order to find the file.
+     XXX - This is a crock, and should be fixed inside of the the
+     symbol parsing routines. */
+  goto got_symtab;
 }
 
 /* Lookup the symbol table of a source file named NAME.  Try a couple
@@ -231,7 +235,7 @@ char *name;
   
   ALL_PSYMTABS (objfile, pst)
     {
-      if (strcmp (name, pst -> filename) == 0)
+      if (STREQ (name, pst -> filename))
        {
          return (pst);
        }
@@ -239,7 +243,9 @@ char *name;
   return (NULL);
 }
 \f
-/* Demangle a GDB method stub type.  */
+/* Demangle a GDB method stub type.
+   Note that this function is g++ specific. */
+
 char *
 gdb_mangle_name (type, i, j)
      struct type *type;
@@ -250,19 +256,28 @@ gdb_mangle_name (type, i, j)
   struct fn_field *f = TYPE_FN_FIELDLIST1 (type, i);
   struct fn_field *method = &f[j];
   char *field_name = TYPE_FN_FIELDLIST_NAME (type, i);
-  int is_constructor = strcmp(field_name, TYPE_NAME (type)) == 0;
-
+  char *physname = TYPE_FN_FIELD_PHYSNAME (f, j);
+  char *newname = type_name_no_tag (type);
+  int is_constructor = newname != NULL && STREQ (field_name, newname);
+  int is_destructor = is_constructor && DESTRUCTOR_PREFIX_P (physname);
   /* Need a new type prefix.  */
   char *const_prefix = method->is_const ? "C" : "";
   char *volatile_prefix = method->is_volatile ? "V" : "";
-  char *newname = type_name_no_tag (type);
   char buf[20];
-  int len = strlen (newname);
+#ifndef GCC_MANGLE_BUG
+  int len = newname == NULL ? 0 : strlen (newname);
+
+  if (is_destructor)
+    {
+      mangled_name = (char*) xmalloc(strlen(physname)+1);
+      strcpy(mangled_name, physname);
+      return mangled_name;
+    }
 
   sprintf (buf, "__%s%s%d", const_prefix, volatile_prefix, len);
   mangled_name_len = ((is_constructor ? 0 : strlen (field_name))
                          + strlen (buf) + len
-                         + strlen (TYPE_FN_FIELD_PHYSNAME (f, j))
+                         + strlen (physname)
                          + 1);
 
   /* Only needed for GNU-mangled names.  ANSI-mangled names
@@ -288,10 +303,58 @@ gdb_mangle_name (type, i, j)
        strcpy (mangled_name, field_name);
     }
   strcat (mangled_name, buf);
-  strcat (mangled_name, newname);
-  strcat (mangled_name, TYPE_FN_FIELD_PHYSNAME (f, j));
+  /* If the class doesn't have a name, i.e. newname NULL, then we just
+     mangle it using 0 for the length of the class.  Thus it gets mangled
+     as something starting with `::' rather than `classname::'.  */
+  if (newname != NULL)
+    strcat (mangled_name, newname);
+#else
+  char *opname;
+
+  if (is_constructor)
+    {
+      buf[0] = '\0';
+    }
+  else
+    {
+      sprintf (buf, "__%s%s", const_prefix, volatile_prefix);
+    }
+
+  mangled_name_len = ((is_constructor ? 0 : strlen (field_name))
+                     + strlen (buf) + strlen (physname) + 1);
 
-  return mangled_name;
+  /* Only needed for GNU-mangled names.  ANSI-mangled names
+     work with the normal mechanisms.  */
+  if (OPNAME_PREFIX_P (field_name))
+    {
+      opname = cplus_mangle_opname (field_name + 3, 0);
+      if (opname == NULL)
+       {
+         error ("No mangling for \"%s\"", field_name);
+       }
+      mangled_name_len += strlen (opname);
+      mangled_name = (char *) xmalloc (mangled_name_len);
+
+      strncpy (mangled_name, field_name, 3);
+      strcpy (mangled_name + 3, opname);
+    }
+  else
+    {
+      mangled_name = (char *) xmalloc (mangled_name_len);
+      if (is_constructor)
+       {
+         mangled_name[0] = '\0';
+       }
+      else
+       {
+         strcpy (mangled_name, field_name);
+       }
+    }
+  strcat (mangled_name, buf);
+
+#endif
+  strcat (mangled_name, physname);
+  return (mangled_name);
 }
 
 \f
@@ -306,10 +369,8 @@ find_pc_psymtab (pc)
 
   ALL_PSYMTABS (objfile, pst)
     {
-      if (pc >= pst -> textlow && pc < pst -> texthigh)
-       {
-         return (pst);
-       }
+      if (pc >= pst->textlow && pc < pst->texthigh)
+       return (pst);
     }
   return (NULL);
 }
@@ -375,20 +436,6 @@ lookup_symbol (name, block, namespace, is_a_field_of_this, symtab)
   register struct objfile *objfile;
   register struct block *b;
   register struct minimal_symbol *msymbol;
-  char *temp;
-  extern char *gdb_completer_word_break_characters;
-
-  /* If NAME contains any characters from gdb_completer_word_break_characters
-     then it is probably from a quoted name string.  So check to see if it
-     has a C++ mangled equivalent, and if so, use the mangled equivalent. */
-
-  if (strpbrk (name, gdb_completer_word_break_characters) != NULL)
-    {
-      if ((temp = expensive_mangler (name)) != NULL)
-       {
-         name = temp;
-       }
-    }
 
   /* Search specified block and its superiors.  */
 
@@ -419,11 +466,11 @@ found:
       block = BLOCK_SUPERBLOCK (block);
     }
 
-  /* But that doesn't do any demangling for the STATIC_BLOCK.
-     I'm not sure whether demangling is needed in the case of
-     nested function in inner blocks; if so this needs to be changed.
-     
-     Don't need to mess with the psymtabs; if we have a block,
+  /* FIXME: this code is never executed--block is always NULL at this
+     point.  What is it trying to do, anyway?  We already should have
+     checked the STATIC_BLOCK above (it is the superblock of top-level
+     blocks).  Why is VAR_NAMESPACE special-cased?  */
+  /* Don't need to mess with the psymtabs; if we have a block,
      that file is read in.  If we don't, then we deal later with
      all the psymtab stuff that needs checking.  */
   if (namespace == VAR_NAMESPACE && block != NULL)
@@ -437,7 +484,7 @@ found:
          if (BLOCK_START (b) <= BLOCK_START (block)
              && BLOCK_END (b) > BLOCK_START (block))
            {
-             sym = lookup_demangled_block_symbol (b, name);
+             sym = lookup_block_symbol (b, name, VAR_NAMESPACE);
              if (sym)
                {
                  block_found = b;
@@ -490,42 +537,22 @@ found:
   if (namespace == VAR_NAMESPACE)
     {
       msymbol = lookup_minimal_symbol (name, (struct objfile *) NULL);
-
-      if (msymbol == NULL)
-       {
-         /* Test each minimal symbol to see if the minimal symbol's name
-            is a C++ mangled name that matches a user visible name.  */
-
-         char *demangled;
-
-         ALL_MSYMBOLS (objfile, msymbol)
-           {
-             demangled = demangle_and_match (msymbol -> name, name, 0);
-             if (demangled != NULL)
-               {
-                 free (demangled);
-                 goto found_msym;
-               }
-           }
-         msymbol = NULL;               /* Not found */
-        }
-
-found_msym:
       if (msymbol != NULL)
        {
-         s = find_pc_symtab (msymbol -> address);
+         s = find_pc_symtab (SYMBOL_VALUE_ADDRESS (msymbol));
          /* If S is NULL, there are no debug symbols for this file.
             Skip this stuff and check for matching static symbols below. */
          if (s != NULL)
            {
              bv = BLOCKVECTOR (s);
              block = BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK);
-             sym = lookup_block_symbol (block, msymbol -> name, namespace);
+             sym = lookup_block_symbol (block, SYMBOL_NAME (msymbol),
+                                        namespace);
               /* We kept static functions in minimal symbol table as well as
                 in static scope. We want to find them in the symbol table. */
                if (!sym) {
                  block = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK);
-                 sym = lookup_block_symbol (block, msymbol -> name,
+                 sym = lookup_block_symbol (block, SYMBOL_NAME (msymbol),
                                             namespace);
                }
 
@@ -558,7 +585,7 @@ found_msym:
          block = BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK);
          sym = lookup_block_symbol (block, name, namespace);
          if (!sym)
-           error ("Internal: global symbol `%s' found in psymtab but not in symtab", name);
+           error ("Internal: global symbol `%s' found in %s psymtab but not in symtab", name, ps->filename);
          if (symtab != NULL)
            *symtab = s;
          return sym;
@@ -592,7 +619,7 @@ found_msym:
          block = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK);
          sym = lookup_block_symbol (block, name, namespace);
          if (!sym)
-           error ("Internal: static symbol `%s' found in psymtab but not in symtab", name);
+           error ("Internal: static symbol `%s' found in %s psymtab but not in symtab", name, ps->filename);
          if (symtab != NULL)
            *symtab = s;
          return sym;
@@ -608,7 +635,7 @@ found_msym:
        {
          bv = BLOCKVECTOR (s);
          block = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK);
-         sym = lookup_demangled_block_symbol (block, name);
+         sym = lookup_block_symbol (block, name, VAR_NAMESPACE);
          if (sym) 
            {
              block_found = block;
@@ -620,14 +647,14 @@ found_msym:
 
       ALL_PSYMTABS (objfile, ps)
        {
-         if (!ps->readin && lookup_demangled_partial_symbol (ps, name))
+         if (!ps->readin && lookup_partial_symbol (ps, name, 0, VAR_NAMESPACE))
            {
              s = PSYMTAB_TO_SYMTAB(ps);
              bv = BLOCKVECTOR (s);
              block = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK);
-             sym = lookup_demangled_block_symbol (block, name);
+             sym = lookup_block_symbol (block, name, VAR_NAMESPACE);
              if (!sym)
-               error ("Internal: mangled static symbol `%s' found in psymtab but not in symtab", name);
+               error ("Internal: mangled static symbol `%s' found in %s psymtab but not in symtab", name, ps->filename);
              if (symtab != NULL)
                *symtab = s;
              return sym;
@@ -640,69 +667,6 @@ found_msym:
   return 0;
 }
 
-/* Look for a static demangled symbol in block BLOCK.  */
-
-static struct symbol *
-lookup_demangled_block_symbol (block, name)
-     register const struct block *block;
-     const char *name;
-{
-  register int bot, top;
-  register struct symbol *sym;
-  char *demangled;
-
-  bot = 0;
-  top = BLOCK_NSYMS (block);
-
-  while (bot < top)
-    {
-      sym = BLOCK_SYM (block, bot);
-      if (SYMBOL_NAMESPACE (sym) == VAR_NAMESPACE)
-       {
-         demangled = demangle_and_match (SYMBOL_NAME (sym), name, 0);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (sym);
-           }
-       }
-      bot++;
-    }
-
-  return (NULL);
-}
-
-/* Look, in partial_symtab PST, for static mangled symbol NAME. */
-
-static struct partial_symbol *
-lookup_demangled_partial_symbol (pst, name)
-     const struct partial_symtab *pst;
-     const char *name;
-{
-  struct partial_symbol *start, *psym;
-  int length = pst->n_static_syms;
-  char *demangled;
-
-  if (!length)
-    return (struct partial_symbol *) 0;
-  
-  start = pst->objfile->static_psymbols.list + pst->statics_offset;
-  for (psym = start; psym < start + length; psym++)
-    {
-      if (SYMBOL_NAMESPACE (psym) == VAR_NAMESPACE)
-       {
-         demangled = demangle_and_match (SYMBOL_NAME (psym), name, 0);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (psym);
-           }
-       }
-    }
-
-  return (NULL);
-}
-
 /* Look, in partial_symtab PST, for symbol NAME.  Check the global
    symbols if GLOBAL, the static symbols if not */
 
@@ -714,59 +678,81 @@ lookup_partial_symbol (pst, name, global, namespace)
      enum namespace namespace;
 {
   struct partial_symbol *start, *psym;
+  struct partial_symbol *top, *bottom, *center;
   int length = (global ? pst->n_global_syms : pst->n_static_syms);
+  int do_linear_search = 1;
 
-  if (!length)
-    return (struct partial_symbol *) 0;
+  if (length == 0)
+    {
+      return (NULL);
+    }
   
   start = (global ?
           pst->objfile->global_psymbols.list + pst->globals_offset :
           pst->objfile->static_psymbols.list + pst->statics_offset  );
 
-  if (global)                  /* This means we can use a binary */
-                               /* search.  */
+  if (global)          /* This means we can use a binary search. */
     {
-      struct partial_symbol *top, *bottom, *center;
+      do_linear_search = 0;
 
       /* Binary search.  This search is guaranteed to end with center
          pointing at the earliest partial symbol with the correct
         name.  At that point *all* partial symbols with that name
         will be checked against the correct namespace. */
+
       bottom = start;
       top = start + length - 1;
       while (top > bottom)
        {
          center = bottom + (top - bottom) / 2;
-
          assert (center < top);
-         
-         if (strcmp (SYMBOL_NAME (center), name) >= 0)
-           top = center;
+         if (!do_linear_search && SYMBOL_LANGUAGE (center) == language_cplus)
+           {
+             do_linear_search = 1;
+           }
+         if (STRCMP (SYMBOL_NAME (center), name) >= 0)
+           {
+             top = center;
+           }
          else
-           bottom = center + 1;
+           {
+             bottom = center + 1;
+           }
        }
       assert (top == bottom);
-      
-      while (!strcmp (SYMBOL_NAME (top), name))
+      while (STREQ (SYMBOL_NAME (top), name))
        {
          if (SYMBOL_NAMESPACE (top) == namespace)
-           return top;
+           {
+             return top;
+           }
          top ++;
        }
     }
-  else
+
+  /* Can't use a binary search or else we found during the binary search that
+     we should also do a linear search. */
+
+  if (do_linear_search)
     {
-      /* Can't use a binary search */
       for (psym = start; psym < start + length; psym++)
-       if (namespace == SYMBOL_NAMESPACE (psym)
-           && !strcmp (name, SYMBOL_NAME (psym)))
-         return psym;
+       {
+         if (namespace == SYMBOL_NAMESPACE (psym))
+           {
+             if (SYMBOL_MATCHES_NAME (psym, name))
+               {
+                 return (psym);
+               }
+           }
+       }
     }
 
-  return (struct partial_symbol *) 0;
+  return (NULL);
 }
 
 /* Find the psymtab containing main(). */
+/* FIXME:  What about languages without main() or specially linked
+   executables that have no main() ? */
 
 struct partial_symtab *
 find_main_psymtab ()
@@ -784,7 +770,16 @@ find_main_psymtab ()
   return (NULL);
 }
 
-/* Look for a symbol in block BLOCK.  */
+/* Search BLOCK for symbol NAME in NAMESPACE.
+
+   Note that if NAME is the demangled form of a C++ symbol, we will fail
+   to find a match during the binary search of the non-encoded names, but
+   for now we don't worry about the slight inefficiency of looking for
+   a match we'll never find, since it will go pretty quick.  Once the
+   binary search terminates, we drop through and do a straight linear
+   search on the symbols.  Each symbol which is marked as being a C++
+   symbol (language_cplus set) has both the encoded and non-encoded names
+   tested for a match. */
 
 struct symbol *
 lookup_block_symbol (block, name, namespace)
@@ -793,42 +788,65 @@ lookup_block_symbol (block, name, namespace)
      const enum namespace namespace;
 {
   register int bot, top, inc;
-  register struct symbol *sym, *parameter_sym;
-
-  top = BLOCK_NSYMS (block);
-  bot = 0;
+  register struct symbol *sym;
+  register struct symbol *sym_found = NULL;
+  register int do_linear_search = 1;
 
   /* If the blocks's symbols were sorted, start with a binary search.  */
 
   if (BLOCK_SHOULD_SORT (block))
     {
-      /* First, advance BOT to not far before
-        the first symbol whose name is NAME.  */
+      /* Reset the linear search flag so if the binary search fails, we
+        won't do the linear search once unless we find some reason to
+        do so, such as finding a C++ symbol during the binary search.
+        Note that for C++ modules, ALL the symbols in a block should
+        end up marked as C++ symbols. */
+
+      do_linear_search = 0;
+      top = BLOCK_NSYMS (block);
+      bot = 0;
+
+      /* Advance BOT to not far before the first symbol whose name is NAME. */
 
       while (1)
        {
          inc = (top - bot + 1);
          /* No need to keep binary searching for the last few bits worth.  */
          if (inc < 4)
-           break;
+           {
+             break;
+           }
          inc = (inc >> 1) + bot;
          sym = BLOCK_SYM (block, inc);
+         if (!do_linear_search && SYMBOL_LANGUAGE (sym) == language_cplus)
+           {
+             do_linear_search = 1;
+           }
          if (SYMBOL_NAME (sym)[0] < name[0])
-           bot = inc;
+           {
+             bot = inc;
+           }
          else if (SYMBOL_NAME (sym)[0] > name[0])
-           top = inc;
-         else if (strcmp (SYMBOL_NAME (sym), name) < 0)
-           bot = inc;
+           {
+             top = inc;
+           }
+         else if (STRCMP (SYMBOL_NAME (sym), name) < 0)
+           {
+             bot = inc;
+           }
          else
-           top = inc;
+           {
+             top = inc;
+           }
        }
 
-      /* Now scan forward until we run out of symbols,
-        find one whose name is greater than NAME,
-        or find one we want.
-        If there is more than one symbol with the right name and namespace,
-        we return the first one.  dbxread.c is careful to make sure
-        that if one is a register then it comes first.  */
+      /* Now scan forward until we run out of symbols, find one whose
+        name is greater than NAME, or find one we want.  If there is
+        more than one symbol with the right name and namespace, we
+        return the first one; I believe it is now impossible for us
+        to encounter two symbols with the same name and namespace
+        here, because blocks containing argument symbols are no
+        longer sorted.  */
 
       top = BLOCK_NSYMS (block);
       while (bot < top)
@@ -836,19 +854,27 @@ lookup_block_symbol (block, name, namespace)
          sym = BLOCK_SYM (block, bot);
          inc = SYMBOL_NAME (sym)[0] - name[0];
          if (inc == 0)
-           inc = strcmp (SYMBOL_NAME (sym), name);
+           {
+             inc = STRCMP (SYMBOL_NAME (sym), name);
+           }
          if (inc == 0 && SYMBOL_NAMESPACE (sym) == namespace)
-           return sym;
+           {
+             return (sym);
+           }
          if (inc > 0)
-           return 0;
+           {
+             break;
+           }
          bot++;
        }
-      return 0;
     }
 
-  /* Here if block isn't sorted.
-     This loop is equivalent to the loop above,
-     but hacked greatly for speed.
+  /* Here if block isn't sorted, or we fail to find a match during the
+     binary search above.  If during the binary search above, we find a
+     symbol which is a C++ symbol, then we have re-enabled the linear
+     search flag which was reset when starting the binary search.
+
+     This loop is equivalent to the loop above, but hacked greatly for speed.
 
      Note that parameter symbols do not always show up last in the
      list; this loop makes sure to take anything else other than
@@ -856,28 +882,33 @@ lookup_block_symbol (block, name, namespace)
      last resort.  Note that this only takes up extra computation
      time on a match.  */
 
-  parameter_sym = (struct symbol *) 0;
-  top = BLOCK_NSYMS (block);
-  inc = name[0];
-  while (bot < top)
+  if (do_linear_search)
     {
-      sym = BLOCK_SYM (block, bot);
-      if (SYMBOL_NAME (sym)[0] == inc
-         && !strcmp (SYMBOL_NAME (sym), name)
-         && SYMBOL_NAMESPACE (sym) == namespace)
+      top = BLOCK_NSYMS (block);
+      bot = 0;
+      while (bot < top)
        {
-         if (SYMBOL_CLASS (sym) == LOC_ARG
-             || SYMBOL_CLASS (sym) == LOC_LOCAL_ARG
-             || SYMBOL_CLASS (sym) == LOC_REF_ARG
-             || SYMBOL_CLASS (sym) == LOC_REGPARM)
-           parameter_sym = sym;
-         else
-           return sym;
+         sym = BLOCK_SYM (block, bot);
+         if (SYMBOL_NAMESPACE (sym) == namespace &&
+             SYMBOL_MATCHES_NAME (sym, name))
+           {
+             sym_found = sym;
+             if (SYMBOL_CLASS (sym) != LOC_ARG &&
+                 SYMBOL_CLASS (sym) != LOC_LOCAL_ARG &&
+                 SYMBOL_CLASS (sym) != LOC_REF_ARG &&
+                 SYMBOL_CLASS (sym) != LOC_REGPARM &&
+                 SYMBOL_CLASS (sym) != LOC_REGPARM_ADDR &&
+                 SYMBOL_CLASS (sym) != LOC_BASEREG_ARG)
+               {
+                 break;
+               }
+           }
+         bot++;
        }
-      bot++;
     }
-  return parameter_sym;                /* Will be 0 if not found. */
+  return (sym_found);          /* Will be NULL if not found. */
 }
+
 \f
 /* Return the symbol for the function which contains a specified
    lexical block, described by a struct block BL.  */
@@ -892,7 +923,8 @@ block_function (bl)
   return BLOCK_FUNCTION (bl);
 }
 
-/* Subroutine of find_pc_line */
+/* Find the symtab associated with PC.  Look through the psymtabs and read in
+   another symtab if necessary. */
 
 struct symtab *
 find_pc_symtab (pc)
@@ -900,35 +932,49 @@ find_pc_symtab (pc)
 {
   register struct block *b;
   struct blockvector *bv;
-  register struct symtab *s = 0;
+  register struct symtab *s = NULL;
+  register struct symtab *best_s = NULL;
   register struct partial_symtab *ps;
   register struct objfile *objfile;
+  int distance = 0;
 
-  /* Search all symtabs for one whose file contains our pc */
+  /* Search all symtabs for the one whose file contains our address, and which
+     is the smallest of all the ones containing the address.  This is designed
+     to deal with a case like symtab a is at 0x1000-0x2000 and 0x3000-0x4000
+     and symtab b is at 0x2000-0x3000.  So the GLOBAL_BLOCK for a is from
+     0x1000-0x4000, but for address 0x2345 we want to return symtab b.
+     This is said to happen for the mips; it might be swifter to create
+     several symtabs with the same name like xcoff does (I'm not sure).  */
 
   ALL_SYMTABS (objfile, s)
     {
       bv = BLOCKVECTOR (s);
       b = BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK);
       if (BLOCK_START (b) <= pc
-         && BLOCK_END (b) > pc)
-       goto found;
-    }
-
-  if (!s)
-    {
-      ps = find_pc_psymtab (pc);
-      if (ps && ps->readin)
-       {
-         printf_filtered ("(Internal error: pc 0x%x in read in psymtab, but not in symtab.)\n", pc);
-       }
-      if (ps)
+         && BLOCK_END (b) > pc
+         && (distance == 0
+             || BLOCK_END (b) - BLOCK_START (b) < distance))
        {
-         s = PSYMTAB_TO_SYMTAB (ps);
+         distance = BLOCK_END (b) - BLOCK_START (b);
+         best_s = s;
        }
     }
 
-found:
+  if (best_s != NULL)
+    return(best_s);
+
+  s = NULL;
+  ps = find_pc_psymtab (pc);
+  if (ps)
+    {
+      if (ps->readin)
+       /* Might want to error() here (in case symtab is corrupt and
+          will cause a core dump), but maybe we can successfully
+          continue, so let's not.  */
+       warning ("\
+(Internal error: pc 0x%x in read in psymtab, but not in symtab.)\n", pc);
+      s = PSYMTAB_TO_SYMTAB (ps);
+    }
   return (s);
 }
 
@@ -940,6 +986,23 @@ found:
    use the line that ends there.  Otherwise, in that case, the line
    that begins there is used.  */
 
+/* The big complication here is that a line may start in one file, and end just
+   before the start of another file.  This usually occurs when you #include
+   code in the middle of a subroutine.  To properly find the end of a line's PC
+   range, we must search all symtabs associated with this compilation unit, and
+   find the one whose first PC is closer than that of the next line in this
+   symtab.
+
+   FIXME:  We used to complain here about zero length or negative length line
+   tables, but there are two problems with this: (1) some symtabs may not have
+   any line numbers due to gcc -g1 compilation, and (2) this function is called
+   during single stepping, when we don't own the terminal and thus can't
+   produce any output.  One solution might be to implement a mechanism whereby
+   complaints can be queued until we regain control of the terminal.  -fnf
+ */
+
+/* If it's worth the effort, we could be using a binary search.  */
+
 struct symtab_and_line
 find_pc_line (pc, notcurrent)
      CORE_ADDR pc;
@@ -955,8 +1018,7 @@ find_pc_line (pc, notcurrent)
 
   /* Info on best line seen so far, and where it starts, and its file.  */
 
-  int best_line = 0;
-  CORE_ADDR best_pc = 0;
+  struct linetable_entry *best = NULL;
   CORE_ADDR best_end = 0;
   struct symtab *best_symtab = 0;
 
@@ -965,19 +1027,12 @@ find_pc_line (pc, notcurrent)
      If we don't find a line whose range contains PC,
      we will use a line one less than this,
      with a range from the start of that file to the first line's pc.  */
-  int alt_line = 0;
-  CORE_ADDR alt_pc = 0;
+  struct linetable_entry *alt = NULL;
   struct symtab *alt_symtab = 0;
 
   /* Info on best line seen in this file.  */
 
-  int prev_line;
-  CORE_ADDR prev_pc;
-
-  /* Info on first line of this file.  */
-
-  int first_line;
-  CORE_ADDR first_pc;
+  struct linetable_entry *prev;
 
   /* If this pc is not from the current frame,
      it is the address of the end of a call instruction.
@@ -988,7 +1043,7 @@ find_pc_line (pc, notcurrent)
   if (notcurrent) pc -= 1;
 
   s = find_pc_symtab (pc);
-  if (s == 0)
+  if (!s)
     {
       val.symtab = 0;
       val.line = 0;
@@ -1010,71 +1065,176 @@ find_pc_line (pc, notcurrent)
       if (!l)
         continue;
       len = l->nitems;
-      prev_line = -1;
-      first_line = -1;
-      for (i = 0; i < len; i++)
+      if (len <= 0)              /* See FIXME above. */
+       {
+         continue;
+       }
+
+      prev = NULL;
+      item = l->item;          /* Get first line info */
+
+      /* Is this file's first line closer than the first lines of other files?
+        If so, record this file, and its first line, as best alternate.  */
+      if (item->pc > pc && (!alt || item->pc < alt->pc))
+       {
+         alt = item;
+         alt_symtab = s;
+       }
+
+      for (i = 0; i < len; i++, item++)
        {
-         item = &(l->item[i]);
-         
-         if (first_line < 0)
-           {
-             first_line = item->line;
-             first_pc = item->pc;
-           }
          /* Return the last line that did not start after PC.  */
-         if (pc >= item->pc)
-           {
-             prev_line = item->line;
-             prev_pc = item->pc;
-           }
-         else
+         if (item->pc > pc)
            break;
+
+         prev = item;
        }
 
+      /* At this point, prev points at the line whose start addr is <= pc, and
+        item points at the next line.  If we ran off the end of the linetable
+        (pc >= start of the last line), then prev == item.  If pc < start of
+        the first line, prev will not be set.  */
+
       /* Is this file's best line closer than the best in the other files?
         If so, record this file, and its best line, as best so far.  */
-      if (prev_line >= 0 && prev_pc > best_pc)
+
+      if (prev && (!best || prev->pc > best->pc))
        {
-         best_pc = prev_pc;
-         best_line = prev_line;
+         best = prev;
          best_symtab = s;
          /* If another line is in the linetable, and its PC is closer
             than the best_end we currently have, take it as best_end.  */
          if (i < len && (best_end == 0 || best_end > item->pc))
            best_end = item->pc;
        }
-      /* Is this file's first line closer than the first lines of other files?
-        If so, record this file, and its first line, as best alternate.  */
-      if (first_line >= 0 && first_pc > pc
-         && (alt_pc == 0 || first_pc < alt_pc))
-       {
-         alt_pc = first_pc;
-         alt_line = first_line;
-         alt_symtab = s;
-       }
     }
-  if (best_symtab == 0)
+
+  if (!best_symtab)
     {
-      val.symtab = alt_symtab;
-      val.line = alt_line - 1;
-      val.pc = BLOCK_END (BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK));
-      val.end = alt_pc;
+      if (!alt_symtab)
+       {                       /* If we didn't find any line # info, just
+                                return zeros.  */
+         val.symtab = 0;
+         val.line = 0;
+         val.pc = pc;
+         val.end = 0;
+       }
+      else
+       {
+         val.symtab = alt_symtab;
+         val.line = alt->line - 1;
+         val.pc = BLOCK_END (BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK));
+         val.end = alt->pc;
+       }
     }
   else
     {
       val.symtab = best_symtab;
-      val.line = best_line;
-      val.pc = best_pc;
-      if (best_end && (alt_pc == 0 || best_end < alt_pc))
+      val.line = best->line;
+      val.pc = best->pc;
+      if (best_end && (!alt || best_end < alt->pc))
        val.end = best_end;
-      else if (alt_pc)
-       val.end = alt_pc;
+      else if (alt)
+       val.end = alt->pc;
       else
        val.end = BLOCK_END (BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK));
     }
   return val;
 }
 \f
+static int find_line_symtab PARAMS ((struct symtab *, int, struct linetable **,
+                                    int *, int *));
+
+/* Find line number LINE in any symtab whose name is the same as
+   SYMTAB.
+
+   If found, return 1, set *LINETABLE to the linetable in which it was
+   found, set *INDEX to the index in the linetable of the best entry
+   found, and set *EXACT_MATCH nonzero if the value returned is an
+   exact match.
+
+   If not found, return 0.  */
+
+static int
+find_line_symtab (symtab, line, linetable, index, exact_match)
+     struct symtab *symtab;
+     int line;
+     struct linetable **linetable;
+     int *index;
+     int *exact_match;
+{
+  int exact;
+
+  /* BEST_INDEX and BEST_LINETABLE identify the smallest linenumber > LINE
+     so far seen.  */
+
+  int best_index;
+  struct linetable *best_linetable;
+
+  /* First try looking it up in the given symtab.  */
+  best_linetable = LINETABLE (symtab);
+  best_index = find_line_common (best_linetable, line, &exact);
+  if (best_index < 0 || !exact)
+    {
+      /* Didn't find an exact match.  So we better keep looking for
+        another symtab with the same name.  In the case of xcoff,
+        multiple csects for one source file (produced by IBM's FORTRAN
+        compiler) produce multiple symtabs (this is unavoidable
+        assuming csects can be at arbitrary places in memory and that
+        the GLOBAL_BLOCK of a symtab has a begin and end address).  */
+
+      /* BEST is the smallest linenumber > LINE so far seen,
+        or 0 if none has been seen so far.
+        BEST_INDEX and BEST_LINETABLE identify the item for it.  */
+      int best;
+
+      struct objfile *objfile;
+      struct symtab *s;
+
+      if (best_index >= 0)
+       best = best_linetable->item[best_index].line;
+      else
+       best = 0;
+
+      ALL_SYMTABS (objfile, s)
+       {
+         struct linetable *l;
+         int ind;
+
+         if (!STREQ (symtab->filename, s->filename))
+           continue;
+         l = LINETABLE (s);
+         ind = find_line_common (l, line, &exact);
+         if (ind >= 0)
+           {
+             if (exact)
+               {
+                 best_index = ind;
+                 best_linetable = l;
+                 goto done;
+               }
+             if (best == 0 || l->item[ind].line < best)
+               {
+                 best = l->item[ind].line;
+                 best_index = ind;
+                 best_linetable = l;
+               }
+           }
+       }
+    }
+ done:
+  if (best_index < 0)
+    return 0;
+
+  if (index)
+    *index = best_index;
+  if (linetable)
+    *linetable = best_linetable;
+  if (exact_match)
+    *exact_match = exact;
+  return 1;
+}
+\f
 /* Find the PC value for a given source file and line number.
    Returns zero for invalid line number.
    The source file is specified with a struct symtab.  */
@@ -1084,15 +1244,15 @@ find_line_pc (symtab, line)
      struct symtab *symtab;
      int line;
 {
-  register struct linetable *l;
-  register int ind;
-  int dummy;
+  struct linetable *l;
+  int ind;
 
   if (symtab == 0)
     return 0;
-  l = LINETABLE (symtab);
-  ind = find_line_common(l, line, &dummy);
-  return (ind >= 0) ? l->item[ind].pc : 0;
+  if (find_line_symtab (symtab, line, &l, &ind, NULL))
+    return l->item[ind].pc;
+  else
+    return 0;
 }
 
 /* Find the range of pc values in a line.
@@ -1107,16 +1267,14 @@ find_line_pc_range (symtab, thisline, startptr, endptr)
      int thisline;
      CORE_ADDR *startptr, *endptr;
 {
-  register struct linetable *l;
-  register int ind;
+  struct linetable *l;
+  int ind;
   int exact_match;             /* did we get an exact linenumber match */
 
   if (symtab == 0)
     return 0;
 
-  l = LINETABLE (symtab);
-  ind = find_line_common (l, thisline, &exact_match);
-  if (ind >= 0)
+  if (find_line_symtab (symtab, thisline, &l, &ind, &exact_match))
     {
       *startptr = l->item[ind].pc;
       /* If we have not seen an entry for the specified line,
@@ -1171,6 +1329,7 @@ find_line_common (l, lineno, exact_match)
 
       if (item->line == lineno)
        {
+         /* Return the first (lowest address) entry which matches.  */
          *exact_match = 1;
          return i;
        }
@@ -1288,41 +1447,25 @@ operator_chars (p, end)
 /* Recursive helper function for decode_line_1.
  * Look for methods named NAME in type T.
  * Return number of matches.
- * Put matches in PHYSNAMES and SYM_ARR (which better be big enough!).
+ * Put matches in SYM_ARR (which better be big enough!).
  * These allocations seem to define "big enough":
  * sym_arr = (struct symbol **) alloca(TYPE_NFN_FIELDS_TOTAL (t) * sizeof(struct symbol*));
- * physnames = (char **) alloca (TYPE_NFN_FIELDS_TOTAL (t) * sizeof(char*));
+ * Note that this function is g++ specific.
  */
 
 int
-find_methods (t, name, physnames, sym_arr)
+find_methods (t, name, sym_arr)
      struct type *t;
      char *name;
-     char **physnames;
      struct symbol **sym_arr;
 {
   int i1 = 0;
   int ibase;
   struct symbol *sym_class;
   char *class_name = type_name_no_tag (t);
-  /* Ignore this class if it doesn't have a name.
-     This prevents core dumps, but is just a workaround
-     because we might not find the function in
-     certain cases, such as
-     struct D {virtual int f();}
-     struct C : D {virtual int g();}
-     (in this case g++ 1.35.1- does not put out a name
-     for D as such, it defines type 19 (for example) in
-     the same stab as C, and then does a
-     .stabs "D:T19" and a .stabs "D:t19".
-     Thus
-     "break C::f" should not be looking for field f in
-     the class named D, 
-     but just for the field f in the baseclasses of C
-     (no matter what their names).
-     
-     However, I don't know how to replace the code below
-     that depends on knowing the name of D.  */
+  /* Ignore this class if it doesn't have a name.  This is ugly, but
+     unless we figure out how to get the physname without the name of
+     the class, then the loop can't do any good.  */
   if (class_name
       && (sym_class = lookup_symbol (class_name,
                                     (struct block *)NULL,
@@ -1331,6 +1474,7 @@ find_methods (t, name, physnames, sym_arr)
                                     (struct symtab **)NULL)))
     {
       int method_counter;
+      /* FIXME: Shouldn't this just be check_stub_type (t)?  */
       t = SYMBOL_TYPE (sym_class);
       for (method_counter = TYPE_NFN_FIELDS (t) - 1;
           method_counter >= 0;
@@ -1340,7 +1484,7 @@ find_methods (t, name, physnames, sym_arr)
          struct fn_field *f = TYPE_FN_FIELDLIST1 (t, method_counter);
 
          char *method_name = TYPE_FN_FIELDLIST_NAME (t, method_counter);
-         if (!strcmp (name, method_name))
+         if (STREQ (name, method_name))
            /* Find all the fields with that name.  */
            for (field_counter = TYPE_FN_FIELDLIST_LENGTH (t, method_counter) - 1;
                 field_counter >= 0;
@@ -1350,8 +1494,17 @@ find_methods (t, name, physnames, sym_arr)
                if (TYPE_FN_FIELD_STUB (f, field_counter))
                  check_stub_method (t, method_counter, field_counter);
                phys_name = TYPE_FN_FIELD_PHYSNAME (f, field_counter);
-               physnames[i1] = (char*) alloca (strlen (phys_name) + 1);
-               strcpy (physnames[i1], phys_name);
+               /* Destructor is handled by caller, dont add it to the list */
+               if (DESTRUCTOR_PREFIX_P (phys_name))
+                 continue;
+
+               /* FIXME: Why are we looking this up in the
+                  SYMBOL_BLOCK_VALUE (sym_class)?  It is intended as a hook
+                  for nested types?  If so, it should probably hook to the
+                  type, not the symbol.  mipsread.c is the only symbol
+                  reader which sets the SYMBOL_BLOCK_VALUE for types, and
+                  this is not documented in symtab.h.  -26Aug93.  */
+
                sym_arr[i1] = lookup_symbol (phys_name,
                                             SYMBOL_BLOCK_VALUE (sym_class),
                                             VAR_NAMESPACE,
@@ -1361,23 +1514,73 @@ find_methods (t, name, physnames, sym_arr)
                else
                  {
                    fputs_filtered("(Cannot find method ", stdout);
-                   fputs_demangled(phys_name, stdout, DMGL_PARAMS);
+                   fprintf_symbol_filtered (stdout, phys_name,
+                                            language_cplus, DMGL_PARAMS);
                    fputs_filtered(" - possibly inlined.)\n", stdout);
                  }
              }
        }
     }
-  /* Only search baseclasses if there is no match yet,
-   * since names in derived classes override those in baseclasses.
-   */
+
+  /* Only search baseclasses if there is no match yet, since names in
+     derived classes override those in baseclasses.
+
+     FIXME: The above is not true; it is only true of member functions
+     if they have the same number of arguments (??? - section 13.1 of the
+     ARM says the function members are not in the same scope but doesn't
+     really spell out the rules in a way I understand.  In any case, if
+     the number of arguments differ this is a case in which we can overload
+     rather than hiding without any problem, and gcc 2.4.5 does overload
+     rather than hiding in this case).  */
+
   if (i1)
     return i1;
   for (ibase = 0; ibase < TYPE_N_BASECLASSES (t); ibase++)
     i1 += find_methods(TYPE_BASECLASS(t, ibase), name,
-                      physnames + i1, sym_arr + i1);
+                      sym_arr + i1);
   return i1;
 }
 
+/* Helper function for decode_line_1.
+   Build a canonical line spec in CANONICAL if it is non-NULL and if
+   the SAL has a symtab.
+   If SYMNAME is non-NULL the canonical line spec is `filename:symname'.
+   If SYMNAME is NULL the line number from SAL is used and the canonical
+   line spec is `filename:linenum'.  */
+
+static void
+build_canonical_line_spec (sal, symname, canonical)
+     struct symtab_and_line *sal;
+     char *symname;
+     char ***canonical;
+{
+  char **canonical_arr;
+  char *canonical_name;
+  char *filename;
+  struct symtab *s = sal->symtab;
+
+  if (s == (struct symtab *)NULL
+      || s->filename == (char *)NULL
+      || canonical == (char ***)NULL)
+    return;
+  canonical_arr = (char **) xmalloc (sizeof (char *));
+  *canonical = canonical_arr;
+
+  filename = s->filename;
+  if (symname != NULL)
+    {
+      canonical_name = xmalloc (strlen (filename) + strlen (symname) + 2);
+      sprintf (canonical_name, "%s:%s", filename, symname);
+    }
+  else
+    {
+      canonical_name = xmalloc (strlen (filename) + 30);
+      sprintf (canonical_name, "%s:%d", filename, sal->line);
+    }
+  canonical_arr[0] = canonical_name;
+}
+
 /* Parse a string that specifies a line number.
    Pass the address of a char * variable; that variable will be
    advanced over the characters actually parsed.
@@ -1402,20 +1605,55 @@ find_methods (t, name, physnames, sym_arr)
    It defaults to current_source_symtab.
    DEFAULT_LINE specifies the line number to use for relative
    line numbers (that start with signs).  Defaults to current_source_line.
+   If CANONICAL is non-NULL, store an array of strings containing the canonical
+   line specs there if necessary. Currently overloaded member functions and
+   line numbers or static functions without a filename yield a canonical
+   line spec. The array and the line spec strings are allocated on the heap,
+   it is the callers responsibility to free them.
 
    Note that it is possible to return zero for the symtab
    if no file is validly specified.  Callers must check that.
    Also, the line number returned may be invalid.  */
 
 struct symtabs_and_lines
-decode_line_1 (argptr, funfirstline, default_symtab, default_line)
+decode_line_1 (argptr, funfirstline, default_symtab, default_line, canonical)
      char **argptr;
      int funfirstline;
      struct symtab *default_symtab;
      int default_line;
+     char ***canonical;
 {
   struct symtabs_and_lines values;
+#ifdef HPPA_COMPILER_BUG
+  /* FIXME: The native HP 9000/700 compiler has a bug which appears
+     when optimizing this file with target i960-vxworks.  I haven't
+     been able to construct a simple test case.  The problem is that
+     in the second call to SKIP_PROLOGUE below, the compiler somehow
+     does not realize that the statement val = find_pc_line (...) will
+     change the values of the fields of val.  It extracts the elements
+     into registers at the top of the block, and does not update the
+     registers after the call to find_pc_line.  You can check this by
+     inserting a printf at the end of find_pc_line to show what values
+     it is returning for val.pc and val.end and another printf after
+     the call to see what values the function actually got (remember,
+     this is compiling with cc -O, with this patch removed).  You can
+     also examine the assembly listing: search for the second call to
+     skip_prologue; the LDO statement before the next call to
+     find_pc_line loads the address of the structure which
+     find_pc_line will return; if there is a LDW just before the LDO,
+     which fetches an element of the structure, then the compiler
+     still has the bug.
+
+     Setting val to volatile avoids the problem.  We must undef
+     volatile, because the HPPA native compiler does not define
+     __STDC__, although it does understand volatile, and so volatile
+     will have been defined away in defs.h.  */
+#undef volatile
+  volatile struct symtab_and_line val;
+#define volatile /*nothing*/
+#else
   struct symtab_and_line val;
+#endif
   register char *p, *p1;
   char *q, *q1;
   register struct symtab *s;
@@ -1429,9 +1667,9 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
   char *copy;
   struct symbol *sym_class;
   int i1;
+  int is_quoted;
   struct symbol **sym_arr;
   struct type *t;
-  char **physnames;
   char *saved_arg = *argptr;
   extern char *gdb_completer_quote_characters;
   
@@ -1443,22 +1681,9 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
       default_line = current_source_line;
     }
 
+  /* See if arg is *PC */
 
-  /* Check to see if *ARGPTR points to a string that has been quoted with
-     gdb_completer_quote_characters.  If so, P will be left pointing at
-     someplace other than *ARGPTR */
-
-  if (((p = skip_quoted (*argptr)) != *argptr) &&
-      ((*(p - 1) != **argptr) ||
-       (strchr (gdb_completer_quote_characters, **argptr) == NULL)))
-    {
-      /* Not quoted symbol string specification, reset P */
-      p = *argptr;
-    }
-
-  /* See if arg is *PC or '<some symbol specifier string>' */
-
-  if ((**argptr == '*') || (p != *argptr))
+  if (**argptr == '*')
     {
       if (**argptr == '*')
        {
@@ -1470,12 +1695,14 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
       values.nelts = 1;
       values.sals[0] = find_pc_line (pc, 0);
       values.sals[0].pc = pc;
+      build_canonical_line_spec (values.sals, NULL, canonical);
       return values;
     }
 
   /* Maybe arg is FILE : LINENUM or FILE : FUNCTION */
 
-  s = 0;
+  s = NULL;
+  is_quoted = (strchr (gdb_completer_quote_characters, **argptr) != NULL);
 
   for (p = *argptr; *p; p++)
     {
@@ -1484,7 +1711,7 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
     }
   while (p[0] == ' ' || p[0] == '\t') p++;
 
-  if (p[0] == ':')
+  if ((p[0] == ':') && !is_quoted)
     {
 
       /*  C++  */
@@ -1521,12 +1748,12 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
                  char *tmp = alloca (q1 - q + 1);
                  memcpy (tmp, q, q1 - q);
                  tmp[q1 - q] = '\0';
-                 opname = cplus_mangle_opname (tmp, 1);
+                 opname = cplus_mangle_opname (tmp, DMGL_ANSI);
                  if (opname == NULL)
                    {
                      warning ("no mangling for \"%s\"", tmp);
                      cplusplus_hint (saved_arg);
-                     return_to_top_level ();
+                     return_to_top_level (RETURN_ERROR);
                    }
                  copy = (char*) alloca (3 + strlen(opname));
                  sprintf (copy, "__%s", opname);
@@ -1547,23 +1774,29 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
              i1 = 0;           /*  counter for the symbol array */
              t = SYMBOL_TYPE (sym_class);
              sym_arr = (struct symbol **) alloca(TYPE_NFN_FIELDS_TOTAL (t) * sizeof(struct symbol*));
-             physnames = (char **) alloca (TYPE_NFN_FIELDS_TOTAL (t) * sizeof(char*));
 
-             if (destructor_name_p (copy, t))
+             /* Cfront objects don't have fieldlists.  */
+             if (destructor_name_p (copy, t) && TYPE_FN_FIELDLISTS (t) != NULL)
                {
                  /* destructors are a special case.  */
                  struct fn_field *f = TYPE_FN_FIELDLIST1 (t, 0);
                  int len = TYPE_FN_FIELDLIST_LENGTH (t, 0) - 1;
+                 /* gcc 1.x puts destructor in last field,
+                    gcc 2.x puts destructor in first field.  */
                  char *phys_name = TYPE_FN_FIELD_PHYSNAME (f, len);
-                 physnames[i1] = (char *)alloca (strlen (phys_name) + 1);
-                 strcpy (physnames[i1], phys_name);
+                 if (!DESTRUCTOR_PREFIX_P (phys_name))
+                   {
+                     phys_name = TYPE_FN_FIELD_PHYSNAME (f, 0);
+                     if (!DESTRUCTOR_PREFIX_P (phys_name))
+                       phys_name = "";
+                   }
                  sym_arr[i1] =
                    lookup_symbol (phys_name, SYMBOL_BLOCK_VALUE (sym_class),
                                   VAR_NAMESPACE, 0, (struct symtab **)NULL);
                  if (sym_arr[i1]) i1++;
                }
              else
-               i1 = find_methods (t, copy, physnames, sym_arr);
+               i1 = find_methods (t, copy, sym_arr);
              if (i1 == 1)
                {
                  /* There is exactly one field with that name.  */
@@ -1590,7 +1823,7 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
                {
                  /* There is more than one field with that name
                     (overloaded).  Ask the user which one to use.  */
-                 return decode_line_2 (sym_arr, i1, funfirstline);
+                 return decode_line_2 (sym_arr, i1, funfirstline, canonical);
                }
              else
                {
@@ -1606,12 +1839,12 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
                    tmp = copy;
                  if (tmp[0] == '~')
                    warning ("the class `%s' does not have destructor defined",
-                            sym_class->name);
+                            SYMBOL_SOURCE_NAME(sym_class));
                  else
                    warning ("the class %s does not have any method named %s",
-                            sym_class->name, tmp);
+                            SYMBOL_SOURCE_NAME(sym_class), tmp);
                  cplusplus_hint (saved_arg);
-                 return_to_top_level ();
+                 return_to_top_level (RETURN_ERROR);
                }
            }
          else
@@ -1620,7 +1853,7 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
              warning ("can't find class, struct, or union named \"%s\"",
                       copy);
              cplusplus_hint (saved_arg);
-             return_to_top_level ();
+             return_to_top_level (RETURN_ERROR);
            }
        }
       /*  end of C++  */
@@ -1663,6 +1896,9 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
       /* We found a token consisting of all digits -- at least one digit.  */
       enum sign {none, plus, minus} sign = none;
 
+      /* We might need a canonical line spec if no file was specified.  */
+      int need_canonical = (s == 0) ? 1 : 0;
+
       /* This is where we need to make sure that we have good defaults.
         We must guarantee that this section of code is never executed
         when we are called with just a function name, since
@@ -1709,6 +1945,8 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
       values.sals = (struct symtab_and_line *)xmalloc (sizeof (struct symtab_and_line));
       values.sals[0] = val;
       values.nelts = 1;
+      if (need_canonical)
+       build_canonical_line_spec (values.sals, NULL, canonical);
       return values;
     }
 
@@ -1716,19 +1954,16 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
      Find the next token (everything up to end or next whitespace).  */
 
   p = skip_quoted (*argptr);
+  if (is_quoted && p[-1] != '\'')
+    error ("Unmatched single quote.");
   copy = (char *) alloca (p - *argptr + 1);
   memcpy (copy, *argptr, p - *argptr);
   copy[p - *argptr] = '\0';
   if ((copy[0] == copy [p - *argptr - 1])
       && strchr (gdb_completer_quote_characters, copy[0]) != NULL)
     {
-      char *temp;
       copy [p - *argptr - 1] = '\0';
       copy++;
-      if ((temp = expensive_mangler (copy)) != NULL)
-       {
-         copy = temp;
-       }
     }
   while (*p == ' ' || *p == '\t') p++;
   *argptr = p;
@@ -1754,15 +1989,17 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
          /* Convex: no need to suppress code on first line, if any */
          val.pc = pc;
 #else
-         /* If SKIP_PROLOGUE left us in mid-line, and the next line is still
-            part of the same function:
-               advance to next line, 
-               recalculate its line number (might not be N+1).  */
-         if (val.pc != pc && val.end &&
-             lookup_minimal_symbol_by_pc (pc) == lookup_minimal_symbol_by_pc (val.end)) {
-           pc = val.end;       /* First pc of next line */
-           val = find_pc_line (pc, 0);
-         }
+         /* Check if SKIP_PROLOGUE left us in mid-line, and the next
+            line is still part of the same function.  */
+         if (val.pc != pc
+             && BLOCK_START (SYMBOL_BLOCK_VALUE (sym)) <= val.end
+             && val.end < BLOCK_END (SYMBOL_BLOCK_VALUE (sym)))
+           {
+             /* First pc of next line */
+             pc = val.end;
+             /* Recalculate the line number (might not be N+1).  */
+             val = find_pc_line (pc, 0);
+           }
          val.pc = pc;
 #endif
          values.sals = (struct symtab_and_line *)xmalloc (sizeof (struct symtab_and_line));
@@ -1775,7 +2012,15 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
             SKIP_PROLOGUE".  */
          if (SYMBOL_LINE (sym) != 0)
            values.sals[0].line = SYMBOL_LINE (sym);
-         
+
+         /* We might need a canonical line spec if it is a static function.  */
+         if (s == 0)
+           {
+             struct blockvector *bv = BLOCKVECTOR (sym_symtab);
+             struct block *b = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK);
+             if (lookup_block_symbol (b, copy, VAR_NAMESPACE) != NULL)
+               build_canonical_line_spec (values.sals, copy, canonical);
+           }
          return values;
        }
       else if (SYMBOL_LINE (sym) != 0)
@@ -1792,6 +2037,8 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
       else
        /* This can happen if it is compiled with a compiler which doesn't
           put out line numbers for variables.  */
+       /* FIXME: Shouldn't we just set .line and .symtab to zero and
+          return?  For example, "info line foo" could print the address.  */
        error ("Line number not known for symbol \"%s\"", copy);
     }
 
@@ -1800,7 +2047,7 @@ decode_line_1 (argptr, funfirstline, default_symtab, default_line)
     {
       val.symtab = 0;
       val.line = 0;
-      val.pc = msymbol -> address + FUNCTION_START_OFFSET;
+      val.pc = SYMBOL_VALUE_ADDRESS (msymbol) + FUNCTION_START_OFFSET;
       if (funfirstline)
        SKIP_PROLOGUE (val.pc);
       values.sals = (struct symtab_and_line *)xmalloc (sizeof (struct symtab_and_line));
@@ -1826,29 +2073,45 @@ decode_line_spec (string, funfirstline)
   if (string == 0)
     error ("Empty line specification.");
   sals = decode_line_1 (&string, funfirstline,
-                       current_source_symtab, current_source_line);
+                       current_source_symtab, current_source_line,
+                       (char ***)NULL);
   if (*string)
     error ("Junk at end of line specification: %s", string);
   return sals;
 }
 
-/* Given a list of NELTS symbols in sym_arr (with corresponding
-   mangled names in physnames), return a list of lines to operate on
-   (ask user if necessary).  */
+/* Given a list of NELTS symbols in SYM_ARR, return a list of lines to
+   operate on (ask user if necessary).
+   If CANONICAL is non-NULL return a corresponding array of mangled names
+   as canonical line specs there.  */
+
 static struct symtabs_and_lines
-decode_line_2 (sym_arr, nelts, funfirstline)
+decode_line_2 (sym_arr, nelts, funfirstline, canonical)
      struct symbol *sym_arr[];
      int nelts;
      int funfirstline;
+     char ***canonical;
 {
   struct symtabs_and_lines values, return_values;
   register CORE_ADDR pc;
   char *args, *arg1;
   int i;
   char *prompt;
+  char *symname;
+  struct cleanup *old_chain;
+  char **canonical_arr = (char **)NULL;
 
   values.sals = (struct symtab_and_line *) alloca (nelts * sizeof(struct symtab_and_line));
   return_values.sals = (struct symtab_and_line *) xmalloc (nelts * sizeof(struct symtab_and_line));
+  old_chain = make_cleanup (free, return_values.sals);
+
+  if (canonical)
+    {
+      canonical_arr = (char **) xmalloc (nelts * sizeof (char *));
+      make_cleanup (free, canonical_arr);
+      memset (canonical_arr, 0, nelts * sizeof (char *));
+      *canonical = canonical_arr;
+    }
 
   i = 0;
   printf("[0] cancel\n[1] all\n");
@@ -1864,8 +2127,8 @@ decode_line_2 (sym_arr, nelts, funfirstline)
          values.sals[i] = find_pc_line (pc, 0);
          values.sals[i].pc = (values.sals[i].end && values.sals[i].pc != pc) ?
                               values.sals[i].end                      :  pc;
-         printf("[%d] file:%s; line number:%d\n",
-                (i+2), values.sals[i].symtab->filename, values.sals[i].line);
+         printf("[%d] %s at %s:%d\n", (i+2), SYMBOL_SOURCE_NAME (sym_arr[i]),
+                values.sals[i].symtab->filename, values.sals[i].line);
        }
       else printf ("?HERE\n");
       i++;
@@ -1880,7 +2143,7 @@ decode_line_2 (sym_arr, nelts, funfirstline)
 
   args = command_line_input ((char *) NULL, 0);
   
-  if (args == 0)
+  if (args == 0 || *args == 0)
     error_no_arg ("one or more choice numbers");
 
   i = 0;
@@ -1899,9 +2162,21 @@ decode_line_2 (sym_arr, nelts, funfirstline)
        error ("cancelled");
       else if (num == 1)
        {
+         if (canonical_arr)
+           {
+             for (i = 0; i < nelts; i++)
+               {
+                 if (canonical_arr[i] == NULL)
+                   {
+                     symname = SYMBOL_NAME (sym_arr[i]);
+                     canonical_arr[i] = savestring (symname, strlen (symname));
+                   }
+               }
+           }
          memcpy (return_values.sals, values.sals,
                  (nelts * sizeof(struct symtab_and_line)));
          return_values.nelts = nelts;
+         discard_cleanups (old_chain);
          return return_values;
        }
 
@@ -1914,6 +2189,12 @@ decode_line_2 (sym_arr, nelts, funfirstline)
          num -= 2;
          if (values.sals[num].pc)
            {
+             if (canonical_arr)
+               {
+                 symname = SYMBOL_NAME (sym_arr[num]);
+                 make_cleanup (free, symname);
+                 canonical_arr[i] = savestring (symname, strlen (symname));
+               }
              return_values.sals[i++] = values.sals[num];
              values.sals[num].pc = 0;
            }
@@ -1927,6 +2208,7 @@ decode_line_2 (sym_arr, nelts, funfirstline)
       while (*args == ' ' || *args == '\t') args++;
     }
   return_values.nelts = i;
+  discard_cleanups (old_chain);
   return return_values;
 }
 
@@ -1966,7 +2248,7 @@ output_source_filename (name, first)
 
   /* Is NAME in tab?  */
   for (p = tab; p < tab + tab_cur_size; p++)
-    if (strcmp (*p, name) == 0)
+    if (STREQ (*p, name))
       /* Yes; don't print it again.  */
       return;
   /* No; add it to tab.  */
@@ -2027,23 +2309,9 @@ sources_info (ignore, from_tty)
   printf_filtered ("\n");
 }
 
-static int
-name_match (name)
-     char *name;
-{
-  char *demangled = cplus_demangle (name, 0);
-  if (demangled != NULL)
-    {
-      int cond = re_exec (demangled);
-      free (demangled);
-      return (cond);
-    }
-  return (re_exec (name));
-}
-#define NAME_MATCH(NAME) name_match(NAME)
-
-/* List all symbols (if REGEXP is 0) or all symbols matching REGEXP.
-   If CLASS is zero, list all symbols except functions and type names.
+/* List all symbols (if REGEXP is NULL) or all symbols matching REGEXP.
+   If CLASS is zero, list all symbols except functions, type names, and
+                    constants (enums).
    If CLASS is 1, list only functions.
    If CLASS is 2, list only type names.
    If CLASS is 3, list only method names.
@@ -2079,7 +2347,7 @@ list_symbols (regexp, class, bpt)
   enum minimal_symbol_type ourtype = types[class];
   enum minimal_symbol_type ourtype2 = types2[class];
 
-  if (regexp)
+  if (regexp != NULL)
     {
       /* Make sure spacing is right for C++ operators.
         This is just a courtesy to make the matching less sensitive
@@ -2152,7 +2420,7 @@ list_symbols (regexp, class, bpt)
 
              /* If it would match (logic taken from loop below)
                 load the file and go on to the next one */
-             if ((regexp == 0 || NAME_MATCH (SYMBOL_NAME (psym)))
+             if ((regexp == NULL || SYMBOL_MATCHES_REGEXP (psym))
                  && ((class == 0 && SYMBOL_CLASS (psym) != LOC_TYPEDEF
                       && SYMBOL_CLASS (psym) != LOC_BLOCK)
                      || (class == 1 && SYMBOL_CLASS (psym) == LOC_BLOCK)
@@ -2177,11 +2445,12 @@ list_symbols (regexp, class, bpt)
     {
       ALL_MSYMBOLS (objfile, msymbol)
        {
-         if (msymbol -> type == ourtype || msymbol -> type == ourtype2)
+         if (MSYMBOL_TYPE (msymbol) == ourtype ||
+             MSYMBOL_TYPE (msymbol) == ourtype2)
            {
-             if (regexp == 0 || NAME_MATCH (msymbol -> name))
+             if (regexp == NULL || SYMBOL_MATCHES_REGEXP (msymbol))
                {
-                 if (0 == find_pc_symtab (msymbol -> address))
+                 if (0 == find_pc_symtab (SYMBOL_VALUE_ADDRESS (msymbol)))
                    {
                      found_misc = 1;
                    }
@@ -2219,9 +2488,10 @@ list_symbols (regexp, class, bpt)
              {
                QUIT;
                sym = BLOCK_SYM (b, j);
-               if ((regexp == 0 || NAME_MATCH (SYMBOL_NAME (sym)))
+               if ((regexp == NULL || SYMBOL_MATCHES_REGEXP (sym))
                    && ((class == 0 && SYMBOL_CLASS (sym) != LOC_TYPEDEF
-                        && SYMBOL_CLASS (sym) != LOC_BLOCK)
+                        && SYMBOL_CLASS (sym) != LOC_BLOCK
+                        && SYMBOL_CLASS (sym) != LOC_CONST)
                        || (class == 1 && SYMBOL_CLASS (sym) == LOC_BLOCK)
                        || (class == 2 && SYMBOL_CLASS (sym) == LOC_TYPEDEF)
                        || (class == 3 && SYMBOL_CLASS (sym) == LOC_BLOCK)))
@@ -2230,7 +2500,21 @@ list_symbols (regexp, class, bpt)
                      {
                        /* Set a breakpoint here, if it's a function */
                        if (class == 1)
-                         break_command (SYMBOL_NAME(sym), 0);
+                         {
+                           /* There may be more than one function with the
+                              same name but in different files.  In order to
+                              set breakpoints on all of them, we must give
+                              both the file name and the function name to
+                              break_command.  */
+                           char *string =
+                             (char *) alloca (strlen (s->filename)
+                                              + strlen (SYMBOL_NAME(sym))
+                                              + 2);
+                           strcpy (string, s->filename);
+                           strcat (string, ":");
+                           strcat (string, SYMBOL_NAME(sym));
+                           break_command (string, 0);
+                         }
                      }
                    else if (!found_in_file)
                      {
@@ -2246,7 +2530,7 @@ list_symbols (regexp, class, bpt)
                    /* Typedef that is not a C++ class */
                    if (class == 2
                        && SYMBOL_NAMESPACE (sym) != STRUCT_NAMESPACE)
-                     typedef_print (SYMBOL_TYPE(sym), sym, stdout);
+                     c_typedef_print (SYMBOL_TYPE(sym), sym, stdout);
                    /* variable, func, or typedef-that-is-c++-class */
                    else if (class < 2 || 
                             (class == 2 && 
@@ -2254,20 +2538,22 @@ list_symbols (regexp, class, bpt)
                      {
                        type_print (SYMBOL_TYPE (sym),
                                    (SYMBOL_CLASS (sym) == LOC_TYPEDEF
-                                    ? "" : SYMBOL_NAME (sym)),
+                                    ? "" : SYMBOL_SOURCE_NAME (sym)),
                                    stdout, 0);
                        
                        printf_filtered (";\n");
                      }
                    else
                      {
-# if 0
-/* FIXME, why is this zapped out? */
+# if 0  /* FIXME, why is this zapped out? */
                        char buf[1024];
-                       type_print_base (TYPE_FN_FIELD_TYPE(t, i), stdout, 0, 0); 
-                       type_print_varspec_prefix (TYPE_FN_FIELD_TYPE(t, i), stdout, 0); 
+                       c_type_print_base (TYPE_FN_FIELD_TYPE(t, i),
+                                          stdout, 0, 0); 
+                       c_type_print_varspec_prefix (TYPE_FN_FIELD_TYPE(t, i),
+                                                    stdout, 0); 
                        sprintf (buf, " %s::", type_name_no_tag (t));
-                       type_print_method_args (TYPE_FN_FIELD_ARGS (t, i), buf, name, stdout);
+                       cp_type_print_method_args (TYPE_FN_FIELD_ARGS (t, i),
+                                                  buf, name, stdout);
 # endif
                      }
                  }
@@ -2284,18 +2570,19 @@ list_symbols (regexp, class, bpt)
       found_in_file = 0;
       ALL_MSYMBOLS (objfile, msymbol)
        {
-         if (msymbol -> type == ourtype || msymbol -> type == ourtype2)
+         if (MSYMBOL_TYPE (msymbol) == ourtype ||
+             MSYMBOL_TYPE (msymbol) == ourtype2)
            {
-             if (regexp == 0 || NAME_MATCH (msymbol -> name))
+             if (regexp == NULL || SYMBOL_MATCHES_REGEXP (msymbol))
                {
                  /* Functions:  Look up by address. */
                  if (class != 1 ||
-                     (0 == find_pc_symtab (msymbol -> address)))
+                     (0 == find_pc_symtab (SYMBOL_VALUE_ADDRESS (msymbol))))
                    {
                      /* Variables/Absolutes:  Look up by name */
-                     if (lookup_symbol (msymbol -> name
-                                        (struct block *) 0, VAR_NAMESPACE, 0,
-                                        (struct symtab **) 0) == NULL)
+                     if (lookup_symbol (SYMBOL_NAME (msymbol)
+                                        (struct block *) NULL, VAR_NAMESPACE,
+                                        0, (struct symtab **) NULL) == NULL)
                        {
                          if (!found_in_file)
                            {
@@ -2303,8 +2590,8 @@ list_symbols (regexp, class, bpt)
                              found_in_file = 1;
                            }
                          printf_filtered ("    %08x  %s\n",
-                                          msymbol -> address,
-                                          msymbol -> name);
+                                          SYMBOL_VALUE_ADDRESS (msymbol),
+                                          SYMBOL_SOURCE_NAME (msymbol));
                        }
                    }
                }
@@ -2377,47 +2664,85 @@ static int return_val_size;
 static int return_val_index;
 static char **return_val;
 
-/*  Test to see if the symbol specified by SYMNAME (or it's demangled
-    equivalent) matches TEXT in the first TEXT_LEN characters.  If so,
-    add it to the current completion list. */
+#define COMPLETION_LIST_ADD_SYMBOL(symbol, sym_text, len, text, word) \
+  do { \
+    if (SYMBOL_DEMANGLED_NAME (symbol) != NULL) \
+      /* Put only the mangled name on the list.  */ \
+      /* Advantage:  "b foo<TAB>" completes to "b foo(int, int)" */ \
+      /* Disadvantage:  "b foo__i<TAB>" doesn't complete.  */ \
+      completion_list_add_name \
+       (SYMBOL_DEMANGLED_NAME (symbol), (sym_text), (len), (text), (word)); \
+    else \
+      completion_list_add_name \
+       (SYMBOL_NAME (symbol), (sym_text), (len), (text), (word)); \
+  } while (0)
+
+/*  Test to see if the symbol specified by SYMNAME (which is already
+    demangled for C++ symbols) matches SYM_TEXT in the first SYM_TEXT_LEN
+    characters.  If so, add it to the current completion list. */
 
 static void
-completion_list_add_symbol (symname, text, text_len)
+completion_list_add_name (symname, sym_text, sym_text_len, text, word)
      char *symname;
+     char *sym_text;
+     int sym_text_len;
      char *text;
-     int text_len;
+     char *word;
 {
-  char *demangled;
   int newsize;
+  int i;
 
-  /* First see if SYMNAME is a C++ mangled name, and if so, use the
-     demangled name instead, including any parameters. */
+  /* clip symbols that cannot match */
 
-  if ((demangled = cplus_demangle (symname, DMGL_PARAMS | DMGL_ANSI)) != NULL)
+  if (strncmp (symname, sym_text, sym_text_len) != 0)
     {
-      symname = demangled;
+      return;
     }
 
-  /* If we have a match for a completion, then add SYMNAME to the current
-     list of matches. Note that we always make a copy of the string, even
-     if it is one that was returned from cplus_demangle and is already
-     in malloc'd memory. */
+  /* Clip any symbol names that we've already considered.  (This is a
+     time optimization)  */
 
-  if (strncmp (symname, text, text_len) == 0)
+  for (i = 0; i < return_val_index; ++i)
     {
-      if (return_val_index + 3 > return_val_size)
+      if (STREQ (symname, return_val[i]))
        {
-         newsize = (return_val_size *= 2) * sizeof (char *);
-         return_val = (char **) xrealloc ((char *) return_val, newsize);
+         return;
        }
-      return_val[return_val_index++] = savestring (symname, strlen (symname));
-      return_val[return_val_index] = NULL;
     }
+  
+  /* We have a match for a completion, so add SYMNAME to the current list
+     of matches. Note that the name is moved to freshly malloc'd space. */
 
-  if (demangled != NULL)
-    {
-      free (demangled);
-    }
+  {
+    char *new;
+    if (word == sym_text)
+      {
+       new = xmalloc (strlen (symname) + 5);
+       strcpy (new, symname);
+      }
+    else if (word > sym_text)
+      {
+       /* Return some portion of symname.  */
+       new = xmalloc (strlen (symname) + 5);
+       strcpy (new, symname + (word - sym_text));
+      }
+    else
+      {
+       /* Return some of SYM_TEXT plus symname.  */
+       new = xmalloc (strlen (symname) + (sym_text - word) + 5);
+       strncpy (new, word, sym_text - word);
+       new[sym_text - word] = '\0';
+       strcat (new, symname);
+      }
+
+    if (return_val_index + 3 > return_val_size)
+      {
+       newsize = (return_val_size *= 2) * sizeof (char *);
+       return_val = (char **) xrealloc ((char *) return_val, newsize);
+      }
+    return_val[return_val_index++] = new;
+    return_val[return_val_index] = NULL;
+  }
 }
 
 /* Return a NULL terminated array of all symbols (regardless of class) which
@@ -2428,8 +2753,9 @@ completion_list_add_symbol (symname, text, text_len)
    I'm not going to worry about this; hopefully there won't be that many.  */
 
 char **
-make_symbol_completion_list (text)
-  char *text;
+make_symbol_completion_list (text, word)
+     char *text;
+     char *word;
 {
   register struct symbol *sym;
   register struct symtab *s;
@@ -2438,17 +2764,70 @@ make_symbol_completion_list (text)
   register struct objfile *objfile;
   register struct block *b, *surrounding_static_block = 0;
   register int i, j;
-  int text_len;
   struct partial_symbol *psym;
+  /* The symbol we are completing on.  Points in same buffer as text.  */
+  char *sym_text;
+  /* Length of sym_text.  */
+  int sym_text_len;
+
+  /* Now look for the symbol we are supposed to complete on.
+     FIXME: This should be language-specific.  */
+  {
+    char *p;
+    char quote_found;
+    char *quote_pos;
+
+    /* First see if this is a quoted string.  */
+    quote_found = '\0';
+    for (p = text; *p != '\0'; ++p)
+      {
+       if (quote_found != '\0')
+         {
+           if (*p == quote_found)
+             /* Found close quote.  */
+             quote_found = '\0';
+           else if (*p == '\\' && p[1] == quote_found)
+             /* A backslash followed by the quote character
+                doesn't end the string.  */
+             ++p;
+         }
+       else if (*p == '\'' || *p == '"')
+         {
+           quote_found = *p;
+           quote_pos = p;
+         }
+      }
+    if (quote_found == '\'')
+      /* A string within single quotes can be a symbol, so complete on it.  */
+      sym_text = quote_pos + 1;
+    else if (quote_found == '"')
+      /* A double-quoted string is never a symbol, nor does it make sense
+        to complete it any other way.  */
+      return NULL;
+    else
+      {
+       /* It is not a quoted string.  Break it based on the characters
+          which are in symbols.  */
+       while (p > text)
+         {
+           if (isalnum (p[-1]) || p[-1] == '_' || p[-1] == '\0')
+             --p;
+           else
+             break;
+         }
+       sym_text = p;
+      }
+  }
+
+  sym_text_len = strlen (sym_text);
 
-  text_len = strlen (text);
   return_val_size = 100;
   return_val_index = 0;
   return_val = (char **) xmalloc ((return_val_size + 1) * sizeof (char *));
   return_val[0] = NULL;
 
   /* Look through the partial symtabs for all symbols which begin
-     by matching TEXT.  Add each one that you find to the list.  */
+     by matching SYM_TEXT.  Add each one that you find to the list.  */
 
   ALL_PSYMTABS (objfile, ps)
     {
@@ -2463,7 +2842,7 @@ make_symbol_completion_list (text)
        {
          /* If interrupted, then quit. */
          QUIT;
-         completion_list_add_symbol (SYMBOL_NAME (psym), text, text_len);
+         COMPLETION_LIST_ADD_SYMBOL (psym, sym_text, sym_text_len, text, word);
        }
       
       for (psym = objfile->static_psymbols.list + ps->statics_offset;
@@ -2472,7 +2851,7 @@ make_symbol_completion_list (text)
           psym++)
        {
          QUIT;
-         completion_list_add_symbol (SYMBOL_NAME (psym), text, text_len);
+         COMPLETION_LIST_ADD_SYMBOL (psym, sym_text, sym_text_len, text, word);
        }
     }
 
@@ -2484,7 +2863,7 @@ make_symbol_completion_list (text)
   ALL_MSYMBOLS (objfile, msymbol)
     {
       QUIT;
-      completion_list_add_symbol (msymbol -> name, text, text_len);
+      COMPLETION_LIST_ADD_SYMBOL (msymbol, sym_text, sym_text_len, text, word);
     }
 
   /* Search upwards from currently selected frame (so that we can
@@ -2503,7 +2882,7 @@ make_symbol_completion_list (text)
       for (i = 0; i < BLOCK_NSYMS (b); i++)
        {
          sym = BLOCK_SYM (b, i);
-         completion_list_add_symbol (SYMBOL_NAME (sym), text, text_len);
+         COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word);
          if (SYMBOL_CLASS (sym) == LOC_TYPEDEF)
            {
              struct type *t = SYMBOL_TYPE (sym);
@@ -2515,8 +2894,8 @@ make_symbol_completion_list (text)
                    {
                      if (TYPE_FIELD_NAME (t, j))
                        {
-                         completion_list_add_symbol (TYPE_FIELD_NAME (t, j),
-                                                     text, text_len);
+                         completion_list_add_name (TYPE_FIELD_NAME (t, j),
+                                                     sym_text, sym_text_len, text, word);
                        }
                    }
                }
@@ -2534,7 +2913,7 @@ make_symbol_completion_list (text)
       for (i = 0; i < BLOCK_NSYMS (b); i++)
        {
          sym = BLOCK_SYM (b, i);
-         completion_list_add_symbol (SYMBOL_NAME (sym), text, text_len);
+         COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word);
        }
     }
 
@@ -2547,174 +2926,13 @@ make_symbol_completion_list (text)
       for (i = 0; i < BLOCK_NSYMS (b); i++)
        {
          sym = BLOCK_SYM (b, i);
-         completion_list_add_symbol (SYMBOL_NAME (sym), text, text_len);
+         COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word);
        }
     }
 
   return (return_val);
 }
 
-\f
-/* Find a mangled symbol that corresponds to LOOKFOR using brute force.
-   Basically we go munging through available symbols, demangling each one,
-   looking for a match on the demangled result. */
-
-static char *
-expensive_mangler (lookfor)
-     const char *lookfor;
-{
-  register struct symbol *sym;
-  register struct symtab *s;
-  register struct partial_symtab *ps;
-  register struct minimal_symbol *msymbol;
-  register struct objfile *objfile;
-  register struct block *b, *surrounding_static_block = 0;
-  register int i, j;
-  struct partial_symbol *psym;
-  char *demangled;
-
-  /* Look through the partial symtabs for a symbol that matches */
-
-  ALL_PSYMTABS (objfile, ps)
-    {
-      /* If the psymtab's been read in we'll get it when we search
-        through the blockvector.  */
-      if (ps->readin) continue;
-      
-      for (psym = objfile->global_psymbols.list + ps->globals_offset;
-          psym < (objfile->global_psymbols.list + ps->globals_offset
-                  + ps->n_global_syms);
-          psym++)
-       {
-         QUIT;                 /* If interrupted, then quit. */
-         demangled = demangle_and_match (SYMBOL_NAME (psym), lookfor,
-                                         DMGL_PARAMS | DMGL_ANSI);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (SYMBOL_NAME (psym));
-           }
-       }
-      
-      for (psym = objfile->static_psymbols.list + ps->statics_offset;
-          psym < (objfile->static_psymbols.list + ps->statics_offset
-                  + ps->n_static_syms);
-          psym++)
-       {
-         QUIT;
-         demangled = demangle_and_match (SYMBOL_NAME (psym), lookfor,
-                                         DMGL_PARAMS | DMGL_ANSI);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (SYMBOL_NAME (psym));
-           }
-       }
-    }
-
-  /* Scan through the misc symbol vectors looking for a match. */
-
-  ALL_MSYMBOLS (objfile, msymbol)
-    {
-      QUIT;
-      demangled = demangle_and_match (msymbol -> name, lookfor,
-                                     DMGL_PARAMS | DMGL_ANSI);
-      if (demangled != NULL)
-       {
-         free (demangled);
-         return (msymbol -> name);
-       }
-    }
-
-  /* Search upwards from currently selected frame looking for a match */
-
-  for (b = get_selected_block (); b; b = BLOCK_SUPERBLOCK (b))
-    {
-      if (!BLOCK_SUPERBLOCK (b))
-       surrounding_static_block = b; /* For elmin of dups */
-      
-      /* Also catch fields of types defined in this places which
-        match our text string.  Only complete on types visible
-        from current context.  */
-      for (i = 0; i < BLOCK_NSYMS (b); i++)
-       {
-         sym = BLOCK_SYM (b, i);
-         demangled = demangle_and_match (SYMBOL_NAME (sym), lookfor,
-                                         DMGL_PARAMS | DMGL_ANSI);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (SYMBOL_NAME (sym));
-           }
-         if (SYMBOL_CLASS (sym) == LOC_TYPEDEF)
-           {
-             struct type *t = SYMBOL_TYPE (sym);
-             enum type_code c = TYPE_CODE (t);
-
-             if (c == TYPE_CODE_UNION || c == TYPE_CODE_STRUCT)
-               {
-                 for (j = TYPE_N_BASECLASSES (t); j < TYPE_NFIELDS (t); j++)
-                   {
-                     if (TYPE_FIELD_NAME (t, j))
-                       {
-                         demangled =
-                           demangle_and_match (TYPE_FIELD_NAME (t, j),
-                                               lookfor,
-                                               DMGL_PARAMS | DMGL_ANSI);
-                         if (demangled != NULL)
-                           {
-                             free (demangled);
-                             return (TYPE_FIELD_NAME (t, j));
-                           }
-                       }
-                   }
-               }
-           }
-       }
-    }
-
-  /* Go through the symtabs and check the externs and statics for
-     symbols which match.  */
-
-  ALL_SYMTABS (objfile, s)
-    {
-      QUIT;
-      b = BLOCKVECTOR_BLOCK (BLOCKVECTOR (s), GLOBAL_BLOCK);
-      for (i = 0; i < BLOCK_NSYMS (b); i++)
-       {
-         sym = BLOCK_SYM (b, i);
-         demangled = demangle_and_match (SYMBOL_NAME (sym), lookfor,
-                                         DMGL_PARAMS | DMGL_ANSI);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (SYMBOL_NAME (sym));
-           }
-       }
-    }
-
-  ALL_SYMTABS (objfile, s)
-    {
-      QUIT;
-      b = BLOCKVECTOR_BLOCK (BLOCKVECTOR (s), STATIC_BLOCK);
-      /* Don't do this block twice.  */
-      if (b == surrounding_static_block) continue;
-      for (i = 0; i < BLOCK_NSYMS (b); i++)
-       {
-         sym = BLOCK_SYM (b, i);
-         demangled = demangle_and_match (SYMBOL_NAME (sym), lookfor,
-                                         DMGL_PARAMS | DMGL_ANSI);
-         if (demangled != NULL)
-           {
-             free (demangled);
-             return (SYMBOL_NAME (sym));
-           }
-       }
-    }
-
-  return (NULL);
-}
-
 \f
 #if 0
 /* Add the type of the symbol sym to the type of the current
This page took 0.051253 seconds and 4 git commands to generate.