OSDN Git Service

*** empty log message ***
[pf3gnuchains/sourceware.git] / gdb / c-typeprint.c
index b18e437..c70fa4b 100644 (file)
@@ -1,6 +1,6 @@
 /* Support for printing C and C++ types for GDB, the GNU debugger.
    Copyright (C) 1986, 1988, 1989, 1991, 1992, 1993, 1994, 1995, 1996, 1998,
-   1999, 2000, 2001, 2002, 2003, 2006, 2007, 2008
+   1999, 2000, 2001, 2002, 2003, 2006, 2007, 2008, 2009, 2010, 2011
    Free Software Foundation, Inc.
 
    This file is part of GDB.
@@ -20,7 +20,7 @@
 
 #include "defs.h"
 #include "gdb_obstack.h"
-#include "bfd.h"               /* Binary File Description */
+#include "bfd.h"               /* Binary File Description */
 #include "symtab.h"
 #include "gdbtypes.h"
 #include "expression.h"
 #include "c-lang.h"
 #include "typeprint.h"
 #include "cp-abi.h"
-
+#include "jv-lang.h"
 #include "gdb_string.h"
 #include <errno.h>
 
-static void cp_type_print_method_args (struct type *mtype, char *prefix,
-                                      char *varstring, int staticp,
-                                      struct ui_file *stream);
-
-static void c_type_print_args (struct type *, struct ui_file *);
-
-static void cp_type_print_derivation_info (struct ui_file *, struct type *);
+static void c_type_print_varspec_prefix (struct type *,
+                                        struct ui_file *,
+                                        int, int, int);
 
-static void c_type_print_varspec_prefix (struct type *, struct ui_file *, int,
-                                        int, int);
-
-/* Print "const", "volatile", or address space modifiers. */
-static void c_type_print_modifier (struct type *, struct ui_file *,
+/* Print "const", "volatile", or address space modifiers.  */
+static void c_type_print_modifier (struct type *,
+                                  struct ui_file *,
                                   int, int);
 \f
-
-
-
 /* LEVEL is the depth to indent lines by.  */
 
 void
-c_print_type (struct type *type, char *varstring, struct ui_file *stream,
+c_print_type (struct type *type,
+             const char *varstring,
+             struct ui_file *stream,
              int show, int level)
 {
   enum type_code code;
@@ -70,17 +63,15 @@ c_print_type (struct type *type, char *varstring, struct ui_file *stream,
   c_type_print_base (type, stream, show, level);
   code = TYPE_CODE (type);
   if ((varstring != NULL && *varstring != '\0')
-      ||
   /* Need a space if going to print stars or brackets;
      but not if we will print just a type name.  */
-      ((show > 0 || TYPE_NAME (type) == 0)
-       &&
-       (code == TYPE_CODE_PTR || code == TYPE_CODE_FUNC
-       || code == TYPE_CODE_METHOD
-       || code == TYPE_CODE_ARRAY
-       || code == TYPE_CODE_MEMBERPTR
-       || code == TYPE_CODE_METHODPTR
-       || code == TYPE_CODE_REF)))
+      || ((show > 0 || TYPE_NAME (type) == 0)
+         && (code == TYPE_CODE_PTR || code == TYPE_CODE_FUNC
+             || code == TYPE_CODE_METHOD
+             || code == TYPE_CODE_ARRAY
+             || code == TYPE_CODE_MEMBERPTR
+             || code == TYPE_CODE_METHODPTR
+             || code == TYPE_CODE_REF)))
     fputs_filtered (" ", stream);
   need_post_space = (varstring != NULL && strcmp (varstring, "") != 0);
   c_type_print_varspec_prefix (type, stream, show, 0, need_post_space);
@@ -89,11 +80,12 @@ c_print_type (struct type *type, char *varstring, struct ui_file *stream,
     {
       fputs_filtered (varstring, stream);
 
-      /* For demangled function names, we have the arglist as part of the name,
-         so don't print an additional pair of ()'s */
+      /* For demangled function names, we have the arglist as part of
+         the name, so don't print an additional pair of ()'s.  */
 
       demangled_args = strchr (varstring, '(') != NULL;
-      c_type_print_varspec_suffix (type, stream, show, 0, demangled_args);
+      c_type_print_varspec_suffix (type, stream, show,
+                                  0, demangled_args);
     }
 }
 
@@ -102,7 +94,8 @@ c_print_type (struct type *type, char *varstring, struct ui_file *stream,
    which to print.  */
 
 void
-c_print_typedef (struct type *type, struct symbol *new_symbol,
+c_print_typedef (struct type *type,
+                struct symbol *new_symbol,
                 struct ui_file *stream)
 {
   CHECK_TYPEDEF (type);
@@ -110,14 +103,15 @@ c_print_typedef (struct type *type, struct symbol *new_symbol,
   type_print (type, "", stream, 0);
   if (TYPE_NAME ((SYMBOL_TYPE (new_symbol))) == 0
       || strcmp (TYPE_NAME ((SYMBOL_TYPE (new_symbol))),
-                SYMBOL_LINKAGE_NAME (new_symbol)) != 0)
+                SYMBOL_LINKAGE_NAME (new_symbol)) != 0
+      || TYPE_CODE (SYMBOL_TYPE (new_symbol)) == TYPE_CODE_TYPEDEF)
     fprintf_filtered (stream, " %s", SYMBOL_PRINT_NAME (new_symbol));
   fprintf_filtered (stream, ";\n");
 }
 
 /* If TYPE is a derived type, then print out derivation information.
-   Print only the actual base classes of this type, not the base classes
-   of the base classes.  I.E.  for the derivation hierarchy:
+   Print only the actual base classes of this type, not the base
+   classes of the base classes.  I.e. for the derivation hierarchy:
 
    class A { int a; };
    class B : public A {int b; };
@@ -129,21 +123,25 @@ c_print_typedef (struct type *type, struct symbol *new_symbol,
    int c;
    }
 
-   Not as the following (like gdb used to), which is not legal C++ syntax for
-   derived types and may be confused with the multiple inheritance form:
+   Not as the following (like gdb used to), which is not legal C++
+   syntax for derived types and may be confused with the multiple
+   inheritance form:
 
    class C : public B : public A {
    int c;
    }
 
-   In general, gdb should try to print the types as closely as possible to
-   the form that they appear in the source code. 
-   Note that in case of protected derivation gcc will not say 'protected' 
-   but 'private'. The HP's aCC compiler emits specific information for 
-   derivation via protected inheritance, so gdb can print it out */
+   In general, gdb should try to print the types as closely as
+   possible to the form that they appear in the source code.
+
+   Note that in case of protected derivation gcc will not say
+   'protected' but 'private'.  The HP's aCC compiler emits specific
+   information for derivation via protected inheritance, so gdb can
+   print it out */
 
 static void
-cp_type_print_derivation_info (struct ui_file *stream, struct type *type)
+cp_type_print_derivation_info (struct ui_file *stream,
+                              struct type *type)
 {
   char *name;
   int i;
@@ -152,8 +150,9 @@ cp_type_print_derivation_info (struct ui_file *stream, struct type *type)
     {
       fputs_filtered (i == 0 ? ": " : ", ", stream);
       fprintf_filtered (stream, "%s%s ",
-                       BASETYPE_VIA_PUBLIC (type, i) ? "public"
-              : (TYPE_FIELD_PROTECTED (type, i) ? "protected" : "private"),
+                       BASETYPE_VIA_PUBLIC (type, i)
+                       ? "public" : (TYPE_FIELD_PROTECTED (type, i)
+                                     ? "protected" : "private"),
                        BASETYPE_VIA_VIRTUAL (type, i) ? " virtual" : "");
       name = type_name_no_tag (TYPE_BASECLASS (type, i));
       fprintf_filtered (stream, "%s", name ? name : "(null)");
@@ -167,16 +166,19 @@ cp_type_print_derivation_info (struct ui_file *stream, struct type *type)
 /* Print the C++ method arguments ARGS to the file STREAM.  */
 
 static void
-cp_type_print_method_args (struct type *mtype, char *prefix, char *varstring,
-                          int staticp, struct ui_file *stream)
+cp_type_print_method_args (struct type *mtype, char *prefix,
+                          char *varstring, int staticp,
+                          struct ui_file *stream)
 {
   struct field *args = TYPE_FIELDS (mtype);
   int nargs = TYPE_NFIELDS (mtype);
   int varargs = TYPE_VARARGS (mtype);
   int i;
 
-  fprintf_symbol_filtered (stream, prefix, language_cplus, DMGL_ANSI);
-  fprintf_symbol_filtered (stream, varstring, language_cplus, DMGL_ANSI);
+  fprintf_symbol_filtered (stream, prefix,
+                          language_cplus, DMGL_ANSI);
+  fprintf_symbol_filtered (stream, varstring,
+                          language_cplus, DMGL_ANSI);
   fputs_filtered ("(", stream);
 
   /* Skip the class variable.  */
@@ -199,6 +201,23 @@ cp_type_print_method_args (struct type *mtype, char *prefix, char *varstring,
     fprintf_filtered (stream, "void");
 
   fprintf_filtered (stream, ")");
+
+  /* For non-static methods, read qualifiers from the type of
+     THIS.  */
+  if (!staticp)
+    {
+      struct type *domain;
+
+      gdb_assert (nargs > 0);
+      gdb_assert (TYPE_CODE (args[0].type) == TYPE_CODE_PTR);
+      domain = TYPE_TARGET_TYPE (args[0].type);
+
+      if (TYPE_CONST (domain))
+       fprintf_filtered (stream, " const");
+
+      if (TYPE_VOLATILE (domain))
+       fprintf_filtered (stream, " volatile");
+    }
 }
 
 
@@ -214,11 +233,14 @@ cp_type_print_method_args (struct type *mtype, char *prefix, char *varstring,
    between a trailing qualifier and a field, variable, or function
    name.  */
 
-void
-c_type_print_varspec_prefix (struct type *type, struct ui_file *stream,
-                            int show, int passed_a_ptr, int need_post_space)
+static void
+c_type_print_varspec_prefix (struct type *type,
+                            struct ui_file *stream,
+                            int show, int passed_a_ptr,
+                            int need_post_space)
 {
   char *name;
+
   if (type == 0)
     return;
 
@@ -230,53 +252,62 @@ c_type_print_varspec_prefix (struct type *type, struct ui_file *stream,
   switch (TYPE_CODE (type))
     {
     case TYPE_CODE_PTR:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 1, 1);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 1, 1);
       fprintf_filtered (stream, "*");
       c_type_print_modifier (type, stream, 1, need_post_space);
       break;
 
     case TYPE_CODE_MEMBERPTR:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 0, 0);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 0, 0);
       name = type_name_no_tag (TYPE_DOMAIN_TYPE (type));
       if (name)
        fputs_filtered (name, stream);
       else
-       c_type_print_base (TYPE_DOMAIN_TYPE (type), stream, 0, passed_a_ptr);
+       c_type_print_base (TYPE_DOMAIN_TYPE (type),
+                          stream, 0, passed_a_ptr);
       fprintf_filtered (stream, "::*");
       break;
 
     case TYPE_CODE_METHODPTR:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 0, 0);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 0, 0);
       fprintf_filtered (stream, "(");
       name = type_name_no_tag (TYPE_DOMAIN_TYPE (type));
       if (name)
        fputs_filtered (name, stream);
       else
-       c_type_print_base (TYPE_DOMAIN_TYPE (type), stream, 0, passed_a_ptr);
+       c_type_print_base (TYPE_DOMAIN_TYPE (type),
+                          stream, 0, passed_a_ptr);
       fprintf_filtered (stream, "::*");
       break;
 
     case TYPE_CODE_REF:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 1, 0);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 1, 0);
       fprintf_filtered (stream, "&");
       c_type_print_modifier (type, stream, 1, need_post_space);
       break;
 
     case TYPE_CODE_METHOD:
     case TYPE_CODE_FUNC:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 0, 0);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 0, 0);
       if (passed_a_ptr)
        fprintf_filtered (stream, "(");
       break;
 
     case TYPE_CODE_ARRAY:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 0, 0);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 0, 0);
       if (passed_a_ptr)
        fprintf_filtered (stream, "(");
       break;
 
     case TYPE_CODE_TYPEDEF:
-      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, show, 0, 0);
+      c_type_print_varspec_prefix (TYPE_TARGET_TYPE (type),
+                                  stream, show, 0, 0);
       break;
 
     case TYPE_CODE_UNDEF:
@@ -294,7 +325,6 @@ c_type_print_varspec_prefix (struct type *type, struct ui_file *stream,
     case TYPE_CODE_STRING:
     case TYPE_CODE_BITSTRING:
     case TYPE_CODE_COMPLEX:
-    case TYPE_CODE_TEMPLATE:
     case TYPE_CODE_NAMESPACE:
     case TYPE_CODE_DECFLOAT:
       /* These types need no prefix.  They are listed here so that
@@ -306,10 +336,12 @@ c_type_print_varspec_prefix (struct type *type, struct ui_file *stream,
     }
 }
 
-/* Print out "const" and "volatile" attributes.
+/* Print out "const" and "volatile" attributes,
+   and address space id if present.
    TYPE is a pointer to the type being printed out.
    STREAM is the output destination.
-   NEED_SPACE = 1 indicates an initial white space is needed */
+   NEED_PRE_SPACE = 1 indicates an initial white space is needed.
+   NEED_POST_SPACE = 1 indicates a final white space is needed.  */
 
 static void
 c_type_print_modifier (struct type *type, struct ui_file *stream,
@@ -338,7 +370,8 @@ c_type_print_modifier (struct type *type, struct ui_file *stream,
       did_print_modifier = 1;
     }
 
-  address_space_id = address_space_int_to_name (TYPE_INSTANCE_FLAGS (type));
+  address_space_id = address_space_int_to_name (get_type_arch (type),
+                                               TYPE_INSTANCE_FLAGS (type));
   if (address_space_id)
     {
       if (did_print_modifier || need_pre_space)
@@ -354,10 +387,17 @@ c_type_print_modifier (struct type *type, struct ui_file *stream,
 
 /* Print out the arguments of TYPE, which should have TYPE_CODE_METHOD
    or TYPE_CODE_FUNC, to STREAM.  Artificial arguments, such as "this"
-   in non-static methods, are displayed.  */
+   in non-static methods, are displayed if SHOW_ARTIFICIAL is
+   non-zero.  If SHOW_ARTIFICIAL is zero and LANGUAGE is language_cplus
+   the topmost parameter types get removed their possible const and volatile
+   qualifiers to match demangled linkage name parameters part of such function
+   type.  LANGUAGE is the language in which TYPE was defined.  This is
+   a necessary evil since this code is used by the C, C++, and Java backends.
+   */
 
-static void
-c_type_print_args (struct type *type, struct ui_file *stream)
+void
+c_type_print_args (struct type *type, struct ui_file *stream,
+                  int show_artificial, enum language language)
 {
   int i, len;
   struct field *args;
@@ -369,13 +409,35 @@ c_type_print_args (struct type *type, struct ui_file *stream)
 
   for (i = 0; i < TYPE_NFIELDS (type); i++)
     {
+      struct type *param_type;
+
+      if (TYPE_FIELD_ARTIFICIAL (type, i) && !show_artificial)
+       continue;
+
       if (printed_any)
        {
          fprintf_filtered (stream, ", ");
          wrap_here ("    ");
        }
 
-      c_print_type (TYPE_FIELD_TYPE (type, i), "", stream, -1, 0);
+      param_type = TYPE_FIELD_TYPE (type, i);
+
+      if (language == language_cplus && !show_artificial)
+       {
+         /* C++ standard, 13.1 Overloadable declarations, point 3, item:
+            - Parameter declarations that differ only in the presence or
+              absence of const and/or volatile are equivalent.
+
+            And the const/volatile qualifiers are not present in the mangled
+            names as produced by GCC.  */
+
+         param_type = make_cv_type (0, 0, param_type, NULL);
+       }
+
+      if (language == language_java)
+       java_print_type (param_type, "", stream, -1, 0);
+      else
+       c_print_type (param_type, "", stream, -1, 0);
       printed_any = 1;
     }
 
@@ -392,18 +454,18 @@ c_type_print_args (struct type *type, struct ui_file *stream)
        }
     }
   else if (!printed_any
-      && (TYPE_PROTOTYPED (type)
-         || current_language->la_language == language_cplus))
+          && ((TYPE_PROTOTYPED (type) && language != language_java)
+              || language == language_cplus))
     fprintf_filtered (stream, "void");
 
   fprintf_filtered (stream, ")");
 }
 
-
 /* Return true iff the j'th overloading of the i'th method of TYPE
    is a type conversion operator, like `operator int () { ... }'.
    When listing a class's methods, we don't print the return type of
    such operators.  */
+
 static int
 is_type_conversion_operator (struct type *type, int i, int j)
 {
@@ -448,11 +510,11 @@ is_type_conversion_operator (struct type *type, int i, int j)
     return 1;
 
   /* That was indeed the end of the name, so it was `operator new' or
-     `operator delete', neither of which are type conversion operators.  */
+     `operator delete', neither of which are type conversion
+     operators.  */
   return 0;
 }
 
-
 /* Given a C++ qualified identifier QID, strip off the qualifiers,
    yielding the unqualified name.  The return value is a pointer into
    the original string.
@@ -460,17 +522,18 @@ is_type_conversion_operator (struct type *type, int i, int j)
    It's a pity we don't have this information in some more structured
    form.  Even the author of this function feels that writing little
    parsers like this everywhere is stupid.  */
+
 static char *
 remove_qualifiers (char *qid)
 {
-  int quoted = 0;              /* zero if we're not in quotes;
-                                  '"' if we're in a double-quoted string;
-                                  '\'' if we're in a single-quoted string.  */
-  int depth = 0;               /* number of unclosed parens we've seen */
+  int quoted = 0;      /* Zero if we're not in quotes;
+                          '"' if we're in a double-quoted string;
+                          '\'' if we're in a single-quoted string.  */
+  int depth = 0;       /* Number of unclosed parens we've seen.  */
   char *parenstack = (char *) alloca (strlen (qid));
   char *scan;
-  char *last = 0;              /* The character after the rightmost
-                                  `::' token we've seen so far.  */
+  char *last = 0;      /* The character after the rightmost
+                          `::' token we've seen so far.  */
 
   for (scan = qid; *scan; scan++)
     {
@@ -486,8 +549,8 @@ remove_qualifiers (char *qid)
          /* If we're inside parenthesis (i.e., an argument list) or
             angle brackets (i.e., a list of template arguments), then
             we don't record the position of this :: token, since it's
-            not relevant to the top-level structure we're trying
-            to operate on.  */
+            not relevant to the top-level structure we're trying to
+            operate on.  */
          if (depth == 0)
            {
              last = scan + 2;
@@ -511,10 +574,10 @@ remove_qualifiers (char *qid)
            depth--;
          else
            {
-             /* We're going to do a little error recovery here.  If we
-                don't find a match for *scan on the paren stack, but
-                there is something lower on the stack that does match, we
-                pop the stack to that point.  */
+             /* We're going to do a little error recovery here.  If
+                we don't find a match for *scan on the paren stack,
+                but there is something lower on the stack that does
+                match, we pop the stack to that point.  */
              int i;
 
              for (i = depth - 1; i >= 0; i--)
@@ -535,14 +598,15 @@ remove_qualifiers (char *qid)
     return qid;
 }
 
-
 /* Print any array sizes, function arguments or close parentheses
    needed after the variable name (to describe its type).
    Args work like c_type_print_varspec_prefix.  */
 
 void
-c_type_print_varspec_suffix (struct type *type, struct ui_file *stream,
-                            int show, int passed_a_ptr, int demangled_args)
+c_type_print_varspec_suffix (struct type *type,
+                            struct ui_file *stream,
+                            int show, int passed_a_ptr,
+                            int demangled_args)
 {
   if (type == 0)
     return;
@@ -555,36 +619,38 @@ c_type_print_varspec_suffix (struct type *type, struct ui_file *stream,
   switch (TYPE_CODE (type))
     {
     case TYPE_CODE_ARRAY:
-      if (passed_a_ptr)
-       fprintf_filtered (stream, ")");
+      {
+       LONGEST low_bound, high_bound;
+
+       if (passed_a_ptr)
+         fprintf_filtered (stream, ")");
 
-      fprintf_filtered (stream, "[");
-      if (TYPE_LENGTH (TYPE_TARGET_TYPE (type)) > 0
-       && !TYPE_ARRAY_UPPER_BOUND_IS_UNDEFINED (type))
-       fprintf_filtered (stream, "%d",
-                         (TYPE_LENGTH (type)
-                          / TYPE_LENGTH (TYPE_TARGET_TYPE (type))));
-      fprintf_filtered (stream, "]");
+       fprintf_filtered (stream, "[");
+       if (get_array_bounds (type, &low_bound, &high_bound))
+         fprintf_filtered (stream, "%d", 
+                           (int) (high_bound - low_bound + 1));
+       fprintf_filtered (stream, "]");
 
-      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, show,
-                                  0, 0);
+       c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream,
+                                    show, 0, 0);
+      }
       break;
 
     case TYPE_CODE_MEMBERPTR:
-      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, show,
-                                  0, 0);
+      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream,
+                                  show, 0, 0);
       break;
 
     case TYPE_CODE_METHODPTR:
       fprintf_filtered (stream, ")");
-      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, show,
-                                  0, 0);
+      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream,
+                                  show, 0, 0);
       break;
 
     case TYPE_CODE_PTR:
     case TYPE_CODE_REF:
-      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, show,
-                                  1, 0);
+      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream,
+                                  show, 1, 0);
       break;
 
     case TYPE_CODE_METHOD:
@@ -592,14 +658,15 @@ c_type_print_varspec_suffix (struct type *type, struct ui_file *stream,
       if (passed_a_ptr)
        fprintf_filtered (stream, ")");
       if (!demangled_args)
-       c_type_print_args (type, stream);
-      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, show,
-                                  passed_a_ptr, 0);
+       c_type_print_args (type, stream, 1,
+                          current_language->la_language);
+      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream,
+                                  show, passed_a_ptr, 0);
       break;
 
     case TYPE_CODE_TYPEDEF:
-      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, show,
-                                  passed_a_ptr, 0);
+      c_type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream,
+                                  show, passed_a_ptr, 0);
       break;
 
     case TYPE_CODE_UNDEF:
@@ -617,11 +684,11 @@ c_type_print_varspec_suffix (struct type *type, struct ui_file *stream,
     case TYPE_CODE_STRING:
     case TYPE_CODE_BITSTRING:
     case TYPE_CODE_COMPLEX:
-    case TYPE_CODE_TEMPLATE:
     case TYPE_CODE_NAMESPACE:
     case TYPE_CODE_DECFLOAT:
       /* These types do not need a suffix.  They are listed so that
-         gcc -Wall will report types that may not have been considered.  */
+         gcc -Wall will report types that may not have been
+         considered.  */
       break;
     default:
       error (_("type not handled in c_type_print_varspec_suffix()"));
@@ -630,24 +697,26 @@ c_type_print_varspec_suffix (struct type *type, struct ui_file *stream,
 }
 
 /* Print the name of the type (or the ultimate pointer target,
-   function value or array element), or the description of a
-   structure or union.
-
-   SHOW positive means print details about the type (e.g. enum values),
-   and print structure elements passing SHOW - 1 for show.
-   SHOW negative means just print the type name or struct tag if there is one.
-   If there is no name, print something sensible but concise like
-   "struct {...}".
-   SHOW zero means just print the type name or struct tag if there is one.
-   If there is no name, print something sensible but not as concise like
-   "struct {int x; int y;}".
+   function value or array element), or the description of a structure
+   or union.
+
+   SHOW positive means print details about the type (e.g. enum
+   values), and print structure elements passing SHOW - 1 for show.
+
+   SHOW negative means just print the type name or struct tag if there
+   is one.  If there is no name, print something sensible but concise
+   like "struct {...}".
+
+   SHOW zero means just print the type name or struct tag if there is
+   one.  If there is no name, print something sensible but not as
+   concise like "struct {int x; int y;}".
 
    LEVEL is the number of spaces to indent by.
    We increase it for some recursive calls.  */
 
 void
-c_type_print_base (struct type *type, struct ui_file *stream, int show,
-                  int level)
+c_type_print_base (struct type *type, struct ui_file *stream,
+                  int show, int level)
 {
   int i;
   int len, real_len;
@@ -672,11 +741,12 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
       return;
     }
 
-  /* When SHOW is zero or less, and there is a valid type name, then always
-     just print the type name directly from the type.  */
-  /* If we have "typedef struct foo {. . .} bar;" do we want to print it
-     as "struct foo" or as "bar"?  Pick the latter, because C++ folk tend
-     to expect things like "class5 *foo" rather than "struct class5 *foo".  */
+  /* When SHOW is zero or less, and there is a valid type name, then
+     always just print the type name directly from the type.  */
+  /* If we have "typedef struct foo {. . .} bar;" do we want to print
+     it as "struct foo" or as "bar"?  Pick the latter, because C++
+     folk tend to expect things like "class5 *foo" rather than "struct
+     class5 *foo".  */
 
   if (show <= 0
       && TYPE_NAME (type) != NULL)
@@ -691,6 +761,13 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
   switch (TYPE_CODE (type))
     {
     case TYPE_CODE_TYPEDEF:
+      /* If we get here, the typedef doesn't have a name, and we
+        couldn't resolve TYPE_TARGET_TYPE.  Not much we can do.  */
+      gdb_assert (TYPE_NAME (type) == NULL);
+      gdb_assert (TYPE_TARGET_TYPE (type) == NULL);
+      fprintf_filtered (stream, _("<unnamed typedef>"));
+      break;
+
     case TYPE_CODE_ARRAY:
     case TYPE_CODE_PTR:
     case TYPE_CODE_MEMBERPTR:
@@ -698,40 +775,16 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
     case TYPE_CODE_FUNC:
     case TYPE_CODE_METHOD:
     case TYPE_CODE_METHODPTR:
-      c_type_print_base (TYPE_TARGET_TYPE (type), stream, show, level);
+      c_type_print_base (TYPE_TARGET_TYPE (type),
+                        stream, show, level);
       break;
 
     case TYPE_CODE_STRUCT:
       c_type_print_modifier (type, stream, 0, 1);
-      /* Note TYPE_CODE_STRUCT and TYPE_CODE_CLASS have the same value,
-       * so we use another means for distinguishing them.
-       */
-      if (HAVE_CPLUS_STRUCT (type))
-       {
-         switch (TYPE_DECLARED_TYPE (type))
-           {
-           case DECLARED_TYPE_CLASS:
-             fprintf_filtered (stream, "class ");
-             break;
-           case DECLARED_TYPE_UNION:
-             fprintf_filtered (stream, "union ");
-             break;
-           case DECLARED_TYPE_STRUCT:
-             fprintf_filtered (stream, "struct ");
-             break;
-           default:
-             /* If there is a CPLUS_STRUCT, assume class if not
-              * otherwise specified in the declared_type field.
-              */
-             fprintf_filtered (stream, "class ");
-             break;
-           }                   /* switch */
-       }
+      if (TYPE_DECLARED_CLASS (type))
+       fprintf_filtered (stream, "class ");
       else
-       {
-         /* If not CPLUS_STRUCT, then assume it's a C struct */
-         fprintf_filtered (stream, "struct ");
-       }
+       fprintf_filtered (stream, "struct ");
       goto struct_union;
 
     case TYPE_CODE_UNION:
@@ -740,14 +793,12 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
 
     struct_union:
 
-      /* Print the tag if it exists. 
-       * The HP aCC compiler emits
-       * a spurious "{unnamed struct}"/"{unnamed union}"/"{unnamed enum}"
-       * tag  for unnamed struct/union/enum's, which we don't
-       * want to print.
-       */
-      if (TYPE_TAG_NAME (type) != NULL &&
-         strncmp (TYPE_TAG_NAME (type), "{unnamed", 8))
+      /* Print the tag if it exists.  The HP aCC compiler emits a
+         spurious "{unnamed struct}"/"{unnamed union}"/"{unnamed
+         enum}" tag for unnamed struct/union/enum's, which we don't
+         want to print.  */
+      if (TYPE_TAG_NAME (type) != NULL
+         && strncmp (TYPE_TAG_NAME (type), "{unnamed", 8))
        {
          fputs_filtered (TYPE_TAG_NAME (type), stream);
          if (show > 0)
@@ -756,36 +807,42 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
       wrap_here ("    ");
       if (show < 0)
        {
-         /* If we just printed a tag name, no need to print anything else.  */
+         /* If we just printed a tag name, no need to print anything
+            else.  */
          if (TYPE_TAG_NAME (type) == NULL)
            fprintf_filtered (stream, "{...}");
        }
       else if (show > 0 || TYPE_TAG_NAME (type) == NULL)
        {
+         struct type *basetype;
+         int vptr_fieldno;
+
          cp_type_print_derivation_info (stream, type);
 
          fprintf_filtered (stream, "{\n");
-         if ((TYPE_NFIELDS (type) == 0) && (TYPE_NFN_FIELDS (type) == 0))
+         if (TYPE_NFIELDS (type) == 0 && TYPE_NFN_FIELDS (type) == 0
+             && TYPE_TYPEDEF_FIELD_COUNT (type) == 0)
            {
              if (TYPE_STUB (type))
-               fprintfi_filtered (level + 4, stream, _("<incomplete type>\n"));
+               fprintfi_filtered (level + 4, stream,
+                                  _("<incomplete type>\n"));
              else
-               fprintfi_filtered (level + 4, stream, _("<no data fields>\n"));
+               fprintfi_filtered (level + 4, stream,
+                                  _("<no data fields>\n"));
            }
 
          /* Start off with no specific section type, so we can print
             one for the first field we find, and use that section type
-            thereafter until we find another type. */
+            thereafter until we find another type.  */
 
          section_type = s_none;
 
          /* For a class, if all members are private, there's no need
             for a "private:" label; similarly, for a struct or union
             masquerading as a class, if all members are public, there's
-            no need for a "public:" label. */
+            no need for a "public:" label.  */
 
-         if ((TYPE_DECLARED_TYPE (type) == DECLARED_TYPE_CLASS) ||
-             (TYPE_DECLARED_TYPE (type) == DECLARED_TYPE_TEMPLATE))
+         if (TYPE_DECLARED_CLASS (type))
            {
              QUIT;
              len = TYPE_NFIELDS (type);
@@ -803,7 +860,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                    {
                      len = TYPE_FN_FIELDLIST_LENGTH (type, j);
                      for (i = 0; i < len; i++)
-                       if (!TYPE_FN_FIELD_PRIVATE (TYPE_FN_FIELDLIST1 (type, j), i))
+                       if (!TYPE_FN_FIELD_PRIVATE (TYPE_FN_FIELDLIST1 (type,
+                                                                       j), i))
                          {
                            need_access_label = 1;
                            break;
@@ -813,13 +871,13 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                    }
                }
            }
-         else if ((TYPE_DECLARED_TYPE (type) == DECLARED_TYPE_STRUCT) ||
-                  (TYPE_DECLARED_TYPE (type) == DECLARED_TYPE_UNION))
+         else
            {
              QUIT;
              len = TYPE_NFIELDS (type);
              for (i = TYPE_N_BASECLASSES (type); i < len; i++)
-               if (TYPE_FIELD_PRIVATE (type, i) || TYPE_FIELD_PROTECTED (type, i))
+               if (TYPE_FIELD_PRIVATE (type, i)
+                   || TYPE_FIELD_PROTECTED (type, i))
                  {
                    need_access_label = 1;
                    break;
@@ -833,8 +891,11 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                      QUIT;
                      len = TYPE_FN_FIELDLIST_LENGTH (type, j);
                      for (i = 0; i < len; i++)
-                       if (TYPE_FN_FIELD_PRIVATE (TYPE_FN_FIELDLIST1 (type, j), i) ||
-                           TYPE_FN_FIELD_PROTECTED (TYPE_FN_FIELDLIST1 (type, j), i))
+                       if (TYPE_FN_FIELD_PROTECTED (TYPE_FN_FIELDLIST1 (type,
+                                                                        j), i)
+                           || TYPE_FN_FIELD_PRIVATE (TYPE_FN_FIELDLIST1 (type,
+                                                                         j),
+                                                     i))
                          {
                            need_access_label = 1;
                            break;
@@ -849,18 +910,19 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
             do not print the field that it occupies.  */
 
          len = TYPE_NFIELDS (type);
+         vptr_fieldno = get_vptr_fieldno (type, &basetype);
          for (i = TYPE_N_BASECLASSES (type); i < len; i++)
            {
              QUIT;
-             /* Don't print out virtual function table.  */
-             if (strncmp (TYPE_FIELD_NAME (type, i), "_vptr", 5) == 0
-                 && is_cplus_marker ((TYPE_FIELD_NAME (type, i))[5]))
-               continue;
 
-             /* If this is a C++ class we can print the various C++ section
-                labels. */
+             /* If we have a virtual table pointer, omit it.  Even if
+                virtual table pointers are not specifically marked in
+                the debug info, they should be artificial.  */
+             if ((i == vptr_fieldno && type == basetype)
+                 || TYPE_FIELD_ARTIFICIAL (type, i))
+               continue;
 
-             if (HAVE_CPLUS_STRUCT (type) && need_access_label)
+             if (need_access_label)
                {
                  if (TYPE_FIELD_PROTECTED (type, i))
                    {
@@ -876,7 +938,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                      if (section_type != s_private)
                        {
                          section_type = s_private;
-                         fprintfi_filtered (level + 2, stream, "private:\n");
+                         fprintfi_filtered (level + 2, stream,
+                                            "private:\n");
                        }
                    }
                  else
@@ -884,7 +947,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                      if (section_type != s_public)
                        {
                          section_type = s_public;
-                         fprintfi_filtered (level + 2, stream, "public:\n");
+                         fprintfi_filtered (level + 2, stream,
+                                            "public:\n");
                        }
                    }
                }
@@ -910,8 +974,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
            }
 
          /* If there are both fields and methods, put a blank line
-             between them.  Make sure to count only method that we will
-             display; artificial methods will be hidden.  */
+            between them.  Make sure to count only method that we
+            will display; artificial methods will be hidden.  */
          len = TYPE_NFN_FIELDS (type);
          real_len = 0;
          for (i = 0; i < len; i++)
@@ -919,6 +983,7 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
              struct fn_field *f = TYPE_FN_FIELDLIST1 (type, i);
              int len2 = TYPE_FN_FIELDLIST_LENGTH (type, i);
              int j;
+
              for (j = 0; j < len2; j++)
                if (!TYPE_FN_FIELD_ARTIFICIAL (f, j))
                  real_len++;
@@ -926,21 +991,23 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
          if (real_len > 0 && section_type != s_none)
            fprintf_filtered (stream, "\n");
 
-         /* C++: print out the methods */
+         /* C++: print out the methods */
          for (i = 0; i < len; i++)
            {
              struct fn_field *f = TYPE_FN_FIELDLIST1 (type, i);
              int j, len2 = TYPE_FN_FIELDLIST_LENGTH (type, i);
              char *method_name = TYPE_FN_FIELDLIST_NAME (type, i);
              char *name = type_name_no_tag (type);
-             int is_constructor = name && strcmp (method_name, name) == 0;
+             int is_constructor = name && strcmp (method_name,
+                                                  name) == 0;
+
              for (j = 0; j < len2; j++)
                {
                  char *physname = TYPE_FN_FIELD_PHYSNAME (f, j);
                  int is_full_physname_constructor =
-                  is_constructor_name (physname) 
-                  || is_destructor_name (physname)
-                  || method_name[0] == '~';
+                   is_constructor_name (physname) 
+                   || is_destructor_name (physname)
+                   || method_name[0] == '~';
 
                  /* Do not print out artificial methods.  */
                  if (TYPE_FN_FIELD_ARTIFICIAL (f, j))
@@ -961,7 +1028,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                      if (section_type != s_private)
                        {
                          section_type = s_private;
-                         fprintfi_filtered (level + 2, stream, "private:\n");
+                         fprintfi_filtered (level + 2, stream,
+                                            "private:\n");
                        }
                    }
                  else
@@ -969,7 +1037,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                      if (section_type != s_public)
                        {
                          section_type = s_public;
-                         fprintfi_filtered (level + 2, stream, "public:\n");
+                         fprintfi_filtered (level + 2, stream,
+                                            "public:\n");
                        }
                    }
 
@@ -981,13 +1050,16 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                  if (TYPE_TARGET_TYPE (TYPE_FN_FIELD_TYPE (f, j)) == 0)
                    {
                      /* Keep GDB from crashing here.  */
-                     fprintf_filtered (stream, _("<undefined type> %s;\n"),
+                     fprintf_filtered (stream,
+                                       _("<undefined type> %s;\n"),
                                        TYPE_FN_FIELD_PHYSNAME (f, j));
                      break;
                    }
-                 else if (!is_constructor &&   /* constructors don't have declared types */
-                          !is_full_physname_constructor &&     /*    " "  */
-                          !is_type_conversion_operator (type, i, j))
+                 else if (!is_constructor      /* Constructors don't
+                                                  have declared
+                                                  types.  */
+                          && !is_full_physname_constructor  /* " " */
+                          && !is_type_conversion_operator (type, i, j))
                    {
                      type_print (TYPE_TARGET_TYPE (TYPE_FN_FIELD_TYPE (f, j)),
                                  "", stream, -1);
@@ -1004,15 +1076,16 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                                    DMGL_ANSI | DMGL_PARAMS);
                  if (demangled_name == NULL)
                    {
-                     /* in some cases (for instance with the HP demangling),
-                        if a function has more than 10 arguments, 
-                        the demangling will fail.
-                        Let's try to reconstruct the function signature from 
-                        the symbol information */
+                     /* In some cases (for instance with the HP
+                        demangling), if a function has more than 10
+                        arguments, the demangling will fail.
+                        Let's try to reconstruct the function
+                        signature from the symbol information.  */
                      if (!TYPE_FN_FIELD_STUB (f, j))
                        {
                          int staticp = TYPE_FN_FIELD_STATIC_P (f, j);
                          struct type *mtype = TYPE_FN_FIELD_TYPE (f, j);
+
                          cp_type_print_method_args (mtype,
                                                     "",
                                                     method_name,
@@ -1020,7 +1093,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                                                     stream);
                        }
                      else
-                       fprintf_filtered (stream, _("<badly mangled name '%s'>"),
+                       fprintf_filtered (stream,
+                                         _("<badly mangled name '%s'>"),
                                          mangled_name);
                    }
                  else
@@ -1029,13 +1103,17 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                      char *demangled_no_class
                        = remove_qualifiers (demangled_name);
 
-                     /* get rid of the `static' appended by the demangler */
+                     /* Get rid of the `static' appended by the
+                        demangler.  */
                      p = strstr (demangled_no_class, " static");
                      if (p != NULL)
                        {
                          int length = p - demangled_no_class;
-                         demangled_no_static = (char *) xmalloc (length + 1);
-                         strncpy (demangled_no_static, demangled_no_class, length);
+
+                         demangled_no_static
+                           = (char *) xmalloc (length + 1);
+                         strncpy (demangled_no_static,
+                                  demangled_no_class, length);
                          *(demangled_no_static + length) = '\0';
                          fputs_filtered (demangled_no_static, stream);
                          xfree (demangled_no_static);
@@ -1052,10 +1130,34 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
                }
            }
 
+         /* Print typedefs defined in this class.  */
+
+         if (TYPE_TYPEDEF_FIELD_COUNT (type) != 0)
+           {
+             if (TYPE_NFIELDS (type) != 0 || TYPE_NFN_FIELDS (type) != 0)
+               fprintf_filtered (stream, "\n");
+
+             for (i = 0; i < TYPE_TYPEDEF_FIELD_COUNT (type); i++)
+               {
+                 struct type *target = TYPE_TYPEDEF_FIELD_TYPE (type, i);
+
+                 /* Dereference the typedef declaration itself.  */
+                 gdb_assert (TYPE_CODE (target) == TYPE_CODE_TYPEDEF);
+                 target = TYPE_TARGET_TYPE (target);
+
+                 print_spaces_filtered (level + 4, stream);
+                 fprintf_filtered (stream, "typedef ");
+                 c_print_type (target, TYPE_TYPEDEF_FIELD_NAME (type, i),
+                               stream, show - 1, level + 4);
+                 fprintf_filtered (stream, ";\n");
+               }
+           }
+
          fprintfi_filtered (level, stream, "}");
 
          if (TYPE_LOCALTYPE_PTR (type) && show >= 0)
-           fprintfi_filtered (level, stream, _(" (Local at %s:%d)\n"),
+           fprintfi_filtered (level,
+                              stream, _(" (Local at %s:%d)\n"),
                               TYPE_LOCALTYPE_FILE (type),
                               TYPE_LOCALTYPE_LINE (type));
        }
@@ -1068,9 +1170,9 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
          The aCC compiler emits a spurious 
          "{unnamed struct}"/"{unnamed union}"/"{unnamed enum}"
          tag for unnamed struct/union/enum's, which we don't
-         want to print. */
-      if (TYPE_TAG_NAME (type) != NULL &&
-         strncmp (TYPE_TAG_NAME (type), "{unnamed", 8))
+         want to print.  */
+      if (TYPE_TAG_NAME (type) != NULL
+         && strncmp (TYPE_TAG_NAME (type), "{unnamed", 8))
        {
          fputs_filtered (TYPE_TAG_NAME (type), stream);
          if (show > 0)
@@ -1080,7 +1182,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
       wrap_here ("    ");
       if (show < 0)
        {
-         /* If we just printed a tag name, no need to print anything else.  */
+         /* If we just printed a tag name, no need to print anything
+            else.  */
          if (TYPE_TAG_NAME (type) == NULL)
            fprintf_filtered (stream, "{...}");
        }
@@ -1098,7 +1201,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
              fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
              if (lastval != TYPE_FIELD_BITPOS (type, i))
                {
-                 fprintf_filtered (stream, " = %d", TYPE_FIELD_BITPOS (type, i));
+                 fprintf_filtered (stream, " = %d", 
+                                   TYPE_FIELD_BITPOS (type, i));
                  lastval = TYPE_FIELD_BITPOS (type, i);
                }
              lastval++;
@@ -1116,43 +1220,24 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
       break;
 
     case TYPE_CODE_ERROR:
-      fprintf_filtered (stream, _("<unknown type>"));
+      fprintf_filtered (stream, "%s", TYPE_ERROR_NAME (type));
       break;
 
     case TYPE_CODE_RANGE:
-      /* This should not occur */
+      /* This should not occur */
       fprintf_filtered (stream, _("<range type>"));
       break;
 
-    case TYPE_CODE_TEMPLATE:
-      /* Called on "ptype t" where "t" is a template.
-         Prints the template header (with args), e.g.:
-         template <class T1, class T2> class "
-         and then merges with the struct/union/class code to
-         print the rest of the definition. */
-      c_type_print_modifier (type, stream, 0, 1);
-      fprintf_filtered (stream, "template <");
-      for (i = 0; i < TYPE_NTEMPLATE_ARGS (type); i++)
-       {
-         struct template_arg templ_arg;
-         templ_arg = TYPE_TEMPLATE_ARG (type, i);
-         fprintf_filtered (stream, "class %s", templ_arg.name);
-         if (i < TYPE_NTEMPLATE_ARGS (type) - 1)
-           fprintf_filtered (stream, ", ");
-       }
-      fprintf_filtered (stream, "> class ");
-      goto struct_union;
-
     case TYPE_CODE_NAMESPACE:
       fputs_filtered ("namespace ", stream);
       fputs_filtered (TYPE_TAG_NAME (type), stream);
       break;
 
     default:
-      /* Handle types not explicitly handled by the other cases,
-         such as fundamental types.  For these, just print whatever
-         the type name is, as recorded in the type itself.  If there
-         is no type name, then complain. */
+      /* Handle types not explicitly handled by the other cases, such
+         as fundamental types.  For these, just print whatever the
+         type name is, as recorded in the type itself.  If there is no
+         type name, then complain.  */
       if (TYPE_NAME (type) != NULL)
        {
          c_type_print_modifier (type, stream, 0, 1);
@@ -1160,8 +1245,8 @@ c_type_print_base (struct type *type, struct ui_file *stream, int show,
        }
       else
        {
-         /* At least for dump_symtab, it is important that this not be
-            an error ().  */
+         /* At least for dump_symtab, it is important that this not
+            be an error ().  */
          fprintf_filtered (stream, _("<invalid type code %d>"),
                            TYPE_CODE (type));
        }