*** empty log message ***
[deliverable/binutils-gdb.git] / binutils / stabs.c
index e1c98ed48ed42a29eda519ff8d98a8370ca3475e..e2b8a2affa2819f364a3a48540dbb5c2c5c14a14 100644 (file)
@@ -1,5 +1,6 @@
 /* stabs.c -- Parse stabs debugging information
-   Copyright (C) 1995, 1996, 1997, 1998 Free Software Foundation, Inc.
+   Copyright 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003
+   Free Software Foundation, Inc.
    Written by Ian Lance Taylor <ian@cygnus.com>.
 
    This file is part of GNU Binutils.
    trying to identify the correct address for anything.  */
 
 #include <stdio.h>
-#include <ctype.h>
 
 #include "bfd.h"
 #include "bucomm.h"
 #include "libiberty.h"
+#include "safe-ctype.h"
 #include "demangle.h"
 #include "debug.h"
 #include "budbg.h"
-
-/* Meaningless definition needs by aout64.h.  FIXME.  */
-#define BYTES_IN_WORD 4
-
+#include "filenames.h"
 #include "aout/aout64.h"
 #include "aout/stab_gnu.h"
 
-#ifndef DIR_SEPARATOR
-#define DIR_SEPARATOR '/'
-#endif
-
 /* The number of predefined XCOFF types.  */
 
 #define XCOFF_TYPE_COUNT 34
@@ -55,8 +49,8 @@ struct stab_handle
 {
   /* The BFD.  */
   bfd *abfd;
-  /* True if this is stabs in sections.  */
-  boolean sections;
+  /* TRUE if this is stabs in sections.  */
+  bfd_boolean sections;
   /* The symbol table.  */
   asymbol **syms;
   /* The number of symbols.  */
@@ -76,7 +70,7 @@ struct stab_handle
   int gcc_compiled;
   /* Whether an N_OPT symbol was seen that was not generated by gcc,
      so that we can detect the SunPRO compiler.  */
-  boolean n_opt_found;
+  bfd_boolean n_opt_found;
   /* The main file name.  */
   char *main_filename;
   /* A stack of unfinished N_BINCL files.  */
@@ -84,7 +78,7 @@ struct stab_handle
   /* A list of finished N_BINCL files.  */
   struct bincl_file *bincl_list;
   /* Whether we are inside a function or not.  */
-  boolean within_function;
+  bfd_boolean within_function;
   /* The address of the end of the function, used if we have seen an
      N_FUN symbol while in a function.  This is -1 if we have not seen
      an N_FUN (the normal case).  */
@@ -103,7 +97,7 @@ struct stab_handle
   struct stab_tag *tags;
   /* Set by parse_stab_type if it sees a structure defined as a cross
      reference to itself.  Reset by parse_stab_type otherwise.  */
-  boolean self_crossref;
+  bfd_boolean self_crossref;
 };
 
 /* A list of these structures is used to hold pending variable
@@ -153,76 +147,65 @@ struct stab_tag
   debug_type type;
 };
 
-static char *savestring PARAMS ((const char *, int));
-static bfd_vma parse_number PARAMS ((const char **, boolean *));
-static void bad_stab PARAMS ((const char *));
-static void warn_stab PARAMS ((const char *, const char *));
-static boolean parse_stab_string
-  PARAMS ((PTR, struct stab_handle *, int, int, bfd_vma, const char *));
+static char *savestring (const char *, int);
+static bfd_vma parse_number (const char **, bfd_boolean *);
+static void bad_stab (const char *);
+static void warn_stab (const char *, const char *);
+static bfd_boolean parse_stab_string
+  (void *, struct stab_handle *, int, int, bfd_vma, const char *);
 static debug_type parse_stab_type
-  PARAMS ((PTR, struct stab_handle *, const char *, const char **,
-          debug_type **));
-static boolean parse_stab_type_number
-  PARAMS ((const char **, int *));
+  (void *, struct stab_handle *, const char *, const char **, debug_type **);
+static bfd_boolean parse_stab_type_number (const char **, int *);
 static debug_type parse_stab_range_type
-  PARAMS ((PTR, struct stab_handle *, const char *, const char **,
-          const int *));
-static debug_type parse_stab_sun_builtin_type PARAMS ((PTR, const char **));
-static debug_type parse_stab_sun_floating_type
-  PARAMS ((PTR, const char **));
-static debug_type parse_stab_enum_type PARAMS ((PTR, const char **));
+  (void *, struct stab_handle *, const char *, const char **, const int *);
+static debug_type parse_stab_sun_builtin_type (void *, const char **);
+static debug_type parse_stab_sun_floating_type (void *, const char **);
+static debug_type parse_stab_enum_type (void *, const char **);
 static debug_type parse_stab_struct_type
-  PARAMS ((PTR, struct stab_handle *, const char *, const char **, boolean,
-          const int *));
-static boolean parse_stab_baseclasses
-  PARAMS ((PTR, struct stab_handle *, const char **, debug_baseclass **));
-static boolean parse_stab_struct_fields
-  PARAMS ((PTR, struct stab_handle *, const char **, debug_field **,
-          boolean *));
-static boolean parse_stab_cpp_abbrev
-  PARAMS ((PTR, struct stab_handle *, const char **, debug_field *));
-static boolean parse_stab_one_struct_field
-  PARAMS ((PTR, struct stab_handle *, const char **, const char *,
-          debug_field *, boolean *));
-static boolean parse_stab_members
-  PARAMS ((PTR, struct stab_handle *, const char *, const char **,
-          const int *, debug_method **));
+  (void *, struct stab_handle *, const char *, const char **,
+   bfd_boolean, const int *);
+static bfd_boolean parse_stab_baseclasses
+  (void *, struct stab_handle *, const char **, debug_baseclass **);
+static bfd_boolean parse_stab_struct_fields
+  (void *, struct stab_handle *, const char **, debug_field **, bfd_boolean *);
+static bfd_boolean parse_stab_cpp_abbrev
+  (void *, struct stab_handle *, const char **, debug_field *);
+static bfd_boolean parse_stab_one_struct_field
+  (void *, struct stab_handle *, const char **, const char *,
+   debug_field *, bfd_boolean *);
+static bfd_boolean parse_stab_members
+  (void *, struct stab_handle *, const char *, const char **, const int *,
+   debug_method **);
 static debug_type parse_stab_argtypes
-  PARAMS ((PTR, struct stab_handle *, debug_type, const char *, const char *,
-          debug_type, const char *, boolean, boolean, const char **));
-static boolean parse_stab_tilde_field
-  PARAMS ((PTR, struct stab_handle *, const char **, const int *,
-          debug_type *, boolean *));
+  (void *, struct stab_handle *, debug_type, const char *, const char *,
+   debug_type, const char *, bfd_boolean, bfd_boolean, const char **);
+static bfd_boolean parse_stab_tilde_field
+  (void *, struct stab_handle *, const char **, const int *, debug_type *,
+   bfd_boolean *);
 static debug_type parse_stab_array_type
-  PARAMS ((PTR, struct stab_handle *, const char **, boolean));
-static void push_bincl PARAMS ((struct stab_handle *, const char *, bfd_vma));
-static const char *pop_bincl PARAMS ((struct stab_handle *));
-static boolean find_excl
-  PARAMS ((struct stab_handle *, const char *, bfd_vma));
-static boolean stab_record_variable
-  PARAMS ((PTR, struct stab_handle *, const char *, debug_type,
-          enum debug_var_kind, bfd_vma));
-static boolean stab_emit_pending_vars PARAMS ((PTR, struct stab_handle *));
-static debug_type *stab_find_slot
-  PARAMS ((struct stab_handle *, const int *));
-static debug_type stab_find_type
-  PARAMS ((PTR, struct stab_handle *, const int *));
-static boolean stab_record_type
-  PARAMS ((PTR, struct stab_handle *, const int *, debug_type));
+  (void *, struct stab_handle *, const char **, bfd_boolean);
+static void push_bincl (struct stab_handle *, const char *, bfd_vma);
+static const char *pop_bincl (struct stab_handle *);
+static bfd_boolean find_excl (struct stab_handle *, const char *, bfd_vma);
+static bfd_boolean stab_record_variable
+  (void *, struct stab_handle *, const char *, debug_type,
+   enum debug_var_kind, bfd_vma);
+static bfd_boolean stab_emit_pending_vars (void *, struct stab_handle *);
+static debug_type *stab_find_slot (struct stab_handle *, const int *);
+static debug_type stab_find_type (void *, struct stab_handle *, const int *);
+static bfd_boolean stab_record_type
+  (void *, struct stab_handle *, const int *, debug_type);
 static debug_type stab_xcoff_builtin_type
-  PARAMS ((PTR, struct stab_handle *, int));
+  (void *, struct stab_handle *, int);
 static debug_type stab_find_tagged_type
-  PARAMS ((PTR, struct stab_handle *, const char *, int,
-          enum debug_type_kind));
+  (void *, struct stab_handle *, const char *, int, enum debug_type_kind);
 static debug_type *stab_demangle_argtypes
-  PARAMS ((PTR, struct stab_handle *, const char *, boolean *));
+  (void *, struct stab_handle *, const char *, bfd_boolean *, unsigned int);
 
 /* Save a string in memory.  */
 
 static char *
-savestring (start, len)
-     const char *start;
-     int len;
+savestring (const char *start, int len)
 {
   char *ret;
 
@@ -235,15 +218,13 @@ savestring (start, len)
 /* Read a number from a string.  */
 
 static bfd_vma
-parse_number (pp, poverflow)
-     const char **pp;
-     boolean *poverflow;
+parse_number (const char **pp, bfd_boolean *poverflow)
 {
   unsigned long ul;
   const char *orig;
 
   if (poverflow != NULL)
-    *poverflow = false;
+    *poverflow = FALSE;
 
   orig = *pp;
 
@@ -261,26 +242,24 @@ parse_number (pp, poverflow)
 
   /* Note that even though strtoul overflowed, it should have set *pp
      to the end of the number, which is where we want it.  */
-
   if (sizeof (bfd_vma) > sizeof (unsigned long))
     {
       const char *p;
-      boolean neg;
+      bfd_boolean neg;
       int base;
       bfd_vma over, lastdig;
-      boolean overflow;
+      bfd_boolean overflow;
       bfd_vma v;
 
       /* Our own version of strtoul, for a bfd_vma.  */
-
       p = orig;
 
-      neg = false;
+      neg = FALSE;
       if (*p == '+')
        ++p;
       else if (*p == '-')
        {
-         neg = true;
+         neg = TRUE;
          ++p;
        }
 
@@ -302,18 +281,18 @@ parse_number (pp, poverflow)
       over = ((bfd_vma) (bfd_signed_vma) -1) / (bfd_vma) base;
       lastdig = ((bfd_vma) (bfd_signed_vma) -1) % (bfd_vma) base;
 
-      overflow = false;
+      overflow = FALSE;
       v = 0;
       while (1)
        {
          int d;
 
          d = *p++;
-         if (isdigit ((unsigned char) d))
+         if (ISDIGIT (d))
            d -= '0';
-         else if (isupper ((unsigned char) d))
+         else if (ISUPPER (d))
            d -= 'A';
-         else if (islower ((unsigned char) d))
+         else if (ISLOWER (d))
            d -= 'a';
          else
            break;
@@ -323,7 +302,7 @@ parse_number (pp, poverflow)
 
          if (v > over || (v == over && (bfd_vma) d > lastdig))
            {
-             overflow = true;
+             overflow = TRUE;
              break;
            }
        }
@@ -338,9 +317,8 @@ parse_number (pp, poverflow)
 
   /* If we get here, the number is too large to represent in a
      bfd_vma.  */
-
   if (poverflow != NULL)
-    *poverflow = true;
+    *poverflow = TRUE;
   else
     warn_stab (orig, _("numeric overflow"));
 
@@ -350,8 +328,7 @@ parse_number (pp, poverflow)
 /* Give an error for a bad stab string.  */
 
 static void
-bad_stab (p)
-     const char *p;
+bad_stab (const char *p)
 {
   fprintf (stderr, _("Bad stab: %s\n"), p);
 }
@@ -359,23 +336,16 @@ bad_stab (p)
 /* Warn about something in a stab string.  */
 
 static void
-warn_stab (p, err)
-     const char *p;
-     const char *err;
+warn_stab (const char *p, const char *err)
 {
   fprintf (stderr, _("Warning: %s: %s\n"), err, p);
 }
 
 /* Create a handle to parse stabs symbols with.  */
 
-/*ARGSUSED*/
-PTR
-start_stab (dhandle, abfd, sections, syms, symcount)
-     PTR dhandle;
-     bfd *abfd;
-     boolean sections;
-     asymbol **syms;
-     long symcount;
+void *
+start_stab (void *dhandle ATTRIBUTE_UNUSED, bfd *abfd, bfd_boolean sections,
+           asymbol **syms, long symcount)
 {
   struct stab_handle *ret;
 
@@ -389,16 +359,14 @@ start_stab (dhandle, abfd, sections, syms, symcount)
   ret->file_types = (struct stab_types **) xmalloc (sizeof *ret->file_types);
   ret->file_types[0] = NULL;
   ret->function_end = (bfd_vma) -1;
-  return (PTR) ret;
+  return (void *) ret;
 }
 
 /* When we have processed all the stabs information, we need to go
    through and fill in all the undefined tags.  */
 
-boolean
-finish_stab (dhandle, handle)
-     PTR dhandle;
-     PTR handle;
+bfd_boolean
+finish_stab (void *dhandle, void *handle)
 {
   struct stab_handle *info = (struct stab_handle *) handle;
   struct stab_tag *st;
@@ -407,8 +375,8 @@ finish_stab (dhandle, handle)
     {
       if (! stab_emit_pending_vars (dhandle, info)
          || ! debug_end_function (dhandle, info->function_end))
-       return false;
-      info->within_function = false;
+       return FALSE;
+      info->within_function = FALSE;
       info->function_end = (bfd_vma) -1;
     }
 
@@ -421,22 +389,17 @@ finish_stab (dhandle, handle)
        kind = DEBUG_KIND_STRUCT;
       st->slot = debug_make_undefined_tagged_type (dhandle, st->name, kind);
       if (st->slot == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Handle a single stabs symbol.  */
 
-boolean
-parse_stab (dhandle, handle, type, desc, value, string)
-     PTR dhandle;
-     PTR handle;
-     int type;
-     int desc;
-     bfd_vma value;
-     const char *string;
+bfd_boolean
+parse_stab (void *dhandle, void *handle, int type, int desc, bfd_vma value,
+           const char *string)
 {
   struct stab_handle *info = (struct stab_handle *) handle;
 
@@ -448,11 +411,11 @@ parse_stab (dhandle, handle, type, desc, value, string)
       && (type != N_SO || *string == '\0' || value != info->so_value))
     {
       if (! debug_set_filename (dhandle, info->so_string))
-       return false;
+       return FALSE;
       info->main_filename = info->so_string;
 
       info->gcc_compiled = 0;
-      info->n_opt_found = false;
+      info->n_opt_found = FALSE;
 
       /* Generally, for stabs in the symbol table, the N_LBRAC and
         N_RBRAC symbols are relative to the N_SO symbol value.  */
@@ -486,7 +449,7 @@ parse_stab (dhandle, handle, type, desc, value, string)
       if (! info->within_function)
        {
          fprintf (stderr, _("N_LBRAC not within function\n"));
-         return false;
+         return FALSE;
        }
 
       /* Start an inner lexical block.  */
@@ -494,11 +457,11 @@ parse_stab (dhandle, handle, type, desc, value, string)
                               (value
                                + info->file_start_offset
                                + info->function_start_offset)))
-       return false;
+       return FALSE;
 
       /* Emit any pending variable definitions.  */
       if (! stab_emit_pending_vars (dhandle, info))
-       return false;
+       return FALSE;
 
       ++info->block_depth;
       break;
@@ -512,20 +475,20 @@ parse_stab (dhandle, handle, type, desc, value, string)
          if we do, we probably need to emit them before closing the
          block.  */
       if (! stab_emit_pending_vars (dhandle, info))
-       return false;
+       return FALSE;
 
       /* End an inner lexical block.  */
       if (! debug_end_block (dhandle,
                             (value
                              + info->file_start_offset
                              + info->function_start_offset)))
-       return false;
+       return FALSE;
 
       --info->block_depth;
       if (info->block_depth < 0)
        {
          fprintf (stderr, _("Too many N_RBRACs\n"));
-         return false;
+         return FALSE;
        }
       break;
 
@@ -542,15 +505,15 @@ parse_stab (dhandle, handle, type, desc, value, string)
            endval = info->function_end;
          if (! stab_emit_pending_vars (dhandle, info)
              || ! debug_end_function (dhandle, endval))
-           return false;
-         info->within_function = false;
+           return FALSE;
+         info->within_function = FALSE;
          info->function_end = (bfd_vma) -1;
        }
 
       /* An empty string is emitted by gcc at the end of a compilation
          unit.  */
       if (*string == '\0')
-       return true;
+       return TRUE;
 
       /* Just accumulate strings until we see a non N_SO symbol.  If
          the string starts with a directory separator or some other
@@ -564,12 +527,7 @@ parse_stab (dhandle, handle, type, desc, value, string)
 
          f = info->so_string;
 
-         if (   (string[0] == '/')
-             || (string[0] == DIR_SEPARATOR)
-             || (   (DIR_SEPARATOR == '\\')
-                 && (string[1] == ':')
-                 && (   (string[2] == DIR_SEPARATOR)
-                     || (string[2] == '/'))))
+         if (IS_ABSOLUTE_PATH (string))
            info->so_string = xstrdup (string);
          else
            info->so_string = concat (info->so_string, string,
@@ -584,43 +542,44 @@ parse_stab (dhandle, handle, type, desc, value, string)
     case N_SOL:
       /* Start an include file.  */
       if (! debug_start_source (dhandle, string))
-       return false;
+       return FALSE;
       break;
 
     case N_BINCL:
       /* Start an include file which may be replaced.  */
       push_bincl (info, string, value);
       if (! debug_start_source (dhandle, string))
-       return false;
+       return FALSE;
       break;
 
     case N_EINCL:
       /* End an N_BINCL include.  */
       if (! debug_start_source (dhandle, pop_bincl (info)))
-       return false;
+       return FALSE;
       break;
 
     case N_EXCL:
       /* This is a duplicate of a header file named by N_BINCL which
          was eliminated by the linker.  */
       if (! find_excl (info, string, value))
-       return false;
+       return FALSE;
       break;
 
     case N_SLINE:
       if (! debug_record_line (dhandle, desc,
-                              value + info->function_start_offset))
-       return false;
+                              value + (info->within_function
+                                       ? info->function_start_offset : 0)))
+       return FALSE;
       break;
 
     case N_BCOMM:
       if (! debug_start_common_block (dhandle, string))
-       return false;
+       return FALSE;
       break;
 
     case N_ECOMM:
       if (! debug_end_common_block (dhandle, string))
-       return false;
+       return FALSE;
       break;
 
     case N_FUN:
@@ -634,8 +593,8 @@ parse_stab (dhandle, handle, type, desc, value, string)
                value += info->function_start_offset;
              if (! stab_emit_pending_vars (dhandle, info)
                  || ! debug_end_function (dhandle, value))
-               return false;
-             info->within_function = false;
+               return FALSE;
+             info->within_function = FALSE;
              info->function_end = (bfd_vma) -1;
            }
          break;
@@ -675,18 +634,18 @@ parse_stab (dhandle, handle, type, desc, value, string)
                  endval = info->function_end;
                if (! stab_emit_pending_vars (dhandle, info)
                    || ! debug_end_function (dhandle, endval))
-                 return false;
+                 return FALSE;
                info->function_end = (bfd_vma) -1;
              }
            /* For stabs in sections, line numbers and block addresses
                are offsets from the start of the function.  */
            if (info->sections)
              info->function_start_offset = value;
-           info->within_function = true;
+           info->within_function = TRUE;
          }
 
        if (! parse_stab_string (dhandle, info, type, desc, value, string))
-         return false;
+         return FALSE;
       }
       break;
 
@@ -696,41 +655,37 @@ parse_stab (dhandle, handle, type, desc, value, string)
       else if (string != NULL && strcmp (string, "gcc_compiled.") == 0)
        info->gcc_compiled = 1;
       else
-       info->n_opt_found = true;
+       info->n_opt_found = TRUE;
       break;
 
     case N_OBJ:
     case N_ENDM:
     case N_MAIN:
+    case N_WARNING:
       break;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Parse the stabs string.  */
 
-static boolean
-parse_stab_string (dhandle, info, stabtype, desc, value, string)
-     PTR dhandle;
-     struct stab_handle *info;
-     int stabtype;
-     int desc;
-     bfd_vma value;
-     const char *string;
+static bfd_boolean
+parse_stab_string (void *dhandle, struct stab_handle *info, int stabtype,
+                  int desc, bfd_vma value, const char *string)
 {
   const char *p;
   char *name;
   int type;
   debug_type dtype;
-  boolean synonym;
-  boolean self_crossref;
+  bfd_boolean synonym;
+  bfd_boolean self_crossref;
   unsigned int lineno;
   debug_type *slot;
 
   p = strchr (string, ':');
   if (p == NULL)
-    return true;
+    return TRUE;
 
   while (p[1] == ':')
     {
@@ -739,7 +694,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       if (p == NULL)
        {
          bad_stab (string);
-         return false;
+         return FALSE;
        }
     }
 
@@ -787,7 +742,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
     }
 
   ++p;
-  if (isdigit ((unsigned char) *p) || *p == '(' || *p == '-')
+  if (ISDIGIT (*p) || *p == '(' || *p == '-')
     type = 'l';
   else
     type = *p++;
@@ -804,7 +759,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       if (*p != '=')
        {
          bad_stab (string);
-         return false;
+         return FALSE;
        }
       ++p;
       switch (*p++)
@@ -812,7 +767,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
        case 'r':
          /* Floating point constant.  */
          if (! debug_record_float_const (dhandle, name, atof (p)))
-           return false;
+           return FALSE;
          break;
        case 'i':
          /* Integer constant.  */
@@ -823,7 +778,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
             other languages probably should have at least unsigned as
             well as signed constants.  */
          if (! debug_record_int_const (dhandle, name, atoi (p)))
-           return false;
+           return FALSE;
          break;
        case 'e':
          /* SYMBOL:c=eTYPE,INTVALUE for a constant symbol whose value
@@ -833,18 +788,18 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
          dtype = parse_stab_type (dhandle, info, (const char *) NULL,
                                   &p, (debug_type **) NULL);
          if (dtype == DEBUG_TYPE_NULL)
-           return false;
+           return FALSE;
          if (*p != ',')
            {
              bad_stab (string);
-             return false;
+             return FALSE;
            }
          if (! debug_record_typed_const (dhandle, name, dtype, atoi (p)))
-           return false;
+           return FALSE;
          break;
        default:
          bad_stab (string);
-         return false;
+         return FALSE;
        }
 
       break;
@@ -854,9 +809,9 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL,
                               &p, (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! debug_record_label (dhandle, name, dtype, value))
-       return false;
+       return FALSE;
       break;
 
     case 'f':
@@ -865,9 +820,9 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! debug_record_function (dhandle, name, dtype, type == 'F', value))
-       return false;
+       return FALSE;
 
       /* Sun acc puts declared types of arguments here.  We don't care
         about their actual types (FIXME -- we should remember the whole
@@ -879,7 +834,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
          if (parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL)
              == DEBUG_TYPE_NULL)
-           return false;
+           return FALSE;
        }
 
       break;
@@ -895,7 +850,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
        dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                                 (debug_type **) NULL);
        if (dtype == DEBUG_TYPE_NULL)
-         return false;
+         return FALSE;
        leading = bfd_get_symbol_leading_char (info->abfd);
        for (c = info->symcount, ps = info->syms; c > 0; --c, ++ps)
          {
@@ -911,7 +866,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
          value = bfd_asymbol_value (*ps);
        if (! stab_record_variable (dhandle, info, name, dtype, DEBUG_GLOBAL,
                                    value))
-         return false;
+         return FALSE;
       }
       break;
 
@@ -923,10 +878,10 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! stab_record_variable (dhandle, info, name, dtype, DEBUG_LOCAL,
                                  value))
-       return false;
+       return FALSE;
       break;
 
     case 'p':
@@ -947,21 +902,20 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
              debug_type ftype;
 
              ftype = debug_make_function_type (dhandle, dtype,
-                                               (debug_type *) NULL, false);
+                                               (debug_type *) NULL, FALSE);
              dtype = debug_make_pointer_type (dhandle, ftype);
            }
        }
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! debug_record_parameter (dhandle, name, dtype, DEBUG_PARM_STACK,
                                    value))
-       return false;
+       return FALSE;
 
       /* FIXME: At this point gdb considers rearranging the parameter
         address on a big endian machine if it is smaller than an int.
         We have no way to do that, since we don't really know much
         about the target.  */
-
       break;
 
     case 'P':
@@ -974,7 +928,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
              if (parse_stab_type (dhandle, info, (const char *) NULL, &p,
                                   (debug_type **) NULL)
                  == DEBUG_TYPE_NULL)
-               return false;
+               return FALSE;
            }
          break;
        }
@@ -984,10 +938,10 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! debug_record_parameter (dhandle, name, dtype, DEBUG_PARM_REG,
                                    value))
-       return false;
+       return FALSE;
       break;
 
     case 'r':
@@ -995,41 +949,40 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! stab_record_variable (dhandle, info, name, dtype, DEBUG_REGISTER,
                                  value))
-       return false;
+       return FALSE;
 
       /* FIXME: At this point gdb checks to combine pairs of 'p' and
         'r' stabs into a single 'P' stab.  */
-
       break;
 
     case 'S':
-      /* Static symbol at top level of file */
+      /* Static symbol at top level of file */
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! stab_record_variable (dhandle, info, name, dtype, DEBUG_STATIC,
                                  value))
-       return false;
+       return FALSE;
       break;
 
     case 't':
       /* A typedef.  */
       dtype = parse_stab_type (dhandle, info, name, &p, &slot);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (name == NULL)
        {
          /* A nameless type.  Nothing to do.  */
-         return true;
+         return TRUE;
        }
 
       dtype = debug_name_type (dhandle, name, dtype);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
 
       if (slot != NULL)
        *slot = dtype;
@@ -1041,21 +994,21 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
         by 't' which means we are typedef'ing it as well.  */
       if (*p != 't')
        {
-         synonym = false;
-         /* FIXME: gdb sets synonym to true if the current language
+         synonym = FALSE;
+         /* FIXME: gdb sets synonym to TRUE if the current language
              is C++.  */
        }
       else
        {
-         synonym = true;
+         synonym = TRUE;
          ++p;
        }
 
       dtype = parse_stab_type (dhandle, info, name, &p, &slot);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (name == NULL)
-       return true;
+       return TRUE;
 
       /* INFO->SELF_CROSSREF is set by parse_stab_type if this type is
          a cross reference to itself.  These are generated by some
@@ -1064,7 +1017,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
 
       dtype = debug_tag_type (dhandle, name, dtype);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (slot != NULL)
        *slot = dtype;
 
@@ -1091,7 +1044,7 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
        {
          dtype = debug_name_type (dhandle, name, dtype);
          if (dtype == DEBUG_TYPE_NULL)
-           return false;
+           return FALSE;
 
          if (slot != NULL)
            *slot = dtype;
@@ -1104,11 +1057,11 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       /* FIXME: gdb checks os9k_stabs here.  */
       if (! stab_record_variable (dhandle, info, name, dtype,
                                  DEBUG_LOCAL_STATIC, value))
-       return false;
+       return FALSE;
       break;
 
     case 'v':
@@ -1116,10 +1069,10 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! debug_record_parameter (dhandle, name, dtype, DEBUG_PARM_REFERENCE,
                                    value))
-       return false;
+       return FALSE;
       break;
 
     case 'a':
@@ -1127,10 +1080,10 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! debug_record_parameter (dhandle, name, dtype, DEBUG_PARM_REF_REG,
                                    value))
-       return false;
+       return FALSE;
       break;
 
     case 'X':
@@ -1141,21 +1094,21 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
       dtype = parse_stab_type (dhandle, info, (const char *) NULL, &p,
                               (debug_type **) NULL);
       if (dtype == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
       if (! stab_record_variable (dhandle, info, name, dtype, DEBUG_LOCAL,
                                  value))
-       return false;
+       return FALSE;
       break;
 
     default:
       bad_stab (string);
-      return false;
+      return FALSE;
     }
 
   /* FIXME: gdb converts structure values to structure pointers in a
      couple of cases, depending upon the target.  */
 
-  return true;
+  return TRUE;
 }
 
 /* Parse a stabs type.  The typename argument is non-NULL if this is a
@@ -1164,17 +1117,12 @@ parse_stab_string (dhandle, info, stabtype, desc, value, string)
    store the slot used if the type is being defined.  */
 
 static debug_type
-parse_stab_type (dhandle, info, typename, pp, slotp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *typename;
-     const char **pp;
-     debug_type **slotp;
+parse_stab_type (void *dhandle, struct stab_handle *info, const char *typename, const char **pp, debug_type **slotp)
 {
   const char *orig;
   int typenums[2];
   int size;
-  boolean stringp;
+  bfd_boolean stringp;
   int descriptor;
   debug_type dtype;
 
@@ -1184,14 +1132,14 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
   orig = *pp;
 
   size = -1;
-  stringp = false;
+  stringp = FALSE;
 
-  info->self_crossref = false;
+  info->self_crossref = FALSE;
 
   /* Read type number if present.  The type number may be omitted.
      for instance in a two-dimensional array declared with type
      "ar1;1;10;ar1;1;10;4".  */
-  if (! isdigit ((unsigned char) **pp) && **pp != '(' && **pp != '-')
+  if (! ISDIGIT (**pp) && **pp != '(' && **pp != '-')
     {
       /* 'typenums=' not present, type is anonymous.  Read and return
         the definition, but don't put it in the type vector.  */
@@ -1203,11 +1151,9 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
        return DEBUG_TYPE_NULL;
 
       if (**pp != '=')
-       {
-         /* Type is not being defined here.  Either it already
-            exists, or this is a forward reference to it.  */
-         return stab_find_type (dhandle, info, typenums);
-       }
+       /* Type is not being defined here.  Either it already
+          exists, or this is a forward reference to it.  */
+       return stab_find_type (dhandle, info, typenums);
 
       /* Only set the slot if the type is being defined.  This means
          that the mapping from type numbers to types will only record
@@ -1234,11 +1180,9 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
          const char *p = *pp + 1;
          const char *attr;
 
-         if (isdigit ((unsigned char) *p) || *p == '(' || *p == '-')
-           {
-             /* Member type.  */
-             break;
-           }
+         if (ISDIGIT (*p) || *p == '(' || *p == '-')
+           /* Member type.  */
+           break;
 
          /* Type attributes.  */
          attr = p;
@@ -1257,12 +1201,13 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
            {
            case 's':
              size = atoi (attr + 1);
+             size /= 8;  /* Size is in bits.  We store it in bytes.  */
              if (size <= 0)
                size = -1;
              break;
 
            case 'S':
-             stringp = true;
+             stringp = TRUE;
              break;
 
            default:
@@ -1284,7 +1229,6 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
        const char *q1, *q2, *p;
 
        /* A cross reference to another type.  */
-
        switch (**pp)
          {
          case 's':
@@ -1312,14 +1256,21 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
            bad_stab (orig);
            return DEBUG_TYPE_NULL;
          }
-       while (q1 != NULL && p > q1 && p[1] == ':')
+       if (q1 != NULL && p > q1 && p[1] == ':')
          {
-           q2 = strchr (q1, '>');
-           if (q2 == NULL || q2 < p)
-             break;
-           p += 2;
-           p = strchr (p, ':');
-           if (p == NULL)
+           int nest = 0;
+
+           for (q2 = q1; *q2 != '\0'; ++q2)
+             {
+               if (*q2 == '<')
+                 ++nest;
+               else if (*q2 == '>')
+                 --nest;
+               else if (*q2 == ':' && nest == 0)
+                 break;
+             }
+           p = q2;
+           if (*p != ':')
              {
                bad_stab (orig);
                return DEBUG_TYPE_NULL;
@@ -1333,7 +1284,7 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
        if (typename != NULL
            && strncmp (typename, *pp, p - *pp) == 0
            && typename[p - *pp] == '\0')
-         info->self_crossref = true;
+         info->self_crossref = TRUE;
 
        dtype = stab_find_tagged_type (dhandle, info, *pp, p - *pp, code);
 
@@ -1358,7 +1309,6 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
        int xtypenums[2];
 
        /* This type is defined as another type.  */
-
        (*pp)--;
        hold = *pp;
 
@@ -1417,7 +1367,7 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
               (dhandle,
                parse_stab_type (dhandle, info, (const char *) NULL, pp,
                                 (debug_type **) NULL),
-               (debug_type *) NULL, false));
+               (debug_type *) NULL, FALSE));
       break;
 
     case 'k':
@@ -1488,7 +1438,7 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
          ++*pp;
          dtype = debug_make_method_type (dhandle, return_type,
                                          DEBUG_TYPE_NULL,
-                                         (debug_type *) NULL, false);
+                                         (debug_type *) NULL, FALSE);
        }
       else
        {
@@ -1497,7 +1447,7 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
          debug_type *args;
          unsigned int n;
          unsigned int alloc;
-         boolean varargs;
+         bfd_boolean varargs;
 
          domain = parse_stab_type (dhandle, info, (const char *) NULL,
                                    pp, (debug_type **) NULL);
@@ -1532,7 +1482,7 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
                {
                  alloc += 10;
                  args = ((debug_type *)
-                         xrealloc ((PTR) args, alloc * sizeof *args));
+                         xrealloc (args, alloc * sizeof *args));
                }
 
              args[n] = parse_stab_type (dhandle, info, (const char *) NULL,
@@ -1548,11 +1498,11 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
             the void type.  */
          if (n == 0
              || debug_get_type_kind (dhandle, args[n - 1]) != DEBUG_KIND_VOID)
-           varargs = true;
+           varargs = TRUE;
          else
            {
              --n;
-             varargs = false;
+             varargs = FALSE;
            }
 
          args[n] = DEBUG_TYPE_NULL;
@@ -1628,7 +1578,7 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
   if (size != -1)
     {
       if (! debug_record_type_size (dhandle, dtype, (unsigned int) size))
-       return false;
+       return DEBUG_TYPE_NULL;
     }
 
   return dtype;
@@ -1639,10 +1589,8 @@ parse_stab_type (dhandle, info, typename, pp, slotp)
    single number N is equivalent to (0,N).  Return the two numbers by
    storing them in the vector TYPENUMS.  */
 
-static boolean
-parse_stab_type_number (pp, typenums)
-     const char **pp;
-     int *typenums;
+static bfd_boolean
+parse_stab_type_number (const char **pp, int *typenums)
 {
   const char *orig;
 
@@ -1651,47 +1599,42 @@ parse_stab_type_number (pp, typenums)
   if (**pp != '(')
     {
       typenums[0] = 0;
-      typenums[1] = (int) parse_number (pp, (boolean *) NULL);
+      typenums[1] = (int) parse_number (pp, (bfd_boolean *) NULL);
     }
   else
     {
       ++*pp;
-      typenums[0] = (int) parse_number (pp, (boolean *) NULL);
+      typenums[0] = (int) parse_number (pp, (bfd_boolean *) NULL);
       if (**pp != ',')
        {
          bad_stab (orig);
-         return false;
+         return FALSE;
        }
       ++*pp;
-      typenums[1] = (int) parse_number (pp, (boolean *) NULL);
+      typenums[1] = (int) parse_number (pp, (bfd_boolean *) NULL);
       if (**pp != ')')
        {
          bad_stab (orig);
-         return false;
+         return FALSE;
        }
       ++*pp;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Parse a range type.  */
 
 static debug_type
-parse_stab_range_type (dhandle, info, typename, pp, typenums)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *typename;
-     const char **pp;
-     const int *typenums;
+parse_stab_range_type (void *dhandle, struct stab_handle *info, const char *typename, const char **pp, const int *typenums)
 {
   const char *orig;
   int rangenums[2];
-  boolean self_subrange;
+  bfd_boolean self_subrange;
   debug_type index_type;
   const char *s2, *s3;
   bfd_signed_vma n2, n3;
-  boolean ov2, ov3;
+  bfd_boolean ov2, ov3;
 
   orig = *pp;
 
@@ -1741,18 +1684,18 @@ parse_stab_range_type (dhandle, info, typename, pp, typenums)
     {
       /* gcc will emit range stabs for long long types.  Handle this
          as a special case.  FIXME: This needs to be more general.  */
-#define LLLOW  "01000000000000000000000;"
-#define LLHIGH "0777777777777777777777;"
+#define LLLOW   "01000000000000000000000;"
+#define LLHIGH   "0777777777777777777777;"
 #define ULLHIGH "01777777777777777777777;"
       if (index_type == DEBUG_TYPE_NULL)
        {
          if (strncmp (s2, LLLOW, sizeof LLLOW - 1) == 0
              && strncmp (s3, LLHIGH, sizeof LLHIGH - 1) == 0)
-           return debug_make_int_type (dhandle, 8, false);
+           return debug_make_int_type (dhandle, 8, FALSE);
          if (! ov2
              && n2 == 0
              && strncmp (s3, ULLHIGH, sizeof ULLHIGH - 1) == 0)
-           return debug_make_int_type (dhandle, 8, true);
+           return debug_make_int_type (dhandle, 8, TRUE);
        }
 
       warn_stab (orig, _("numeric overflow"));
@@ -1785,44 +1728,51 @@ parse_stab_range_type (dhandle, info, typename, pp, typenums)
          if (typename != NULL)
            {
              if (strcmp (typename, "long long int") == 0)
-               return debug_make_int_type (dhandle, 8, false);
+               return debug_make_int_type (dhandle, 8, FALSE);
              else if (strcmp (typename, "long long unsigned int") == 0)
-               return debug_make_int_type (dhandle, 8, true);
+               return debug_make_int_type (dhandle, 8, TRUE);
            }
          /* FIXME: The size here really depends upon the target.  */
-         return debug_make_int_type (dhandle, 4, true);
+         return debug_make_int_type (dhandle, 4, TRUE);
        }
 
       /* A range of 0 to 127 is char.  */
       if (self_subrange && n2 == 0 && n3 == 127)
-       return debug_make_int_type (dhandle, 1, false);
+       return debug_make_int_type (dhandle, 1, FALSE);
 
       /* FIXME: gdb checks for the language CHILL here.  */
 
       if (n2 == 0)
        {
          if (n3 < 0)
-           return debug_make_int_type (dhandle, - n3, true);
+           return debug_make_int_type (dhandle, - n3, TRUE);
          else if (n3 == 0xff)
-           return debug_make_int_type (dhandle, 1, true);
+           return debug_make_int_type (dhandle, 1, TRUE);
          else if (n3 == 0xffff)
-           return debug_make_int_type (dhandle, 2, true);
-         /* -1 is used for the upper bound of (4 byte) "unsigned int"
-            and "unsigned long", and we already checked for that, so
-            don't need to test for it here.  */
+           return debug_make_int_type (dhandle, 2, TRUE);
+         else if (n3 == (bfd_signed_vma) 0xffffffff)
+           return debug_make_int_type (dhandle, 4, TRUE);
+#ifdef BFD64
+         else if (n3 == ((((bfd_signed_vma) 0xffffffff) << 32) | 0xffffffff))
+           return debug_make_int_type (dhandle, 8, TRUE);
+#endif
        }
       else if (n3 == 0
               && n2 < 0
               && (self_subrange || n2 == -8))
-       return debug_make_int_type (dhandle, - n2, true);
-      else if (n2 == - n3 - 1)
+       return debug_make_int_type (dhandle, - n2, TRUE);
+      else if (n2 == - n3 - 1 || n2 == n3 + 1)
        {
          if (n3 == 0x7f)
-           return debug_make_int_type (dhandle, 1, false);
+           return debug_make_int_type (dhandle, 1, FALSE);
          else if (n3 == 0x7fff)
-           return debug_make_int_type (dhandle, 2, false);
+           return debug_make_int_type (dhandle, 2, FALSE);
          else if (n3 == 0x7fffffff)
-           return debug_make_int_type (dhandle, 4, false);
+           return debug_make_int_type (dhandle, 4, FALSE);
+#ifdef BFD64
+         else if (n3 == ((((bfd_vma) 0x7fffffff) << 32) | 0xffffffff))
+           return debug_make_int_type (dhandle, 8, FALSE);
+#endif
        }
     }
 
@@ -1841,7 +1791,7 @@ parse_stab_range_type (dhandle, info, typename, pp, typenums)
       /* Does this actually ever happen?  Is that why we are worrying
          about dealing with it rather than just calling error_type?  */
       warn_stab (orig, _("missing index type"));
-      index_type = debug_make_int_type (dhandle, 4, false);
+      index_type = debug_make_int_type (dhandle, 4, FALSE);
     }
 
   return debug_make_range_type (dhandle, index_type, n2, n3);
@@ -1860,12 +1810,10 @@ parse_stab_range_type (dhandle, info, typename, pp, typenums)
    FIXME.  */
 
 static debug_type
-parse_stab_sun_builtin_type (dhandle, pp)
-     PTR dhandle;
-     const char **pp;
+parse_stab_sun_builtin_type (void *dhandle, const char **pp)
 {
   const char *orig;
-  boolean unsignedp;
+  bfd_boolean unsignedp;
   bfd_vma bits;
 
   orig = *pp;
@@ -1873,10 +1821,10 @@ parse_stab_sun_builtin_type (dhandle, pp)
   switch (**pp)
     {
     case 's':
-      unsignedp = false;
+      unsignedp = FALSE;
       break;
     case 'u':
-      unsignedp = true;
+      unsignedp = TRUE;
       break;
     default:
       bad_stab (orig);
@@ -1895,7 +1843,7 @@ parse_stab_sun_builtin_type (dhandle, pp)
      by this type, except that unsigned short is 4 instead of 2.
      Since this information is redundant with the third number,
      we will ignore it.  */
-  (void) parse_number (pp, (boolean *) NULL);
+  (void) parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
@@ -1903,8 +1851,8 @@ parse_stab_sun_builtin_type (dhandle, pp)
     }
   ++*pp;
 
-  /* The second number is always 0, so ignore it too. */
-  (void) parse_number (pp, (boolean *) NULL);
+  /* The second number is always 0, so ignore it too.  */
+  (void) parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
@@ -1912,8 +1860,8 @@ parse_stab_sun_builtin_type (dhandle, pp)
     }
   ++*pp;
 
-  /* The third number is the number of bits for this type. */
-  bits = parse_number (pp, (boolean *) NULL);
+  /* The third number is the number of bits for this type.  */
+  bits = parse_number (pp, (bfd_boolean *) NULL);
 
   /* The type *should* end with a semicolon.  If it are embedded
      in a larger type the semicolon may be the only way to know where
@@ -1933,9 +1881,7 @@ parse_stab_sun_builtin_type (dhandle, pp)
 /* Parse a builtin floating type generated by the Sun compiler.  */
 
 static debug_type
-parse_stab_sun_floating_type (dhandle, pp)
-     PTR dhandle;
-     const char **pp;
+parse_stab_sun_floating_type (void *dhandle, const char **pp)
 {
   const char *orig;
   bfd_vma details;
@@ -1945,7 +1891,7 @@ parse_stab_sun_floating_type (dhandle, pp)
 
   /* The first number has more details about the type, for example
      FN_COMPLEX.  */
-  details = parse_number (pp, (boolean *) NULL);
+  details = parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
@@ -1953,7 +1899,7 @@ parse_stab_sun_floating_type (dhandle, pp)
     }
 
   /* The second number is the number of bytes occupied by this type */
-  bytes = parse_number (pp, (boolean *) NULL);
+  bytes = parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
@@ -1965,15 +1911,13 @@ parse_stab_sun_floating_type (dhandle, pp)
       || details == NF_COMPLEX32)
     return debug_make_complex_type (dhandle, bytes);
 
-  return debug_make_float_type (dhandle, bytes);      
+  return debug_make_float_type (dhandle, bytes);
 }
 
 /* Handle an enum type.  */
 
 static debug_type
-parse_stab_enum_type (dhandle, pp)
-     PTR dhandle;
-     const char **pp;
+parse_stab_enum_type (void *dhandle, const char **pp)
 {
   const char *orig;
   const char **names;
@@ -2014,7 +1958,7 @@ parse_stab_enum_type (dhandle, pp)
       name = savestring (*pp, p - *pp);
 
       *pp = p + 1;
-      val = (bfd_signed_vma) parse_number (pp, (boolean *) NULL);
+      val = (bfd_signed_vma) parse_number (pp, (bfd_boolean *) NULL);
       if (**pp != ',')
        {
          bad_stab (orig);
@@ -2026,9 +1970,9 @@ parse_stab_enum_type (dhandle, pp)
        {
          alloc += 10;
          names = ((const char **)
-                  xrealloc ((PTR) names, alloc * sizeof *names));
+                  xrealloc (names, alloc * sizeof *names));
          values = ((bfd_signed_vma *)
-                   xrealloc ((PTR) values, alloc * sizeof *values));
+                   xrealloc (values, alloc * sizeof *values));
        }
 
       names[n] = name;
@@ -2053,27 +1997,23 @@ parse_stab_enum_type (dhandle, pp)
    *PP will point to "4a:1,0,32;;".  */
 
 static debug_type
-parse_stab_struct_type (dhandle, info, tagname, pp, structp, typenums)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *tagname;
-     const char **pp;
-     boolean structp;
-     const int *typenums;
+parse_stab_struct_type (void *dhandle, struct stab_handle *info,
+                       const char *tagname, const char **pp,
+                       bfd_boolean structp, const int *typenums)
 {
   const char *orig;
   bfd_vma size;
   debug_baseclass *baseclasses;
   debug_field *fields;
-  boolean statics;
+  bfd_boolean statics;
   debug_method *methods;
   debug_type vptrbase;
-  boolean ownvptr;
+  bfd_boolean ownvptr;
 
   orig = *pp;
 
   /* Get the size.  */
-  size = parse_number (pp, (boolean *) NULL);
+  size = parse_number (pp, (bfd_boolean *) NULL);
 
   /* Get the other information.  */
   if (! parse_stab_baseclasses (dhandle, info, pp, &baseclasses)
@@ -2106,7 +2046,7 @@ parse_stab_struct_type (dhandle, info, tagname, pp, structp, typenums)
    the type for the base class, and a terminating semicolon.
 
    A typical example, with two base classes, would be "!2,020,19;0264,21;".
-                                                      ^^ ^ ^ ^  ^ ^  ^
+                                                      ^^ ^ ^ ^  ^ ^  ^
        Baseclass information marker __________________|| | | |  | |  |
        Number of baseclasses __________________________| | | |  | |  |
        Visibility specifiers (2) ________________________| | |  | |  |
@@ -2116,14 +2056,11 @@ parse_stab_struct_type (dhandle, info, tagname, pp, structp, typenums)
        Offset in bits from start of class ________________________|  |
        Type number of base class ____________________________________|
 
-  Return true for success, false for failure.  */
+  Return TRUE for success, FALSE for failure.  */
 
-static boolean
-parse_stab_baseclasses (dhandle, info, pp, retp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char **pp;
-     debug_baseclass **retp;
+static bfd_boolean
+parse_stab_baseclasses (void *dhandle, struct stab_handle *info,
+                       const char **pp, debug_baseclass **retp)
 {
   const char *orig;
   unsigned int c, i;
@@ -2136,16 +2073,16 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
   if (**pp != '!')
     {
       /* No base classes.  */
-      return true;
+      return TRUE;
     }
   ++*pp;
 
-  c = (unsigned int) parse_number (pp, (boolean *) NULL);
+  c = (unsigned int) parse_number (pp, (bfd_boolean *) NULL);
 
   if (**pp != ',')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
@@ -2153,7 +2090,7 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
 
   for (i = 0; i < c; i++)
     {
-      boolean virtual;
+      bfd_boolean virtual;
       enum debug_visibility visibility;
       bfd_vma bitpos;
       debug_type type;
@@ -2161,14 +2098,14 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
       switch (**pp)
        {
        case '0':
-         virtual = false;
+         virtual = FALSE;
          break;
        case '1':
-         virtual = true;
+         virtual = TRUE;
          break;
        default:
          warn_stab (orig, _("unknown virtual character for baseclass"));
-         virtual = false;
+         virtual = FALSE;
          break;
        }
       ++*pp;
@@ -2194,26 +2131,26 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
       /* The remaining value is the bit offset of the portion of the
         object corresponding to this baseclass.  Always zero in the
         absence of multiple inheritance.  */
-      bitpos = parse_number (pp, (boolean *) NULL);
+      bitpos = parse_number (pp, (bfd_boolean *) NULL);
       if (**pp != ',')
        {
          bad_stab (orig);
-         return false;
+         return FALSE;
        }
       ++*pp;
 
       type = parse_stab_type (dhandle, info, (const char *) NULL, pp,
                              (debug_type **) NULL);
       if (type == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
 
       classes[i] = debug_make_baseclass (dhandle, type, bitpos, virtual,
                                         visibility);
       if (classes[i] == DEBUG_BASECLASS_NULL)
-       return false;
+       return FALSE;
 
       if (**pp != ';')
-       return false;
+       return FALSE;
       ++*pp;
     }
 
@@ -2221,12 +2158,12 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
 
   *retp = classes;
 
-  return true;
+  return TRUE;
 }
 
 /* Read struct or class data fields.  They have the form:
 
-       NAME : [VISIBILITY] TYPENUM , BITPOS , BITSIZE ;
+       NAME : [VISIBILITY] TYPENUM , BITPOS , BITSIZE ;
 
    At the end, we see a semicolon instead of a field.
 
@@ -2235,7 +2172,7 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
 
    The optional VISIBILITY is one of:
 
-       '/0'    (VISIBILITY_PRIVATE)
+       '/0'    (VISIBILITY_PRIVATE)
        '/1'    (VISIBILITY_PROTECTED)
        '/2'    (VISIBILITY_PUBLIC)
        '/9'    (VISIBILITY_IGNORE)
@@ -2244,13 +2181,10 @@ parse_stab_baseclasses (dhandle, info, pp, retp)
 
    Returns 1 for success, 0 for failure.  */
 
-static boolean
-parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char **pp;
-     debug_field **retp;
-     boolean *staticsp;
+static bfd_boolean
+parse_stab_struct_fields (void *dhandle, struct stab_handle *info,
+                         const char **pp, debug_field **retp,
+                         bfd_boolean *staticsp)
 {
   const char *orig;
   const char *p;
@@ -2259,7 +2193,7 @@ parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
   unsigned int alloc;
 
   *retp = NULL;
-  *staticsp = false;
+  *staticsp = FALSE;
 
   orig = *pp;
 
@@ -2277,7 +2211,7 @@ parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
        {
          alloc += 10;
          fields = ((debug_field *)
-                   xrealloc ((PTR) fields, alloc * sizeof *fields));
+                   xrealloc (fields, alloc * sizeof *fields));
        }
 
       /* If it starts with CPLUS_MARKER it is a special abbreviation,
@@ -2291,7 +2225,7 @@ parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
        {
          ++*pp;
          if (! parse_stab_cpp_abbrev (dhandle, info, pp, fields + c))
-           return false;
+           return FALSE;
          ++c;
          continue;
        }
@@ -2299,13 +2233,13 @@ parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
       /* Look for the ':' that separates the field name from the field
         values.  Data members are delimited by a single ':', while member
         functions are delimited by a pair of ':'s.  When we hit the member
-        functions (if any), terminate scan loop and return. */
+        functions (if any), terminate scan loop and return.  */
 
       p = strchr (p, ':');
       if (p == NULL)
        {
          bad_stab (orig);
-         return false;
+         return FALSE;
        }
 
       if (p[1] == ':')
@@ -2313,7 +2247,7 @@ parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
 
       if (! parse_stab_one_struct_field (dhandle, info, pp, p, fields + c,
                                         staticsp))
-       return false;
+       return FALSE;
 
       ++c;
     }
@@ -2322,17 +2256,14 @@ parse_stab_struct_fields (dhandle, info, pp, retp, staticsp)
 
   *retp = fields;
 
-  return true;
+  return TRUE;
 }
 
 /* Special GNU C++ name.  */
 
-static boolean
-parse_stab_cpp_abbrev (dhandle, info, pp, retp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char **pp;
-     debug_field *retp;
+static bfd_boolean
+parse_stab_cpp_abbrev (void *dhandle, struct stab_handle *info,
+                      const char **pp, debug_field *retp)
 {
   const char *orig;
   int cpp_abbrev;
@@ -2349,7 +2280,7 @@ parse_stab_cpp_abbrev (dhandle, info, pp, retp)
   if (**pp != 'v')
     {
       bad_stab (*pp);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
@@ -2364,7 +2295,7 @@ parse_stab_cpp_abbrev (dhandle, info, pp, retp)
   context = parse_stab_type (dhandle, info, (const char *) NULL, pp,
                             (debug_type **) NULL);
   if (context == DEBUG_TYPE_NULL)
-    return false;
+    return FALSE;
 
   switch (cpp_abbrev)
     {
@@ -2391,7 +2322,7 @@ parse_stab_cpp_abbrev (dhandle, info, pp, retp)
   if (**pp != ':')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
@@ -2400,36 +2331,32 @@ parse_stab_cpp_abbrev (dhandle, info, pp, retp)
   if (**pp != ',')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
-  bitpos = parse_number (pp, (boolean *) NULL);
+  bitpos = parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
   *retp = debug_make_field (dhandle, name, type, bitpos, 0,
                            DEBUG_VISIBILITY_PRIVATE);
   if (*retp == DEBUG_FIELD_NULL)
-    return false;
+    return FALSE;
 
-  return true;
+  return TRUE;
 }
 
 /* Parse a single field in a struct or union.  */
 
-static boolean
-parse_stab_one_struct_field (dhandle, info, pp, p, retp, staticsp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char **pp;
-     const char *p;
-     debug_field *retp;
-     boolean *staticsp;
+static bfd_boolean
+parse_stab_one_struct_field (void *dhandle, struct stab_handle *info,
+                            const char **pp, const char *p,
+                            debug_field *retp, bfd_boolean *staticsp)
 {
   const char *orig;
   char *name;
@@ -2473,7 +2400,7 @@ parse_stab_one_struct_field (dhandle, info, pp, p, retp, staticsp)
   type = parse_stab_type (dhandle, info, (const char *) NULL, pp,
                          (debug_type **) NULL);
   if (type == DEBUG_TYPE_NULL)
-    return false;
+    return FALSE;
 
   if (**pp == ':')
     {
@@ -2485,7 +2412,7 @@ parse_stab_one_struct_field (dhandle, info, pp, p, retp, staticsp)
       if (p == NULL)
        {
          bad_stab (orig);
-         return false;
+         return FALSE;
        }
 
       varname = savestring (*pp, p - *pp);
@@ -2494,31 +2421,31 @@ parse_stab_one_struct_field (dhandle, info, pp, p, retp, staticsp)
 
       *retp = debug_make_static_member (dhandle, name, type, varname,
                                        visibility);
-      *staticsp = true;
+      *staticsp = TRUE;
 
-      return true;
+      return TRUE;
     }
 
   if (**pp != ',')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
-  bitpos = parse_number (pp, (boolean *) NULL);
+  bitpos = parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ',')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
-  bitsize = parse_number (pp, (boolean *) NULL);
+  bitsize = parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
-      return false;
+      return FALSE;
     }
   ++*pp;
 
@@ -2544,7 +2471,7 @@ parse_stab_one_struct_field (dhandle, info, pp, p, retp, staticsp)
 
   *retp = debug_make_field (dhandle, name, type, bitpos, bitsize, visibility);
 
-  return true;
+  return TRUE;
 }
 
 /* Read member function stabs info for C++ classes.  The form of each member
@@ -2560,14 +2487,10 @@ parse_stab_one_struct_field (dhandle, info, pp, p, retp, staticsp)
    $ is the CPLUS_MARKER (usually '$'), `*' holds the place for an operator
    name (such as `+=') and `.' marks the end of the operator name.  */
 
-static boolean
-parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *tagname;
-     const char **pp;
-     const int *typenums;
-     debug_method **retp;
+static bfd_boolean
+parse_stab_members (void *dhandle, struct stab_handle *info,
+                   const char *tagname, const char **pp,
+                   const int *typenums, debug_method **retp)
 {
   const char *orig;
   debug_method *methods;
@@ -2615,7 +2538,7 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
          if (*p != '.')
            {
              bad_stab (orig);
-             return false;
+             return FALSE;
            }
          name = savestring (*pp, p - *pp);
          *pp = p + 1;
@@ -2631,14 +2554,14 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
       do
        {
          debug_type type;
-         boolean stub;
+         bfd_boolean stub;
          char *argtypes;
          enum debug_visibility visibility;
-         boolean constp, volatilep, staticp;
+         bfd_boolean constp, volatilep, staticp;
          bfd_vma voffset;
          debug_type context;
          const char *physname;
-         boolean varargs;
+         bfd_boolean varargs;
 
          if (look_ahead_type != DEBUG_TYPE_NULL)
            {
@@ -2651,11 +2574,11 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
              type = parse_stab_type (dhandle, info, (const char *) NULL, pp,
                                      (debug_type **) NULL);
              if (type == DEBUG_TYPE_NULL)
-               return false;
+               return FALSE;
              if (**pp != ':')
                {
                  bad_stab (orig);
-                 return false;
+                 return FALSE;
                }
            }
 
@@ -2664,13 +2587,13 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
          if (p == NULL)
            {
              bad_stab (orig);
-             return false;
+             return FALSE;
            }
 
-         stub = false;
+         stub = FALSE;
          if (debug_get_type_kind (dhandle, type) == DEBUG_KIND_METHOD
              && debug_get_parameter_types (dhandle, type, &varargs) == NULL)
-           stub = true;
+           stub = TRUE;
 
          argtypes = savestring (*pp, p - *pp);
          *pp = p + 1;
@@ -2689,8 +2612,8 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
            }
          ++*pp;
 
-         constp = false;
-         volatilep = false;
+         constp = FALSE;
+         volatilep = FALSE;
          switch (**pp)
            {
            case 'A':
@@ -2699,18 +2622,18 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
              break;
            case 'B':
              /* const member function.  */
-             constp = true;
+             constp = TRUE;
              ++*pp;
              break;
            case 'C':
              /* volatile member function.  */
-             volatilep = true;
+             volatilep = TRUE;
              ++*pp;
              break;
            case 'D':
              /* const volatile member function.  */
-             constp = true;
-             volatilep = true;
+             constp = TRUE;
+             volatilep = TRUE;
              ++*pp;
              break;
            case '*':
@@ -2723,7 +2646,7 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
              break;
            }
 
-         staticp = false;
+         staticp = FALSE;
          switch (**pp)
            {
            case '*':
@@ -2731,11 +2654,11 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
                 bit is supposedly set to distinguish
                 pointers-to-methods from virtual function indicies.  */
              ++*pp;
-             voffset = parse_number (pp, (boolean *) NULL);
+             voffset = parse_number (pp, (bfd_boolean *) NULL);
              if (**pp != ';')
                {
                  bad_stab (orig);
-                 return false;
+                 return FALSE;
                }
              ++*pp;
              voffset &= 0x7fffffff;
@@ -2750,37 +2673,37 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
                  /* Figure out from whence this virtual function
                     came.  It may belong to virtual function table of
                     one of its baseclasses.  */
-                   look_ahead_type = parse_stab_type (dhandle, info,
-                                                      (const char *) NULL,
-                                                      pp,
-                                                      (debug_type **) NULL);
-                   if (**pp == ':')
-                     {
-                       /* g++ version 1 overloaded methods.  */
-                       context = DEBUG_TYPE_NULL;
-                     }
-                   else
-                     {
-                       context = look_ahead_type;
-                       look_ahead_type = DEBUG_TYPE_NULL;
-                       if (**pp != ';')
-                         {
-                           bad_stab (orig);
-                           return false;
-                         }
-                       ++*pp;
-                     }
-                 }
+                 look_ahead_type = parse_stab_type (dhandle, info,
+                                                    (const char *) NULL,
+                                                    pp,
+                                                    (debug_type **) NULL);
+                 if (**pp == ':')
+                   {
+                     /* g++ version 1 overloaded methods.  */
+                     context = DEBUG_TYPE_NULL;
+                   }
+                 else
+                   {
+                     context = look_ahead_type;
+                     look_ahead_type = DEBUG_TYPE_NULL;
+                     if (**pp != ';')
+                       {
+                         bad_stab (orig);
+                         return FALSE;
+                       }
+                     ++*pp;
+                   }
+               }
              break;
 
            case '?':
              /* static member function.  */
              ++*pp;
-             staticp = true;
+             staticp = TRUE;
              voffset = 0;
              context = DEBUG_TYPE_NULL;
              if (strncmp (argtypes, name, strlen (name)) != 0)
-               stub = true;
+               stub = TRUE;
              break;
 
            default:
@@ -2809,25 +2732,25 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
 
              class_type = stab_find_type (dhandle, info, typenums);
              if (class_type == DEBUG_TYPE_NULL)
-               return false;
+               return FALSE;
              return_type = debug_get_return_type (dhandle, type);
              if (return_type == DEBUG_TYPE_NULL)
                {
                  bad_stab (orig);
-                 return false;
+                 return FALSE;
                }
              type = parse_stab_argtypes (dhandle, info, class_type, name,
                                          tagname, return_type, argtypes,
                                          constp, volatilep, &physname);
              if (type == DEBUG_TYPE_NULL)
-               return false;
+               return FALSE;
            }
 
          if (cvars + 1 >= allocvars)
            {
              allocvars += 10;
              variants = ((debug_method_variant *)
-                         xrealloc ((PTR) variants,
+                         xrealloc (variants,
                                    allocvars * sizeof *variants));
            }
 
@@ -2844,7 +2767,7 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
                                                                constp,
                                                                volatilep);
          if (variants[cvars] == DEBUG_METHOD_VARIANT_NULL)
-           return false;
+           return FALSE;
 
          ++cvars;
        }
@@ -2859,7 +2782,7 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
        {
          alloc += 10;
          methods = ((debug_method *)
-                    xrealloc ((PTR) methods, alloc * sizeof *methods));
+                    xrealloc (methods, alloc * sizeof *methods));
        }
 
       methods[c] = debug_make_method (dhandle, name, variants);
@@ -2872,7 +2795,7 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
 
   *retp = methods;
 
-  return true;
+  return TRUE;
 }
 
 /* Parse a string representing argument types for a method.  Stabs
@@ -2882,29 +2805,23 @@ parse_stab_members (dhandle, info, tagname, pp, typenums, retp)
    the tag name.  */
 
 static debug_type
-parse_stab_argtypes (dhandle, info, class_type, fieldname, tagname,
-                    return_type, argtypes, constp, volatilep, pphysname)
-     PTR dhandle;
-     struct stab_handle *info;
-     debug_type class_type;
-     const char *fieldname;
-     const char *tagname;
-     debug_type return_type;
-     const char *argtypes;
-     boolean constp;
-     boolean volatilep;
-     const char **pphysname;
+parse_stab_argtypes (void *dhandle, struct stab_handle *info,
+                    debug_type class_type, const char *fieldname,
+                    const char *tagname, debug_type return_type,
+                    const char *argtypes, bfd_boolean constp,
+                    bfd_boolean volatilep, const char **pphysname)
 {
-  boolean is_full_physname_constructor;
-  boolean is_constructor;
-  boolean is_destructor;
+  bfd_boolean is_full_physname_constructor;
+  bfd_boolean is_constructor;
+  bfd_boolean is_destructor;
   debug_type *args;
-  boolean varargs;
+  bfd_boolean varargs;
+  unsigned int physname_len = 0;
 
   /* Constructors are sometimes handled specially.  */
   is_full_physname_constructor = ((argtypes[0] == '_'
                                   && argtypes[1] == '_'
-                                  && (isdigit ((unsigned char) argtypes[2])
+                                  && (ISDIGIT (argtypes[2])
                                       || argtypes[2] == 'Q'
                                       || argtypes[2] == 't'))
                                  || strncmp (argtypes, "__ct", 4) == 0);
@@ -2976,6 +2893,7 @@ parse_stab_argtypes (dhandle, info, class_type, fieldname, tagname,
            strcpy (physname, fieldname);
        }
 
+      physname_len = strlen (physname);
       strcat (physname, buf);
       if (tagname != NULL)
        strcat (physname, tagname);
@@ -2989,10 +2907,10 @@ parse_stab_argtypes (dhandle, info, class_type, fieldname, tagname,
       args = (debug_type *) xmalloc (sizeof *args);
       *args = NULL;
       return debug_make_method_type (dhandle, return_type, class_type, args,
-                                    false);
+                                    FALSE);
     }
 
-  args = stab_demangle_argtypes (dhandle, info, *pphysname, &varargs);
+  args = stab_demangle_argtypes (dhandle, info, *pphysname, &varargs, physname_len);
   if (args == NULL)
     return DEBUG_TYPE_NULL;
 
@@ -3008,42 +2926,38 @@ parse_stab_argtypes (dhandle, info, class_type, fieldname, tagname,
    This function is called when we have parsed all the method declarations,
    so we can look for the vptr base class info.  */
 
-static boolean
-parse_stab_tilde_field (dhandle, info, pp, typenums, retvptrbase, retownvptr)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char **pp;
-     const int *typenums;
-     debug_type *retvptrbase;
-     boolean *retownvptr;
+static bfd_boolean
+parse_stab_tilde_field (void *dhandle, struct stab_handle *info,
+                       const char **pp, const int *typenums,
+                       debug_type *retvptrbase, bfd_boolean *retownvptr)
 {
   const char *orig;
   const char *hold;
   int vtypenums[2];
 
   *retvptrbase = DEBUG_TYPE_NULL;
-  *retownvptr = false;
+  *retownvptr = FALSE;
 
   orig = *pp;
 
-  /* If we are positioned at a ';', then skip it. */
+  /* If we are positioned at a ';', then skip it.  */
   if (**pp == ';')
     ++*pp;
 
   if (**pp != '~')
-    return true;
+    return TRUE;
 
   ++*pp;
 
   if (**pp == '=' || **pp == '+' || **pp == '-')
     {
       /* Obsolete flags that used to indicate the presence of
-        constructors and/or destructors. */
+        constructors and/or destructors.  */
       ++*pp;
     }
 
   if (**pp != '%')
-    return true;
+    return TRUE;
 
   ++*pp;
 
@@ -3052,11 +2966,11 @@ parse_stab_tilde_field (dhandle, info, pp, typenums, retvptrbase, retownvptr)
   /* The next number is the type number of the base class (possibly
      our own class) which supplies the vtable for this class.  */
   if (! parse_stab_type_number (pp, vtypenums))
-    return false;
+    return FALSE;
 
   if (vtypenums[0] == typenums[0]
       && vtypenums[1] == typenums[1])
-    *retownvptr = true;
+    *retownvptr = TRUE;
   else
     {
       debug_type vtype;
@@ -3071,7 +2985,7 @@ parse_stab_tilde_field (dhandle, info, pp, typenums, retvptrbase, retownvptr)
       if (*p != ';')
        {
          bad_stab (orig);
-         return false;
+         return FALSE;
        }
 
       *retvptrbase = vtype;
@@ -3079,23 +2993,20 @@ parse_stab_tilde_field (dhandle, info, pp, typenums, retvptrbase, retownvptr)
       *pp = p + 1;
     }
 
-  return true;    
+  return TRUE;
 }
 
 /* Read a definition of an array type.  */
 
 static debug_type
-parse_stab_array_type (dhandle, info, pp, stringp)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char **pp;
-     boolean stringp;
+parse_stab_array_type (void *dhandle, struct stab_handle *info,
+                      const char **pp, bfd_boolean stringp)
 {
   const char *orig;
   const char *p;
   int typenums[2];
   debug_type index_type;
-  boolean adjustable;
+  bfd_boolean adjustable;
   bfd_signed_vma lower, upper;
   debug_type element_type;
 
@@ -3113,15 +3024,15 @@ parse_stab_array_type (dhandle, info, pp, stringp)
   /* If the index type is type 0, we take it as int.  */
   p = *pp;
   if (! parse_stab_type_number (&p, typenums))
-    return false;
+    return DEBUG_TYPE_NULL;
   if (typenums[0] == 0 && typenums[1] == 0 && **pp != '=')
     {
       index_type = debug_find_named_type (dhandle, "int");
       if (index_type == DEBUG_TYPE_NULL)
        {
-         index_type = debug_make_int_type (dhandle, 4, false);
+         index_type = debug_make_int_type (dhandle, 4, FALSE);
          if (index_type == DEBUG_TYPE_NULL)
-           return false;
+           return DEBUG_TYPE_NULL;
        }
       *pp = p;
     }
@@ -3138,40 +3049,40 @@ parse_stab_array_type (dhandle, info, pp, stringp)
     }
   ++*pp;
 
-  adjustable = false;
+  adjustable = FALSE;
 
-  if (! isdigit ((unsigned char) **pp) && **pp != '-')
+  if (! ISDIGIT (**pp) && **pp != '-')
     {
       ++*pp;
-      adjustable = true;
+      adjustable = TRUE;
     }
 
-  lower = (bfd_signed_vma) parse_number (pp, (boolean *) NULL);
+  lower = (bfd_signed_vma) parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
-      return false;
+      return DEBUG_TYPE_NULL;
     }
   ++*pp;
 
-  if (! isdigit ((unsigned char) **pp) && **pp != '-')
+  if (! ISDIGIT (**pp) && **pp != '-')
     {
       ++*pp;
-      adjustable = true;
+      adjustable = TRUE;
     }
 
-  upper = (bfd_signed_vma) parse_number (pp, (boolean *) NULL);
+  upper = (bfd_signed_vma) parse_number (pp, (bfd_boolean *) NULL);
   if (**pp != ';')
     {
       bad_stab (orig);
-      return false;
+      return DEBUG_TYPE_NULL;
     }
   ++*pp;
 
   element_type = parse_stab_type (dhandle, info, (const char *) NULL, pp,
                                  (debug_type **) NULL);
   if (element_type == DEBUG_TYPE_NULL)
-    return false;
+    return DEBUG_TYPE_NULL;
 
   if (adjustable)
     {
@@ -3205,10 +3116,7 @@ struct bincl_file
 /* Start a new N_BINCL file, pushing it onto the stack.  */
 
 static void
-push_bincl (info, name, hash)
-     struct stab_handle *info;
-     const char *name;
-     bfd_vma hash;
+push_bincl (struct stab_handle *info, const char *name, bfd_vma hash)
 {
   struct bincl_file *n;
 
@@ -3224,7 +3132,7 @@ push_bincl (info, name, hash)
 
   ++info->files;
   info->file_types = ((struct stab_types **)
-                     xrealloc ((PTR) info->file_types,
+                     xrealloc (info->file_types,
                                (info->files
                                 * sizeof *info->file_types)));
   info->file_types[n->file] = NULL;
@@ -3234,8 +3142,7 @@ push_bincl (info, name, hash)
    stack.  */
 
 static const char *
-pop_bincl (info)
-     struct stab_handle *info;
+pop_bincl (struct stab_handle *info)
 {
   struct bincl_file *o;
 
@@ -3253,17 +3160,14 @@ pop_bincl (info)
 
 /* Handle an N_EXCL: get the types from the corresponding N_BINCL.  */
 
-static boolean
-find_excl (info, name, hash)
-     struct stab_handle *info;
-     const char *name;
-     bfd_vma hash;
+static bfd_boolean
+find_excl (struct stab_handle *info, const char *name, bfd_vma hash)
 {
   struct bincl_file *l;
 
   ++info->files;
   info->file_types = ((struct stab_types **)
-                     xrealloc ((PTR) info->file_types,
+                     xrealloc (info->file_types,
                                (info->files
                                 * sizeof *info->file_types)));
 
@@ -3274,12 +3178,12 @@ find_excl (info, name, hash)
     {
       warn_stab (name, _("Undefined N_EXCL"));
       info->file_types[info->files - 1] = NULL;
-      return true;
+      return TRUE;
     }
 
   info->file_types[info->files - 1] = l->file_types;
 
-  return true;
+  return TRUE;
 }
 
 /* Handle a variable definition.  gcc emits variable definitions for a
@@ -3287,14 +3191,10 @@ find_excl (info, name, hash)
    it.  The SunPRO compiler emits variable definitions after the
    N_LBRAC, so we can call debug_record_variable immediately.  */
 
-static boolean
-stab_record_variable (dhandle, info, name, type, kind, val)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *name;
-     debug_type type;
-     enum debug_var_kind kind;
-     bfd_vma val;
+static bfd_boolean
+stab_record_variable (void *dhandle, struct stab_handle *info,
+                     const char *name, debug_type type,
+                     enum debug_var_kind kind, bfd_vma val)
 {
   struct stab_pending_var *v;
 
@@ -3313,16 +3213,14 @@ stab_record_variable (dhandle, info, name, type, kind, val)
   v->val = val;
   info->pending = v;
 
-  return true;
+  return TRUE;
 }
 
 /* Emit pending variable definitions.  This is called after we see the
    N_LBRAC that starts the block.  */
 
-static boolean
-stab_emit_pending_vars (dhandle, info)
-     PTR dhandle;
-     struct stab_handle *info;
+static bfd_boolean
+stab_emit_pending_vars (void *dhandle, struct stab_handle *info)
 {
   struct stab_pending_var *v;
 
@@ -3332,7 +3230,7 @@ stab_emit_pending_vars (dhandle, info)
       struct stab_pending_var *next;
 
       if (! debug_record_variable (dhandle, v->name, v->type, v->kind, v->val))
-       return false;
+       return FALSE;
 
       next = v->next;
       free (v);
@@ -3341,15 +3239,13 @@ stab_emit_pending_vars (dhandle, info)
 
   info->pending = NULL;
 
-  return true;
+  return TRUE;
 }
 
 /* Find the slot for a type in the database.  */
 
 static debug_type *
-stab_find_slot (info, typenums)
-     struct stab_handle *info;
-     const int *typenums;
+stab_find_slot (struct stab_handle *info, const int *typenums)
 {
   int filenum;
   int index;
@@ -3394,10 +3290,7 @@ stab_find_slot (info, typenums)
    allocated yet, create an indirect type.  */
 
 static debug_type
-stab_find_type (dhandle, info, typenums)
-     PTR dhandle;
-     struct stab_handle *info;
-     const int *typenums;
+stab_find_type (void *dhandle, struct stab_handle *info, const int *typenums)
 {
   debug_type *slot;
 
@@ -3419,33 +3312,28 @@ stab_find_type (dhandle, info, typenums)
 
 /* Record that a given type number refers to a given type.  */
 
-static boolean
-stab_record_type (dhandle, info, typenums, type)
-     PTR dhandle;
-     struct stab_handle *info;
-     const int *typenums;
-     debug_type type;
+static bfd_boolean
+stab_record_type (void *dhandle ATTRIBUTE_UNUSED, struct stab_handle *info,
+                 const int *typenums, debug_type type)
 {
   debug_type *slot;
 
   slot = stab_find_slot (info, typenums);
   if (slot == NULL)
-    return false;
+    return FALSE;
 
   /* gdb appears to ignore type redefinitions, so we do as well.  */
 
   *slot = type;
 
-  return true;
+  return TRUE;
 }
 
 /* Return an XCOFF builtin type.  */
 
 static debug_type
-stab_xcoff_builtin_type (dhandle, info, typenum)
-     PTR dhandle;
-     struct stab_handle *info;
-     int typenum;
+stab_xcoff_builtin_type (void *dhandle, struct stab_handle *info,
+                        int typenum)
 {
   debug_type rettype;
   const char *name;
@@ -3464,42 +3352,42 @@ stab_xcoff_builtin_type (dhandle, info, typenum)
       /* The size of this and all the other types are fixed, defined
         by the debugging format.  */
       name = "int";
-      rettype = debug_make_int_type (dhandle, 4, false);
+      rettype = debug_make_int_type (dhandle, 4, FALSE);
       break;
     case 2:
       name = "char";
-      rettype = debug_make_int_type (dhandle, 1, false);
+      rettype = debug_make_int_type (dhandle, 1, FALSE);
       break;
     case 3:
       name = "short";
-      rettype = debug_make_int_type (dhandle, 2, false);
+      rettype = debug_make_int_type (dhandle, 2, FALSE);
       break;
     case 4:
       name = "long";
-      rettype = debug_make_int_type (dhandle, 4, false);
+      rettype = debug_make_int_type (dhandle, 4, FALSE);
       break;
     case 5:
       name = "unsigned char";
-      rettype = debug_make_int_type (dhandle, 1, true);
+      rettype = debug_make_int_type (dhandle, 1, TRUE);
       break;
     case 6:
       name = "signed char";
-      rettype = debug_make_int_type (dhandle, 1, false);
+      rettype = debug_make_int_type (dhandle, 1, FALSE);
       break;
     case 7:
       name = "unsigned short";
-      rettype = debug_make_int_type (dhandle, 2, true);
+      rettype = debug_make_int_type (dhandle, 2, TRUE);
       break;
     case 8:
       name = "unsigned int";
-      rettype = debug_make_int_type (dhandle, 4, true);
+      rettype = debug_make_int_type (dhandle, 4, TRUE);
       break;
     case 9:
       name = "unsigned";
-      rettype = debug_make_int_type (dhandle, 4, true);
+      rettype = debug_make_int_type (dhandle, 4, TRUE);
     case 10:
       name = "unsigned long";
-      rettype = debug_make_int_type (dhandle, 4, true);
+      rettype = debug_make_int_type (dhandle, 4, TRUE);
       break;
     case 11:
       name = "void";
@@ -3524,7 +3412,7 @@ stab_xcoff_builtin_type (dhandle, info, typenum)
       break;
     case 15:
       name = "integer";
-      rettype = debug_make_int_type (dhandle, 4, false);
+      rettype = debug_make_int_type (dhandle, 4, FALSE);
       break;
     case 16:
       name = "boolean";
@@ -3546,7 +3434,7 @@ stab_xcoff_builtin_type (dhandle, info, typenum)
     case 20:
       /* FIXME */
       name = "character";
-      rettype = debug_make_int_type (dhandle, 1, true);
+      rettype = debug_make_int_type (dhandle, 1, TRUE);
       break;
     case 21:
       name = "logical*1";
@@ -3576,28 +3464,28 @@ stab_xcoff_builtin_type (dhandle, info, typenum)
       break;
     case 27:
       name = "integer*1";
-      rettype = debug_make_int_type (dhandle, 1, false);
+      rettype = debug_make_int_type (dhandle, 1, FALSE);
       break;
     case 28:
       name = "integer*2";
-      rettype = debug_make_int_type (dhandle, 2, false);
+      rettype = debug_make_int_type (dhandle, 2, FALSE);
       break;
     case 29:
       name = "integer*4";
-      rettype = debug_make_int_type (dhandle, 4, false);
+      rettype = debug_make_int_type (dhandle, 4, FALSE);
       break;
     case 30:
       /* FIXME */
       name = "wchar";
-      rettype = debug_make_int_type (dhandle, 2, false);
+      rettype = debug_make_int_type (dhandle, 2, FALSE);
       break;
     case 31:
       name = "long long";
-      rettype = debug_make_int_type (dhandle, 8, false);
+      rettype = debug_make_int_type (dhandle, 8, FALSE);
       break;
     case 32:
       name = "unsigned long long";
-      rettype = debug_make_int_type (dhandle, 8, true);
+      rettype = debug_make_int_type (dhandle, 8, TRUE);
       break;
     case 33:
       name = "logical*8";
@@ -3605,7 +3493,7 @@ stab_xcoff_builtin_type (dhandle, info, typenum)
       break;
     case 34:
       name = "integer*8";
-      rettype = debug_make_int_type (dhandle, 8, false);
+      rettype = debug_make_int_type (dhandle, 8, FALSE);
       break;
     default:
       abort ();
@@ -3621,12 +3509,8 @@ stab_xcoff_builtin_type (dhandle, info, typenum)
 /* Find or create a tagged type.  */
 
 static debug_type
-stab_find_tagged_type (dhandle, info, p, len, kind)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *p;
-     int len;
-     enum debug_type_kind kind;
+stab_find_tagged_type (void *dhandle, struct stab_handle *info,
+                      const char *p, int len, enum debug_type_kind kind)
 {
   char *name;
   debug_type dtype;
@@ -3701,13 +3585,13 @@ struct stab_demangle_typestring
 struct stab_demangle_info
 {
   /* The debugging information handle.  */
-  PTR dhandle;
+  void *dhandle;
   /* The stab information handle.  */
   struct stab_handle *info;
   /* The array of arguments we are building.  */
   debug_type *args;
   /* Whether the method takes a variable number of arguments.  */
-  boolean varargs;
+  bfd_boolean varargs;
   /* The array of types we have remembered.  */
   struct stab_demangle_typestring *typestrings;
   /* The number of typestrings.  */
@@ -3716,40 +3600,37 @@ struct stab_demangle_info
   unsigned int typestring_alloc;
 };
 
-static void stab_bad_demangle PARAMS ((const char *));
-static unsigned int stab_demangle_count PARAMS ((const char **));
-static boolean stab_demangle_get_count
-  PARAMS ((const char **, unsigned int *));
-static boolean stab_demangle_prefix
-  PARAMS ((struct stab_demangle_info *, const char **));
-static boolean stab_demangle_function_name
-  PARAMS ((struct stab_demangle_info *, const char **, const char *));
-static boolean stab_demangle_signature
-  PARAMS ((struct stab_demangle_info *, const char **));
-static boolean stab_demangle_qualified
-  PARAMS ((struct stab_demangle_info *, const char **, debug_type *));
-static boolean stab_demangle_template
-  PARAMS ((struct stab_demangle_info *, const char **));
-static boolean stab_demangle_class
-  PARAMS ((struct stab_demangle_info *, const char **, const char **));
-static boolean stab_demangle_args
-  PARAMS ((struct stab_demangle_info *, const char **, debug_type **,
-          boolean *));
-static boolean stab_demangle_arg
-  PARAMS ((struct stab_demangle_info *, const char **, debug_type **,
-          unsigned int *, unsigned int *));
-static boolean stab_demangle_type
-  PARAMS ((struct stab_demangle_info *, const char **, debug_type *));
-static boolean stab_demangle_fund_type
-  PARAMS ((struct stab_demangle_info *, const char **, debug_type *));
-static boolean stab_demangle_remember_type
-  PARAMS ((struct stab_demangle_info *, const char *, int));
+static void stab_bad_demangle (const char *);
+static unsigned int stab_demangle_count (const char **);
+static bfd_boolean stab_demangle_get_count (const char **, unsigned int *);
+static bfd_boolean stab_demangle_prefix
+  (struct stab_demangle_info *, const char **, unsigned int);
+static bfd_boolean stab_demangle_function_name
+  (struct stab_demangle_info *, const char **, const char *);
+static bfd_boolean stab_demangle_signature
+  (struct stab_demangle_info *, const char **);
+static bfd_boolean stab_demangle_qualified
+  (struct stab_demangle_info *, const char **, debug_type *);
+static bfd_boolean stab_demangle_template
+  (struct stab_demangle_info *, const char **, char **);
+static bfd_boolean stab_demangle_class
+  (struct stab_demangle_info *, const char **, const char **);
+static bfd_boolean stab_demangle_args
+  (struct stab_demangle_info *, const char **, debug_type **, bfd_boolean *);
+static bfd_boolean stab_demangle_arg
+  (struct stab_demangle_info *, const char **, debug_type **,
+   unsigned int *, unsigned int *);
+static bfd_boolean stab_demangle_type
+  (struct stab_demangle_info *, const char **, debug_type *);
+static bfd_boolean stab_demangle_fund_type
+  (struct stab_demangle_info *, const char **, debug_type *);
+static bfd_boolean stab_demangle_remember_type
+  (struct stab_demangle_info *, const char *, int);
 
 /* Warn about a bad demangling.  */
 
 static void
-stab_bad_demangle (s)
-     const char *s;
+stab_bad_demangle (const char *s)
 {
   fprintf (stderr, _("bad mangled name `%s'\n"), s);
 }
@@ -3757,13 +3638,12 @@ stab_bad_demangle (s)
 /* Get a count from a stab string.  */
 
 static unsigned int
-stab_demangle_count (pp)
-     const char **pp;
+stab_demangle_count (const char **pp)
 {
   unsigned int count;
 
   count = 0;
-  while (isdigit ((unsigned char) **pp))
+  while (ISDIGIT (**pp))
     {
       count *= 10;
       count += **pp - '0';
@@ -3775,17 +3655,15 @@ stab_demangle_count (pp)
 /* Require a count in a string.  The count may be multiple digits, in
    which case it must end in an underscore.  */
 
-static boolean
-stab_demangle_get_count (pp, pi)
-     const char **pp;
-     unsigned int *pi;
+static bfd_boolean
+stab_demangle_get_count (const char **pp, unsigned int *pi)
 {
-  if (! isdigit ((unsigned char) **pp))
-    return false;
+  if (! ISDIGIT (**pp))
+    return FALSE;
 
   *pi = **pp - '0';
   ++*pp;
-  if (isdigit ((unsigned char) **pp))
+  if (ISDIGIT (**pp))
     {
       unsigned int count;
       const char *p;
@@ -3798,7 +3676,7 @@ stab_demangle_get_count (pp, pi)
          count += *p - '0';
          ++p;
        }
-      while (isdigit ((unsigned char) *p));
+      while (ISDIGIT (*p));
       if (*p == '_')
        {
          *pp = p + 1;
@@ -3806,25 +3684,23 @@ stab_demangle_get_count (pp, pi)
        }
     }
 
-  return true;
+  return TRUE;
 }
 
 /* This function demangles a physical name, returning a NULL
    terminated array of argument types.  */
 
 static debug_type *
-stab_demangle_argtypes (dhandle, info, physname, pvarargs)
-     PTR dhandle;
-     struct stab_handle *info;
-     const char *physname;
-     boolean *pvarargs;
+stab_demangle_argtypes (void *dhandle, struct stab_handle *info,
+                       const char *physname, bfd_boolean *pvarargs,
+                       unsigned int physname_len)
 {
   struct stab_demangle_info minfo;
 
   minfo.dhandle = dhandle;
   minfo.info = info;
   minfo.args = NULL;
-  minfo.varargs = false;
+  minfo.varargs = FALSE;
   minfo.typestring_alloc = 10;
   minfo.typestrings = ((struct stab_demangle_typestring *)
                       xmalloc (minfo.typestring_alloc
@@ -3834,7 +3710,7 @@ stab_demangle_argtypes (dhandle, info, physname, pvarargs)
   /* cplus_demangle checks for special GNU mangled forms, but we can't
      see any of them in mangled method argument types.  */
 
-  if (! stab_demangle_prefix (&minfo, &physname))
+  if (! stab_demangle_prefix (&minfo, &physname, physname_len))
     goto error_return;
 
   if (*physname != '\0')
@@ -3860,10 +3736,9 @@ stab_demangle_argtypes (dhandle, info, physname, pvarargs)
 
 /* Demangle the prefix of the mangled name.  */
 
-static boolean
-stab_demangle_prefix (minfo, pp)
-     struct stab_demangle_info *minfo;
-     const char **pp;
+static bfd_boolean
+stab_demangle_prefix (struct stab_demangle_info *minfo, const char **pp,
+                     unsigned int physname_len)
 {
   const char *scan;
   unsigned int i;
@@ -3871,38 +3746,41 @@ stab_demangle_prefix (minfo, pp)
   /* cplus_demangle checks for global constructors and destructors,
      but we can't see them in mangled argument types.  */
 
-  /* Look for `__'.  */
-  scan = *pp;
-  do
+  if (physname_len)
+    scan = *pp + physname_len;
+  else
     {
-      scan = strchr (scan, '_');
-    }
-  while (scan != NULL && *++scan != '_');
+      /* Look for `__'.  */
+      scan = *pp;
+      do
+       scan = strchr (scan, '_');
+      while (scan != NULL && *++scan != '_');
 
-  if (scan == NULL)
-    {
-      stab_bad_demangle (*pp);
-      return false;
-    }
+      if (scan == NULL)
+       {
+         stab_bad_demangle (*pp);
+         return FALSE;
+       }
 
-  --scan;
+      --scan;
 
-  /* We found `__'; move ahead to the last contiguous `__' pair.  */
-  i = strspn (scan, "_");
-  if (i > 2)
-    scan += i - 2;
+      /* We found `__'; move ahead to the last contiguous `__' pair.  */
+      i = strspn (scan, "_");
+      if (i > 2)
+       scan += i - 2;
+    }
 
   if (scan == *pp
-      && (isdigit ((unsigned char) scan[2])
+      && (ISDIGIT (scan[2])
          || scan[2] == 'Q'
          || scan[2] == 't'))
     {
       /* This is a GNU style constructor name.  */
       *pp = scan + 2;
-      return true;
+      return TRUE;
     }
   else if (scan == *pp
-          && ! isdigit ((unsigned char) scan[2])
+          && ! ISDIGIT (scan[2])
           && scan[2] != 't')
     {
       /* Look for the `__' that separates the prefix from the
@@ -3913,7 +3791,7 @@ stab_demangle_prefix (minfo, pp)
       if (scan == NULL || scan[2] == '\0')
        {
          stab_bad_demangle (*pp);
-         return false;
+         return FALSE;
        }
 
       return stab_demangle_function_name (minfo, pp, scan);
@@ -3926,7 +3804,7 @@ stab_demangle_prefix (minfo, pp)
   else
     {
       stab_bad_demangle (*pp);
-      return false;
+      return FALSE;
     }
   /*NOTREACHED*/
 }
@@ -3935,11 +3813,9 @@ stab_demangle_prefix (minfo, pp)
    double underscore which separates the function name from the
    signature.  */
 
-static boolean
-stab_demangle_function_name (minfo, pp, scan)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     const char *scan;
+static bfd_boolean
+stab_demangle_function_name (struct stab_demangle_info *minfo,
+                            const char **pp, const char *scan)
 {
   const char *name;
 
@@ -3960,7 +3836,7 @@ stab_demangle_function_name (minfo, pp, scan)
       /* This is a type conversion operator.  */
       tem = name + 5;
       if (! stab_demangle_type (minfo, &tem, (debug_type *) NULL))
-       return false;
+       return FALSE;
     }
   else if (name[0] == '_'
           && name[1] == '_'
@@ -3972,28 +3848,26 @@ stab_demangle_function_name (minfo, pp, scan)
       /* This is a type conversion operator.  */
       tem = name + 4;
       if (! stab_demangle_type (minfo, &tem, (debug_type *) NULL))
-       return false;
+       return FALSE;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Demangle the signature.  This is where the argument types are
    found.  */
 
-static boolean
-stab_demangle_signature (minfo, pp)
-     struct stab_demangle_info *minfo;
-     const char **pp;
+static bfd_boolean
+stab_demangle_signature (struct stab_demangle_info *minfo, const char **pp)
 {
   const char *orig;
-  boolean expect_func, func_done;
+  bfd_boolean expect_func, func_done;
   const char *hold;
 
   orig = *pp;
 
-  expect_func = false;
-  func_done = false;
+  expect_func = FALSE;
+  func_done = FALSE;
   hold = NULL;
 
   while (**pp != '\0')
@@ -4004,8 +3878,8 @@ stab_demangle_signature (minfo, pp)
          hold = *pp;
          if (! stab_demangle_qualified (minfo, pp, (debug_type *) NULL)
              || ! stab_demangle_remember_type (minfo, hold, *pp - hold))
-           return false;
-         expect_func = true;
+           return FALSE;
+         expect_func = TRUE;
          hold = NULL;
          break;
 
@@ -4029,8 +3903,8 @@ stab_demangle_signature (minfo, pp)
            hold = *pp;
          if (! stab_demangle_class (minfo, pp, (const char **) NULL)
              || ! stab_demangle_remember_type (minfo, hold, *pp - hold))
-           return false;
-         expect_func = true;
+           return FALSE;
+         expect_func = TRUE;
          hold = NULL;
          break;
 
@@ -4038,21 +3912,21 @@ stab_demangle_signature (minfo, pp)
          /* Function.  I don't know if this actually happens with g++
              output.  */
          hold = NULL;
-         func_done = true;
+         func_done = TRUE;
          ++*pp;
          if (! stab_demangle_args (minfo, pp, &minfo->args, &minfo->varargs))
-           return false;
+           return FALSE;
          break;
 
        case 't':
          /* Template.  */
          if (hold == NULL)
            hold = *pp;
-         if (! stab_demangle_template (minfo, pp)
+         if (! stab_demangle_template (minfo, pp, (char **) NULL)
              || ! stab_demangle_remember_type (minfo, hold, *pp - hold))
-           return false;
+           return FALSE;
          hold = NULL;
-         expect_func = true;
+         expect_func = TRUE;
          break;
 
        case '_':
@@ -4062,22 +3936,22 @@ stab_demangle_signature (minfo, pp)
             has been mangled by some algorithm we don't know how to
             deal with.  So just reject the entire demangling.  */
          stab_bad_demangle (orig);
-         return false;
+         return FALSE;
 
        default:
          /* Assume we have stumbled onto the first outermost function
             argument token, and start processing args.  */
-         func_done = true;
+         func_done = TRUE;
          if (! stab_demangle_args (minfo, pp, &minfo->args, &minfo->varargs))
-           return false;
+           return FALSE;
          break;
        }
 
       if (expect_func)
        {
-         func_done = true;
+         func_done = TRUE;
          if (! stab_demangle_args (minfo, pp, &minfo->args, &minfo->varargs))
-           return false;
+           return FALSE;
        }
     }
 
@@ -4088,20 +3962,18 @@ stab_demangle_signature (minfo, pp)
         first case, and need to ensure that the '(void)' gets added
         to the current declp.  */
       if (! stab_demangle_args (minfo, pp, &minfo->args, &minfo->varargs))
-       return false;
+       return FALSE;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Demangle a qualified name, such as "Q25Outer5Inner" which is the
    mangled form of "Outer::Inner".  */
 
-static boolean
-stab_demangle_qualified (minfo, pp, ptype)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     debug_type *ptype;
+static bfd_boolean
+stab_demangle_qualified (struct stab_demangle_info *minfo, const char **pp,
+                        debug_type *ptype)
 {
   const char *orig;
   const char *p;
@@ -4117,18 +3989,18 @@ stab_demangle_qualified (minfo, pp, ptype)
         preceded by an underscore (to distinguish it from the <= 9
         case) and followed by an underscore.  */
       p = *pp + 2;
-      if (! isdigit ((unsigned char) *p) || *p == '0')
+      if (! ISDIGIT (*p) || *p == '0')
        {
          stab_bad_demangle (orig);
-         return false;
+         return FALSE;
        }
       qualifiers = atoi (p);
-      while (isdigit ((unsigned char) *p))
+      while (ISDIGIT (*p))
        ++p;
       if (*p != '_')
        {
          stab_bad_demangle (orig);
-         return false;
+         return FALSE;
        }
       *pp = p + 1;
       break;
@@ -4145,7 +4017,7 @@ stab_demangle_qualified (minfo, pp, ptype)
     case '0':
     default:
       stab_bad_demangle (orig);
-      return false;
+      return FALSE;
     }
 
   context = DEBUG_TYPE_NULL;
@@ -4157,10 +4029,21 @@ stab_demangle_qualified (minfo, pp, ptype)
        ++*pp;
       if (**pp == 't')
        {
-         /* FIXME: I don't know how to handle the ptype != NULL case
-             here.  */
-         if (! stab_demangle_template (minfo, pp))
-           return false;
+         char *name;
+
+         if (! stab_demangle_template (minfo, pp,
+                                       ptype != NULL ? &name : NULL))
+           return FALSE;
+
+         if (ptype != NULL)
+           {
+             context = stab_find_tagged_type (minfo->dhandle, minfo->info,
+                                              name, strlen (name),
+                                              DEBUG_KIND_CLASS);
+             free (name);
+             if (context == DEBUG_TYPE_NULL)
+               return FALSE;
+           }
        }
       else
        {
@@ -4170,7 +4053,7 @@ stab_demangle_qualified (minfo, pp, ptype)
          if (strlen (*pp) < len)
            {
              stab_bad_demangle (orig);
-             return false;
+             return FALSE;
            }
 
          if (ptype != NULL)
@@ -4204,7 +4087,7 @@ stab_demangle_qualified (minfo, pp, ptype)
 
                      ft = debug_get_field_type (minfo->dhandle, *fields);
                      if (ft == NULL)
-                       return false;
+                       return FALSE;
                      dn = debug_get_type_name (minfo->dhandle, ft);
                      if (dn != NULL && strcmp (dn, name) == 0)
                        {
@@ -4218,7 +4101,7 @@ stab_demangle_qualified (minfo, pp, ptype)
 
              if (context == DEBUG_TYPE_NULL)
                {
-         /* We have to fall back on finding the type by name.
+                 /* We have to fall back on finding the type by name.
                      If there are more types to come, then this must
                      be a class.  Otherwise, it could be anything.  */
 
@@ -4241,7 +4124,7 @@ stab_demangle_qualified (minfo, pp, ptype)
                                                        ? DEBUG_KIND_ILLEGAL
                                                        : DEBUG_KIND_CLASS));
                      if (context == DEBUG_TYPE_NULL)
-                       return false;
+                       return FALSE;
                    }
                }
            }
@@ -4253,15 +4136,15 @@ stab_demangle_qualified (minfo, pp, ptype)
   if (ptype != NULL)
     *ptype = context;
 
-  return true;
+  return TRUE;
 }
 
-/* Demangle a template.  */
+/* Demangle a template.  If PNAME is not NULL, this sets *PNAME to a
+   string representation of the template.  */
 
-static boolean
-stab_demangle_template (minfo, pp)
-     struct stab_demangle_info *minfo;
-     const char **pp;
+static bfd_boolean
+stab_demangle_template (struct stab_demangle_info *minfo, const char **pp,
+                       char **pname)
 {
   const char *orig;
   unsigned int r, i;
@@ -4275,7 +4158,7 @@ stab_demangle_template (minfo, pp)
   if (r == 0 || strlen (*pp) < r)
     {
       stab_bad_demangle (orig);
-      return false;
+      return FALSE;
     }
   *pp += r;
 
@@ -4283,7 +4166,7 @@ stab_demangle_template (minfo, pp)
   if (stab_demangle_get_count (pp, &r) == 0)
     {
       stab_bad_demangle (orig);
-      return false;
+      return FALSE;
     }
 
   for (i = 0; i < r; i++)
@@ -4293,26 +4176,26 @@ stab_demangle_template (minfo, pp)
          /* This is a type parameter.  */
          ++*pp;
          if (! stab_demangle_type (minfo, pp, (debug_type *) NULL))
-           return false;
+           return FALSE;
        }
       else
        {
          const char *old_p;
-         boolean pointerp, realp, integralp, charp, boolp;
-         boolean done;
+         bfd_boolean pointerp, realp, integralp, charp, boolp;
+         bfd_boolean done;
 
          old_p = *pp;
-         pointerp = false;
-         realp = false;
-         integralp = false;
-         charp = false;
-         boolp = false;
-         done = false;
+         pointerp = FALSE;
+         realp = FALSE;
+         integralp = FALSE;
+         charp = FALSE;
+         boolp = FALSE;
+         done = FALSE;
 
          /* This is a value parameter.  */
 
          if (! stab_demangle_type (minfo, pp, (debug_type *) NULL))
-           return false;
+           return FALSE;
 
          while (*old_p != '\0' && ! done)
            {
@@ -4321,8 +4204,8 @@ stab_demangle_template (minfo, pp)
                case 'P':
                case 'p':
                case 'R':
-                 pointerp = true;
-                 done = true;
+                 pointerp = TRUE;
+                 done = TRUE;
                  break;
                case 'C':       /* Const.  */
                case 'S':       /* Signed.  */
@@ -4334,8 +4217,8 @@ stab_demangle_template (minfo, pp)
                  ++old_p;
                  break;
                case 'Q':       /* Qualified name.  */
-                 integralp = true;
-                 done = true;
+                 integralp = TRUE;
+                 done = TRUE;
                  break;
                case 'T':       /* Remembered type.  */
                  abort ();
@@ -4346,27 +4229,27 @@ stab_demangle_template (minfo, pp)
                case 'i':       /* Int.  */
                case 's':       /* Short.  */
                case 'w':       /* Wchar_t.  */
-                 integralp = true;
-                 done = true;
+                 integralp = TRUE;
+                 done = TRUE;
                  break;
                case 'b':       /* Bool.  */
-                 boolp = true;
-                 done = true;
+                 boolp = TRUE;
+                 done = TRUE;
                  break;
                case 'c':       /* Char.  */
-                 charp = true;
-                 done = true;
+                 charp = TRUE;
+                 done = TRUE;
                  break;
                case 'r':       /* Long double.  */
                case 'd':       /* Double.  */
                case 'f':       /* Float.  */
-                 realp = true;
-                 done = true;
+                 realp = TRUE;
+                 done = TRUE;
                  break;
                default:
                  /* Assume it's a user defined integral type.  */
-                 integralp = true;
-                 done = true;
+                 integralp = TRUE;
+                 done = TRUE;
                  break;
                }
            }
@@ -4375,7 +4258,7 @@ stab_demangle_template (minfo, pp)
            {
              if (**pp == 'm')
                ++*pp;
-             while (isdigit ((unsigned char) **pp))
+             while (ISDIGIT (**pp))
                ++*pp;
            }
          else if (charp)
@@ -4388,7 +4271,7 @@ stab_demangle_template (minfo, pp)
              if (val == 0)
                {
                  stab_bad_demangle (orig);
-                 return false;
+                 return FALSE;
                }
            }
          else if (boolp)
@@ -4399,25 +4282,25 @@ stab_demangle_template (minfo, pp)
              if (val != 0 && val != 1)
                {
                  stab_bad_demangle (orig);
-                 return false;
+                 return FALSE;
                }
            }
          else if (realp)
            {
              if (**pp == 'm')
                ++*pp;
-             while (isdigit ((unsigned char) **pp))
+             while (ISDIGIT (**pp))
                ++*pp;
              if (**pp == '.')
                {
                  ++*pp;
-                 while (isdigit ((unsigned char) **pp))
+                 while (ISDIGIT (**pp))
                    ++*pp;
                }
              if (**pp == 'e')
                {
                  ++*pp;
-                 while (isdigit ((unsigned char) **pp))
+                 while (ISDIGIT (**pp))
                    ++*pp;
                }
            }
@@ -4428,23 +4311,61 @@ stab_demangle_template (minfo, pp)
              if (! stab_demangle_get_count (pp, &len))
                {
                  stab_bad_demangle (orig);
-                 return false;
+                 return FALSE;
                }
              *pp += len;
            }
        }
     }
 
-  return true;
+  /* We can translate this to a string fairly easily by invoking the
+     regular demangling routine.  */
+  if (pname != NULL)
+    {
+      char *s1, *s2, *s3, *s4 = NULL;
+      char *from, *to;
+
+      s1 = savestring (orig, *pp - orig);
+
+      s2 = concat ("NoSuchStrinG__", s1, (const char *) NULL);
+
+      free (s1);
+
+      s3 = cplus_demangle (s2, DMGL_ANSI);
+
+      free (s2);
+
+      if (s3 != NULL)
+       s4 = strstr (s3, "::NoSuchStrinG");
+      if (s3 == NULL || s4 == NULL)
+       {
+         stab_bad_demangle (orig);
+         if (s3 != NULL)
+           free (s3);
+         return FALSE;
+       }
+
+      /* Eliminating all spaces, except those between > characters,
+         makes it more likely that the demangled name will match the
+         name which g++ used as the structure name.  */
+      for (from = to = s3; from != s4; ++from)
+       if (*from != ' '
+           || (from[1] == '>' && from > s3 && from[-1] == '>'))
+         *to++ = *from;
+
+      *pname = savestring (s3, to - s3);
+
+      free (s3);
+    }
+
+  return TRUE;
 }
 
 /* Demangle a class name.  */
 
-static boolean
-stab_demangle_class (minfo, pp, pstart)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     const char **pstart;
+static bfd_boolean
+stab_demangle_class (struct stab_demangle_info *minfo ATTRIBUTE_UNUSED,
+                    const char **pp, const char **pstart)
 {
   const char *orig;
   unsigned int n;
@@ -4455,7 +4376,7 @@ stab_demangle_class (minfo, pp, pstart)
   if (strlen (*pp) < n)
     {
       stab_bad_demangle (orig);
-      return false;
+      return FALSE;
     }
 
   if (pstart != NULL)
@@ -4463,18 +4384,15 @@ stab_demangle_class (minfo, pp, pstart)
 
   *pp += n;
 
-  return true;
+  return TRUE;
 }
 
 /* Demangle function arguments.  If the pargs argument is not NULL, it
    is set to a NULL terminated array holding the arguments.  */
 
-static boolean
-stab_demangle_args (minfo, pp, pargs, pvarargs)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     debug_type **pargs;
-     boolean *pvarargs;
+static bfd_boolean
+stab_demangle_args (struct stab_demangle_info *minfo, const char **pp,
+                   debug_type **pargs, bfd_boolean *pvarargs)
 {
   const char *orig;
   unsigned int alloc, count;
@@ -4485,7 +4403,7 @@ stab_demangle_args (minfo, pp, pargs, pvarargs)
   if (pargs != NULL)
     {
       *pargs = (debug_type *) xmalloc (alloc * sizeof **pargs);
-      *pvarargs = false;
+      *pvarargs = FALSE;
     }
   count = 0;
 
@@ -4506,20 +4424,20 @@ stab_demangle_args (minfo, pp, pargs, pvarargs)
              if (! stab_demangle_get_count (pp, &r))
                {
                  stab_bad_demangle (orig);
-                 return false;
+                 return FALSE;
                }
            }
 
          if (! stab_demangle_get_count (pp, &t))
            {
              stab_bad_demangle (orig);
-             return false;
+             return FALSE;
            }
 
          if (t >= minfo->typestring_count)
            {
              stab_bad_demangle (orig);
-             return false;
+             return FALSE;
            }
          while (r-- > 0)
            {
@@ -4527,13 +4445,13 @@ stab_demangle_args (minfo, pp, pargs, pvarargs)
 
              tem = minfo->typestrings[t].typestring;
              if (! stab_demangle_arg (minfo, &tem, pargs, &count, &alloc))
-               return false;
+               return FALSE;
            }
        }
       else
        {
          if (! stab_demangle_arg (minfo, pp, pargs, &count, &alloc))
-           return false;
+           return FALSE;
        }
     }
 
@@ -4543,22 +4461,19 @@ stab_demangle_args (minfo, pp, pargs, pvarargs)
   if (**pp == 'e')
     {
       if (pargs != NULL)
-       *pvarargs = true;
+       *pvarargs = TRUE;
       ++*pp;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Demangle a single argument.  */
 
-static boolean
-stab_demangle_arg (minfo, pp, pargs, pcount, palloc)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     debug_type **pargs;
-     unsigned int *pcount;
-     unsigned int *palloc;
+static bfd_boolean
+stab_demangle_arg (struct stab_demangle_info *minfo, const char **pp,
+                  debug_type **pargs, unsigned int *pcount,
+                  unsigned int *palloc)
 {
   const char *start;
   debug_type type;
@@ -4567,12 +4482,12 @@ stab_demangle_arg (minfo, pp, pargs, pcount, palloc)
   if (! stab_demangle_type (minfo, pp,
                            pargs == NULL ? (debug_type *) NULL : &type)
       || ! stab_demangle_remember_type (minfo, start, *pp - start))
-    return false;
+    return FALSE;
 
   if (pargs != NULL)
     {
       if (type == DEBUG_TYPE_NULL)
-       return false;
+       return FALSE;
 
       if (*pcount + 1 >= *palloc)
        {
@@ -4584,17 +4499,15 @@ stab_demangle_arg (minfo, pp, pargs, pcount, palloc)
       ++*pcount;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Demangle a type.  If the ptype argument is not NULL, *ptype is set
    to the newly allocated type.  */
 
-static boolean
-stab_demangle_type (minfo, pp, ptype)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     debug_type *ptype;
+static bfd_boolean
+stab_demangle_type (struct stab_demangle_info *minfo, const char **pp,
+                   debug_type *ptype)
 {
   const char *orig;
 
@@ -4607,7 +4520,7 @@ stab_demangle_type (minfo, pp, ptype)
       /* A pointer type.  */
       ++*pp;
       if (! stab_demangle_type (minfo, pp, ptype))
-       return false;
+       return FALSE;
       if (ptype != NULL)
        *ptype = debug_make_pointer_type (minfo->dhandle, *ptype);
       break;
@@ -4616,7 +4529,7 @@ stab_demangle_type (minfo, pp, ptype)
       /* A reference type.  */
       ++*pp;
       if (! stab_demangle_type (minfo, pp, ptype))
-       return false;
+       return FALSE;
       if (ptype != NULL)
        *ptype = debug_make_reference_type (minfo->dhandle, *ptype);
       break;
@@ -4630,10 +4543,10 @@ stab_demangle_type (minfo, pp, ptype)
        high = 0;
        while (**pp != '\0' && **pp != '_')
          {
-           if (! isdigit ((unsigned char) **pp))
+           if (! ISDIGIT (**pp))
              {
                stab_bad_demangle (orig);
-               return false;
+               return FALSE;
              }
            high *= 10;
            high += **pp - '0';
@@ -4642,21 +4555,21 @@ stab_demangle_type (minfo, pp, ptype)
        if (**pp != '_')
          {
            stab_bad_demangle (orig);
-           return false;
+           return FALSE;
          }
        ++*pp;
 
        if (! stab_demangle_type (minfo, pp, ptype))
-         return false;
+         return FALSE;
        if (ptype != NULL)
          {
            debug_type int_type;
 
            int_type = debug_find_named_type (minfo->dhandle, "int");
            if (int_type == NULL)
-             int_type = debug_make_int_type (minfo->dhandle, 4, false);
+             int_type = debug_make_int_type (minfo->dhandle, 4, FALSE);
            *ptype = debug_make_array_type (minfo->dhandle, *ptype, int_type,
-                                           0, high, false);
+                                           0, high, FALSE);
          }
       }
       break;
@@ -4671,16 +4584,16 @@ stab_demangle_type (minfo, pp, ptype)
        if (! stab_demangle_get_count (pp, &i))
          {
            stab_bad_demangle (orig);
-           return false;
+           return FALSE;
          }
        if (i >= minfo->typestring_count)
          {
            stab_bad_demangle (orig);
-           return false;
+           return FALSE;
          }
        p = minfo->typestrings[i].typestring;
        if (! stab_demangle_type (minfo, &p, ptype))
-         return false;
+         return FALSE;
       }
       break;
 
@@ -4688,7 +4601,7 @@ stab_demangle_type (minfo, pp, ptype)
       /* A function.  */
       {
        debug_type *args;
-       boolean varargs;
+       bfd_boolean varargs;
 
        ++*pp;
        if (! stab_demangle_args (minfo, pp,
@@ -4696,20 +4609,20 @@ stab_demangle_type (minfo, pp, ptype)
                                   ? (debug_type **) NULL
                                   : &args),
                                  (ptype == NULL
-                                  ? (boolean *) NULL
+                                  ? (bfd_boolean *) NULL
                                   : &varargs)))
-         return false;
+         return FALSE;
        if (**pp != '_')
          {
            /* cplus_demangle will accept a function without a return
               type, but I don't know when that will happen, or what
               to do if it does.  */
            stab_bad_demangle (orig);
-           return false;
+           return FALSE;
          }
        ++*pp;
        if (! stab_demangle_type (minfo, pp, ptype))
-         return false;
+         return FALSE;
        if (ptype != NULL)
          *ptype = debug_make_function_type (minfo->dhandle, *ptype, args,
                                             varargs);
@@ -4720,49 +4633,71 @@ stab_demangle_type (minfo, pp, ptype)
     case 'M':
     case 'O':
       {
-       boolean memberp, constp, volatilep;
+       bfd_boolean memberp, constp, volatilep;
+       debug_type class_type = DEBUG_TYPE_NULL;
        debug_type *args;
-       boolean varargs;
+       bfd_boolean varargs;
        unsigned int n;
        const char *name;
 
        memberp = **pp == 'M';
-       constp = false;
-       volatilep = false;
+       constp = FALSE;
+       volatilep = FALSE;
        args = NULL;
-       varargs = false;
+       varargs = FALSE;
 
        ++*pp;
-       if (! isdigit ((unsigned char) **pp))
+       if (ISDIGIT (**pp))
          {
-           stab_bad_demangle (orig);
-           return false;
+           n = stab_demangle_count (pp);
+           if (strlen (*pp) < n)
+             {
+               stab_bad_demangle (orig);
+               return FALSE;
+             }
+           name = *pp;
+           *pp += n;
+
+           if (ptype != NULL)
+             {
+               class_type = stab_find_tagged_type (minfo->dhandle,
+                                                   minfo->info,
+                                                   name, (int) n,
+                                                   DEBUG_KIND_CLASS);
+               if (class_type == DEBUG_TYPE_NULL)
+                 return FALSE;
+             }
+         }
+       else if (**pp == 'Q')
+         {
+           if (! stab_demangle_qualified (minfo, pp,
+                                          (ptype == NULL
+                                           ? (debug_type *) NULL
+                                           : &class_type)))
+             return FALSE;
          }
-       n = stab_demangle_count (pp);
-       if (strlen (*pp) < n)
+       else
          {
            stab_bad_demangle (orig);
-           return false;
+           return FALSE;
          }
-       name = *pp;
-       *pp += n;
 
        if (memberp)
          {
            if (**pp == 'C')
              {
-               constp = true;
+               constp = TRUE;
                ++*pp;
              }
            else if (**pp == 'V')
              {
-               volatilep = true;
+               volatilep = TRUE;
                ++*pp;
              }
            if (**pp != 'F')
              {
                stab_bad_demangle (orig);
-               return false;
+               return FALSE;
              }
            ++*pp;
            if (! stab_demangle_args (minfo, pp,
@@ -4770,31 +4705,23 @@ stab_demangle_type (minfo, pp, ptype)
                                       ? (debug_type **) NULL
                                       : &args),
                                      (ptype == NULL
-                                      ? (boolean *) NULL
+                                      ? (bfd_boolean *) NULL
                                       : &varargs)))
-             return false;
+             return FALSE;
          }
 
        if (**pp != '_')
          {
            stab_bad_demangle (orig);
-           return false;
+           return FALSE;
          }
        ++*pp;
 
        if (! stab_demangle_type (minfo, pp, ptype))
-         return false;
+         return FALSE;
 
        if (ptype != NULL)
          {
-           debug_type class_type;
-
-           class_type = stab_find_tagged_type (minfo->dhandle, minfo->info,
-                                               name, (int) n,
-                                               DEBUG_KIND_CLASS);
-           if (class_type == DEBUG_TYPE_NULL)
-             return false;
-
            if (! memberp)
              *ptype = debug_make_offset_type (minfo->dhandle, class_type,
                                               *ptype);
@@ -4812,13 +4739,13 @@ stab_demangle_type (minfo, pp, ptype)
     case 'G':
       ++*pp;
       if (! stab_demangle_type (minfo, pp, ptype))
-       return false;
+       return FALSE;
       break;
 
     case 'C':
       ++*pp;
       if (! stab_demangle_type (minfo, pp, ptype))
-       return false;
+       return FALSE;
       if (ptype != NULL)
        *ptype = debug_make_const_type (minfo->dhandle, *ptype);
       break;
@@ -4829,66 +4756,64 @@ stab_demangle_type (minfo, pp, ptype)
 
        hold = *pp;
        if (! stab_demangle_qualified (minfo, pp, ptype))
-         return false;
+         return FALSE;
       }
       break;
 
     default:
       if (! stab_demangle_fund_type (minfo, pp, ptype))
-       return false;
+       return FALSE;
       break;
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Demangle a fundamental type.  If the ptype argument is not NULL,
    *ptype is set to the newly allocated type.  */
 
-static boolean
-stab_demangle_fund_type (minfo, pp, ptype)
-     struct stab_demangle_info *minfo;
-     const char **pp;
-     debug_type *ptype;
+static bfd_boolean
+stab_demangle_fund_type (struct stab_demangle_info *minfo, const char **pp,
+                        debug_type *ptype)
 {
   const char *orig;
-  boolean constp, volatilep, unsignedp, signedp;
-  boolean done;
+  bfd_boolean constp, volatilep, unsignedp, signedp;
+  bfd_boolean done;
 
   orig = *pp;
 
-  constp = false;
-  volatilep = false;
-  unsignedp = false;
-  signedp = false;
+  constp = FALSE;
+  volatilep = FALSE;
+  unsignedp = FALSE;
+  signedp = FALSE;
 
-  done = false;
+  done = FALSE;
   while (! done)
     {
       switch (**pp)
        {
        case 'C':
-         constp = true;
+         constp = TRUE;
          ++*pp;
          break;
 
        case 'U':
-         unsignedp = true;
+         unsignedp = TRUE;
          ++*pp;
          break;
 
        case 'S':
-         signedp = true;
+         signedp = TRUE;
          ++*pp;
          break;
 
        case 'V':
-         volatilep = true;
+         volatilep = TRUE;
          ++*pp;
          break;
 
        default:
-         done = true;
+         done = TRUE;
          break;
        }
     }
@@ -4993,7 +4918,7 @@ stab_demangle_fund_type (minfo, pp, ptype)
        {
          *ptype = debug_find_named_type (minfo->dhandle, "__wchar_t");
          if (*ptype == DEBUG_TYPE_NULL)
-           *ptype = debug_make_int_type (minfo->dhandle, 2, true);
+           *ptype = debug_make_int_type (minfo->dhandle, 2, TRUE);
        }
       ++*pp;
       break;
@@ -5030,10 +4955,10 @@ stab_demangle_fund_type (minfo, pp, ptype)
 
     case 'G':
       ++*pp;
-      if (! isdigit ((unsigned char) **pp))
+      if (! ISDIGIT (**pp))
        {
          stab_bad_demangle (orig);
-         return false;
+         return FALSE;
        }
       /* Fall through.  */
     case '0': case '1': case '2': case '3': case '4':
@@ -5042,13 +4967,14 @@ stab_demangle_fund_type (minfo, pp, ptype)
        const char *hold;
 
        if (! stab_demangle_class (minfo, pp, &hold))
-         return false;
+         return FALSE;
        if (ptype != NULL)
          {
            char *name;
 
            name = savestring (hold, *pp - hold);
            *ptype = debug_find_named_type (minfo->dhandle, name);
+           free (name);
            if (*ptype == DEBUG_TYPE_NULL)
              {
                /* FIXME: It is probably incorrect to assume that
@@ -5056,34 +4982,35 @@ stab_demangle_fund_type (minfo, pp, ptype)
                *ptype = stab_find_tagged_type (minfo->dhandle, minfo->info,
                                                hold, *pp - hold,
                                                DEBUG_KIND_ILLEGAL);
+               if (*ptype == DEBUG_TYPE_NULL)
+                 return FALSE;
              }
-           free (name);
          }
       }
       break;
 
     case 't':
-      if (! stab_demangle_template (minfo, pp))
-       return false;
-      if (ptype != NULL)
-       {
-         debug_type t;
-
-         /* FIXME: I really don't know how a template should be
-             represented in the current type system.  Perhaps the
-             template should be demangled into a string, and the type
-             should be represented as a named type.  However, I don't
-             know what the base type of the named type should be.  */
-         t = debug_make_void_type (minfo->dhandle);
-         t = debug_make_pointer_type (minfo->dhandle, t);
-         t = debug_name_type (minfo->dhandle, "TEMPLATE", t);
-         *ptype = t;
-       }
+      {
+       char *name;
+
+       if (! stab_demangle_template (minfo, pp,
+                                     ptype != NULL ? &name : NULL))
+         return FALSE;
+       if (ptype != NULL)
+         {
+           *ptype = stab_find_tagged_type (minfo->dhandle, minfo->info,
+                                           name, strlen (name),
+                                           DEBUG_KIND_CLASS);
+           free (name);
+           if (*ptype == DEBUG_TYPE_NULL)
+             return FALSE;
+         }
+      }
       break;
 
     default:
       stab_bad_demangle (orig);
-      return false;
+      return FALSE;
     }
 
   if (ptype != NULL)
@@ -5094,16 +5021,14 @@ stab_demangle_fund_type (minfo, pp, ptype)
        *ptype = debug_make_volatile_type (minfo->dhandle, *ptype);
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Remember a type string in a demangled string.  */
 
-static boolean
-stab_demangle_remember_type (minfo, p, len)
-     struct stab_demangle_info *minfo;
-     const char *p;
-     int len;
+static bfd_boolean
+stab_demangle_remember_type (struct stab_demangle_info *minfo,
+                            const char *p, int len)
 {
   if (minfo->typestring_count >= minfo->typestring_alloc)
     {
@@ -5118,5 +5043,5 @@ stab_demangle_remember_type (minfo, p, len)
   minfo->typestrings[minfo->typestring_count].len = (unsigned int) len;
   ++minfo->typestring_count;
 
-  return true;
+  return TRUE;
 }
This page took 0.080064 seconds and 4 git commands to generate.