> * gdb.texinfo (hbreak, watch): Fix typo, "date" -> "data".
[deliverable/binutils-gdb.git] / gdb / valprint.c
index 9a3ab9055fd27b90445e3319b3f506efc08ae7ff..c5d01b188de5e99e6ddbca030e4020e05be325e0 100644 (file)
@@ -1,50 +1,75 @@
-/* Print values for GNU debugger gdb.
-   Copyright (C) 1986, 1988, 1989 Free Software Foundation, Inc.
+/* Print values for GDB, the GNU debugger.
+   Copyright 1986, 1988, 1989, 1991, 1992, 1993, 1994
+             Free Software Foundation, Inc.
 
 This file is part of GDB.
 
-GDB is free software; you can redistribute it and/or modify
+This program is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 1, or (at your option)
-any later version.
+the Free Software Foundation; either version 2 of the License, or
+(at your option) any later version.
 
-GDB is distributed in the hope that it will be useful,
+This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
-along with GDB; see the file COPYING.  If not, write to
-the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
+along with this program; if not, write to the Free Software
+Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
-#include <stdio.h>
-#include <string.h>
 #include "defs.h"
-#include "param.h"
+#include "gdb_string.h"
 #include "symtab.h"
+#include "gdbtypes.h"
 #include "value.h"
 #include "gdbcore.h"
 #include "gdbcmd.h"
 #include "target.h"
 #include "obstack.h"
+#include "language.h"
+#include "demangle.h"
+#include "annotate.h"
+#include "valprint.h"
 
 #include <errno.h>
-extern int sys_nerr;
-extern char *sys_errlist[];
 
-extern void print_scalar_formatted();  /* printcmd.c */
-extern void print_address_demangle();  /* printcmd.c */
-extern int demangle;   /* whether to print C++ syms raw or source-form */
+/* Prototypes for local functions */
 
-/* Maximum number of chars to print for a string pointer value
-   or vector contents, or UINT_MAX for no limit.  */
+static void
+print_hex_chars PARAMS ((GDB_FILE *, unsigned char *, unsigned int));
+
+static void
+show_print PARAMS ((char *, int));
+
+static void
+set_print PARAMS ((char *, int));
+
+static void
+set_radix PARAMS ((char *, int));
+
+static void
+show_radix PARAMS ((char *, int));
+
+static void
+set_input_radix PARAMS ((char *, int, struct cmd_list_element *));
+
+static void
+set_input_radix_1 PARAMS ((int, unsigned));
+
+static void
+set_output_radix PARAMS ((char *, int, struct cmd_list_element *));
+
+static void
+set_output_radix_1 PARAMS ((int, unsigned));
 
-static unsigned int print_max;
+/* Maximum number of chars to print for a string pointer value or vector
+   contents, or UINT_MAX for no limit.  Note that "set print elements 0"
+   stores UINT_MAX in print_max, which displays in a show command as
+   "unlimited". */
 
-static void type_print_varspec_suffix ();
-static void type_print_varspec_prefix ();
-static void type_print_base ();
-static void type_print_method_args ();
+unsigned int print_max;
+#define PRINT_MAX_DEFAULT 200  /* Start print_max off at this value. */
 
 /* Default input and output radixes, and output format letter.  */
 
@@ -52,107 +77,342 @@ unsigned input_radix = 10;
 unsigned output_radix = 10;
 int output_format = 0;
 
+/* Print repeat counts if there are more than this many repetitions of an
+   element in an array.  Referenced by the low level language dependent
+   print routines. */
+
+unsigned int repeat_count_threshold = 10;
+
+/* If nonzero, stops printing of char arrays at first null. */
+
+int stop_print_at_null;
+
+/* Controls pretty printing of structures. */
+
+int prettyprint_structs;
+
+/* Controls pretty printing of arrays.  */
+
+int prettyprint_arrays;
 
-char **unsigned_type_table;
-char **signed_type_table;
-char **float_type_table;
+/* If nonzero, causes unions inside structures or other unions to be
+   printed. */
 
+int unionprint;                        /* Controls printing of nested unions.  */
+
+/* If nonzero, causes machine addresses to be printed in certain contexts. */
+
+int addressprint;              /* Controls printing of machine addresses */
 
-/* Print repeat counts if there are more than this
-   many repetitions of an element in an array.  */
-#define        REPEAT_COUNT_THRESHOLD  10
 \f
-/* Print the character string STRING, printing at most LENGTH characters.
-   Printing stops early if the number hits print_max; repeat counts
-   are printed as appropriate.  Print ellipses at the end if we
-   had to stop before printing LENGTH characters, or if FORCE_ELLIPSES.  */
+/* Print data of type TYPE located at VALADDR (within GDB), which came from
+   the inferior at address ADDRESS, onto stdio stream STREAM according to
+   FORMAT (a letter, or 0 for natural format using TYPE).
 
-void
-print_string (stream, string, length, force_ellipses)
-     FILE *stream;
-     char *string;
-     unsigned int length;
-     int force_ellipses;
+   If DEREF_REF is nonzero, then dereference references, otherwise just print
+   them like pointers.
+
+   The PRETTY parameter controls prettyprinting.
+
+   If the data are a string pointer, returns the number of string characters
+   printed.
+
+   FIXME:  The data at VALADDR is in target byte order.  If gdb is ever
+   enhanced to be able to debug more than the single target it was compiled
+   for (specific CPU type and thus specific target byte ordering), then
+   either the print routines are going to have to take this into account,
+   or the data is going to have to be passed into here already converted
+   to the host byte ordering, whichever is more convenient. */
+
+
+int
+val_print (type, valaddr, address, stream, format, deref_ref, recurse, pretty)
+     struct type *type;
+     char *valaddr;
+     CORE_ADDR address;
+     GDB_FILE *stream;
+     int format;
+     int deref_ref;
+     int recurse;
+     enum val_prettyprint pretty;
 {
-  register unsigned int i;
-  unsigned int things_printed = 0;
-  int in_quotes = 0;
-  int need_comma = 0;
-  extern int inspect_it;
+  struct type *real_type = check_typedef (type);
+  if (pretty == Val_pretty_default)
+    {
+      pretty = prettyprint_structs ? Val_prettyprint : Val_no_prettyprint;
+    }
+  
+  QUIT;
+
+  /* Ensure that the type is complete and not just a stub.  If the type is
+     only a stub and we can't find and substitute its complete type, then
+     print appropriate string and return.  */
 
-  if (length == 0)
+  if (TYPE_FLAGS (real_type) & TYPE_FLAG_STUB)
     {
-      fputs_filtered ("\"\"", stdout);
-      return;
+      fprintf_filtered (stream, "<incomplete type>");
+      gdb_flush (stream);
+      return (0);
     }
+  
+  return (LA_VAL_PRINT (type, valaddr, address, stream, format, deref_ref,
+                       recurse, pretty));
+}
+
+/* Print the value VAL in C-ish syntax on stream STREAM.
+   FORMAT is a format-letter, or 0 for print in natural format of data type.
+   If the object printed is a string pointer, returns
+   the number of string bytes printed.  */
 
-  for (i = 0; i < length && things_printed < print_max; ++i)
+int
+value_print (val, stream, format, pretty)
+     value_ptr val;
+     GDB_FILE *stream;
+     int format;
+     enum val_prettyprint pretty;
+{
+  if (val == 0)
+    {
+      printf_filtered ("<address of value unknown>");
+      return 0;
+    }
+  if (VALUE_OPTIMIZED_OUT (val))
     {
-      /* Position of the character we are examining
-        to see whether it is repeated.  */
-      unsigned int rep1;
-      /* Number of repetitions we have detected so far.  */
-      unsigned int reps;
+      printf_filtered ("<value optimized out>");
+      return 0;
+    }
+  return LA_VALUE_PRINT (val, stream, format, pretty);
+}
 
-      QUIT;
+/* Called by various <lang>_val_print routines to print
+   TYPE_CODE_INT's.  TYPE is the type.  VALADDR is the address of the
+   value.  STREAM is where to print the value.  */
+
+void
+val_print_type_code_int (type, valaddr, stream)
+     struct type *type;
+     char *valaddr;
+     GDB_FILE *stream;
+{
+  if (TYPE_LENGTH (type) > sizeof (LONGEST))
+    {
+      LONGEST val;
 
-      if (need_comma)
+      if (TYPE_UNSIGNED (type)
+         && extract_long_unsigned_integer (valaddr, TYPE_LENGTH (type),
+                                           &val))
        {
-         fputs_filtered (", ", stream);
-         need_comma = 0;
+         print_longest (stream, 'u', 0, val);
        }
-
-      rep1 = i + 1;
-      reps = 1;
-      while (rep1 < length && string[rep1] == string[i])
+      else
        {
-         ++rep1;
-         ++reps;
+         /* Signed, or we couldn't turn an unsigned value into a
+            LONGEST.  For signed values, one could assume two's
+            complement (a reasonable assumption, I think) and do
+            better than this.  */
+         print_hex_chars (stream, (unsigned char *) valaddr,
+                          TYPE_LENGTH (type));
        }
+    }
+  else
+    {
+#ifdef PRINT_TYPELESS_INTEGER
+      PRINT_TYPELESS_INTEGER (stream, type, unpack_long (type, valaddr));
+#else
+      print_longest (stream, TYPE_UNSIGNED (type) ? 'u' : 'd', 0,
+                    unpack_long (type, valaddr));
+#endif
+    }
+}
+
+/* Print a number according to FORMAT which is one of d,u,x,o,b,h,w,g.
+   The raison d'etre of this function is to consolidate printing of LONG_LONG's
+   into this one function.  Some platforms have long longs but don't have a
+   printf() that supports "ll" in the format string.  We handle these by seeing
+   if the number is actually a long, and if not we just bail out and print the
+   number in hex.  The format chars b,h,w,g are from
+   print_scalar_formatted().  If USE_LOCAL, format it according to the current
+   language (this should be used for most integers which GDB prints, the
+   exception is things like protocols where the format of the integer is
+   a protocol thing, not a user-visible thing).  */
 
-      if (reps > REPEAT_COUNT_THRESHOLD)
+void
+print_longest (stream, format, use_local, val_long)
+     GDB_FILE *stream;
+     int format;
+     int use_local;
+     LONGEST val_long;
+{
+#if defined (CC_HAS_LONG_LONG) && !defined (PRINTF_HAS_LONG_LONG)
+  long vtop, vbot;
+  unsigned int ui_max = UINT_MAX;
+  unsigned long long val_ulonglong;
+
+  /* Do shift in two operations so that if sizeof (long) == sizeof (LONGEST)
+     we can avoid warnings from picky compilers about shifts >= the size of
+     the shiftee in bits */
+  vtop = val_long >> (sizeof (long) * HOST_CHAR_BIT - 1);
+  vtop >>= 1;
+  vbot = (long) val_long;
+  val_ulonglong = (unsigned long long) val_long;
+  switch (format)
+    {
+    case 'd':
+      if (val_long < INT_MIN || val_long > INT_MAX)
        {
-         if (in_quotes)
+         if (sizeof (long long) > sizeof (long))
            {
-             if (inspect_it)
-               fputs_filtered ("\\\", ", stream);
-             else
-               fputs_filtered ("\", ", stream);
-             in_quotes = 0;
+             fprintf_filtered (stream, "0x%lx%08lx", vtop, vbot);
            }
-         fputs_filtered ("'", stream);
-         printchar (string[i], stream, '\'');
-         fprintf_filtered (stream, "' <repeats %u times>", reps);
-         i = rep1 - 1;
-         things_printed += REPEAT_COUNT_THRESHOLD;
-         need_comma = 1;
+         else
+           {
+             fprintf_filtered (stream, "%d", vbot);
+           }
+         return;
        }
-      else
+      break;
+    case 'x':
+      if (val_ulonglong > ui_max)
+       {
+         if (sizeof (long long) > sizeof (long))
+           {
+             fprintf_filtered (stream, "0x%lx%08lx", vtop, vbot);
+           }
+         else
+           {
+             fprintf_filtered (stream, "0x%lx", vbot);
+           }
+         return;
+       }
+      break;
+    case 'u':
+      if (val_ulonglong > ui_max)
        {
-         if (!in_quotes)
+         if (sizeof (long long) > sizeof (long))
+           {
+             fprintf_filtered (stream, "0x%lx%08lx", vtop, vbot);
+           }
+         else
            {
-             if (inspect_it)
-               fputs_filtered ("\\\"", stream);
-             else
-               fputs_filtered ("\"", stream);
-             in_quotes = 1;
+             fprintf_filtered (stream, "%lu", (unsigned long) vbot);
            }
-         printchar (string[i], stream, '"');
-         ++things_printed;
+         return;
        }
+      break;
+    }
+#endif
+
+#ifdef PRINTF_HAS_LONG_LONG
+  switch (format)
+    {
+    case 'd':
+      fprintf_filtered (stream,
+                       use_local ? local_decimal_format_custom ("ll")
+                                 : "%lld",
+                       val_long);
+      break;
+    case 'u':
+      fprintf_filtered (stream, "%llu", val_long);
+      break;
+    case 'x':
+      fprintf_filtered (stream,
+                       use_local ? local_hex_format_custom ("ll")
+                                 : "%llx",
+                       val_long);
+      break;
+    case 'o':
+      fprintf_filtered (stream,
+                       use_local ? local_octal_format_custom ("ll")
+                                 : "%llo",
+                       val_long);
+      break;
+    case 'b':
+      fprintf_filtered (stream, local_hex_format_custom ("02ll"), val_long);
+      break;
+    case 'h':
+      fprintf_filtered (stream, local_hex_format_custom ("04ll"), val_long);
+      break;
+    case 'w':
+      fprintf_filtered (stream, local_hex_format_custom ("08ll"), val_long);
+      break;
+    case 'g':
+      fprintf_filtered (stream, local_hex_format_custom ("016ll"), val_long);
+      break;
+    default:
+      abort ();
     }
+#else /* !PRINTF_HAS_LONG_LONG */
+  /* In the following it is important to coerce (val_long) to a long. It does
+     nothing if !LONG_LONG, but it will chop off the top half (which we know
+     we can ignore) if the host supports long longs.  */
 
-  /* Terminate the quotes if necessary.  */
-  if (in_quotes)
+  switch (format)
     {
-      if (inspect_it)
-       fputs_filtered ("\\\"", stream);
-      else
-       fputs_filtered ("\"", stream);
+    case 'd':
+      fprintf_filtered (stream,
+                       use_local ? local_decimal_format_custom ("l")
+                                 : "%ld",
+                       (long) val_long);
+      break;
+    case 'u':
+      fprintf_filtered (stream, "%lu", (unsigned long) val_long);
+      break;
+    case 'x':
+      fprintf_filtered (stream,
+                       use_local ? local_hex_format_custom ("l")
+                                 : "%lx",
+                       (long) val_long);
+      break;
+    case 'o':
+      fprintf_filtered (stream,
+                       use_local ? local_octal_format_custom ("l")
+                                 : "%lo",
+                       (long) val_long);
+      break;
+    case 'b':
+      fprintf_filtered (stream, local_hex_format_custom ("02l"),
+                       (long) val_long);
+      break;
+    case 'h':
+      fprintf_filtered (stream, local_hex_format_custom ("04l"),
+                       (long) val_long);
+      break;
+    case 'w':
+      fprintf_filtered (stream, local_hex_format_custom ("08l"),
+                       (long) val_long);
+      break;
+    case 'g':
+      fprintf_filtered (stream, local_hex_format_custom ("016l"),
+                       (long) val_long);
+      break;
+    default:
+      abort ();
     }
+#endif /* !PRINTF_HAS_LONG_LONG */
+}
+
+/* This used to be a macro, but I don't think it is called often enough
+   to merit such treatment.  */
+/* Convert a LONGEST to an int.  This is used in contexts (e.g. number of
+   arguments to a function, number in a value history, register number, etc.)
+   where the value must not be larger than can fit in an int.  */
+
+int
+longest_to_int (arg)
+     LONGEST arg;
+{
+
+  /* This check is in case a system header has botched the
+     definition of INT_MIN, like on BSDI.  */
+  if (sizeof (LONGEST) <= sizeof (int))
+    return arg;
 
-  if (force_ellipses || i < length)
-    fputs_filtered ("...", stream);
+  if (arg > INT_MAX)
+    error ("Value is larger than largest signed integer.");
+  if (arg < INT_MIN)
+    error ("Value is smaller than smallest signed integer.");
+
+  return arg;
 }
 
 /* Print a floating point value of type TYPE, pointed to in GDB by VALADDR,
@@ -162,9 +422,9 @@ void
 print_floating (valaddr, type, stream)
      char *valaddr;
      struct type *type;
-     FILE *stream;
+     GDB_FILE *stream;
 {
-  double doub;
+  DOUBLEST doub;
   int inv;
   unsigned len = TYPE_LENGTH (type);
   
@@ -178,43 +438,51 @@ print_floating (valaddr, type, stream)
      representation, but is not IEEE conforming.  */
 
   {
-    long low, high;
+    unsigned long low, high;
     /* Is the sign bit 0?  */
     int nonnegative;
     /* Is it is a NaN (i.e. the exponent is all ones and
        the fraction is nonzero)?  */
     int is_nan;
 
-    if (len == sizeof (float))
+    if (len == 4)
       {
-       /* It's single precision. */
-       bcopy (valaddr, &low, sizeof (low));
-       /* target -> host.  */
-       SWAP_TARGET_AND_HOST (&low, sizeof (float));
-       nonnegative = low >= 0;
+       /* It's single precision.  */
+       /* Assume that floating point byte order is the same as
+          integer byte order.  */
+       low = extract_unsigned_integer (valaddr, 4);
+       nonnegative = ((low & 0x80000000) == 0);
        is_nan = ((((low >> 23) & 0xFF) == 0xFF) 
                  && 0 != (low & 0x7FFFFF));
        low &= 0x7fffff;
        high = 0;
       }
-    else
+    else if (len == 8)
       {
        /* It's double precision.  Get the high and low words.  */
 
-#if TARGET_BYTE_ORDER == BIG_ENDIAN
-         bcopy (valaddr+4, &low,  sizeof (low));
-         bcopy (valaddr+0, &high, sizeof (high));
-#else
-         bcopy (valaddr+0, &low,  sizeof (low));
-         bcopy (valaddr+4, &high, sizeof (high));
-#endif
-         SWAP_TARGET_AND_HOST (&low, sizeof (low));
-         SWAP_TARGET_AND_HOST (&high, sizeof (high));
-         nonnegative = high >= 0;
-         is_nan = (((high >> 20) & 0x7ff) == 0x7ff
-                   && ! ((((high & 0xfffff) == 0)) && (low == 0)));
-         high &= 0xfffff;
-       }
+       /* Assume that floating point byte order is the same as
+          integer byte order.  */
+       if (TARGET_BYTE_ORDER == BIG_ENDIAN)
+         {
+           low = extract_unsigned_integer (valaddr + 4, 4);
+           high = extract_unsigned_integer (valaddr, 4);
+         }
+       else
+         {
+           low = extract_unsigned_integer (valaddr, 4);
+           high = extract_unsigned_integer (valaddr + 4, 4);
+         }
+       nonnegative = ((high & 0x80000000) == 0);
+       is_nan = (((high >> 20) & 0x7ff) == 0x7ff
+                 && ! ((((high & 0xfffff) == 0)) && (low == 0)));
+       high &= 0xfffff;
+      }
+    else
+      /* Extended.  We can't detect NaNs for extendeds yet.  Also note
+        that currently extendeds get nuked to double in
+        REGISTER_CONVERTIBLE.  */
+      is_nan = 0;
 
     if (is_nan)
       {
@@ -234,1575 +502,461 @@ print_floating (valaddr, type, stream)
 
   doub = unpack_double (type, valaddr, &inv);
   if (inv)
-    fprintf_filtered (stream, "<invalid float value>");
+    {
+      fprintf_filtered (stream, "<invalid float value>");
+      return;
+    }
+
+  if (len < sizeof (double))
+    fprintf_filtered (stream, "%.9g", (double) doub);
+  else if (len == sizeof (double))
+    fprintf_filtered (stream, "%.17g", (double) doub);
   else
-    fprintf_filtered (stream, len <= sizeof(float) ? "%.6g" : "%.17g", doub);
+#ifdef PRINTF_HAS_LONG_DOUBLE
+    fprintf_filtered (stream, "%.35Lg", doub);
+#else
+    /* This at least wins with values that are representable as doubles */
+    fprintf_filtered (stream, "%.17g", (double) doub);
+#endif
 }
 
 /* VALADDR points to an integer of LEN bytes.  Print it in hex on stream.  */
+
 static void
 print_hex_chars (stream, valaddr, len)
-     FILE *stream;
+     GDB_FILE *stream;
      unsigned char *valaddr;
      unsigned len;
 {
   unsigned char *p;
-  
-  fprintf_filtered (stream, "0x");
-#if TARGET_BYTE_ORDER == BIG_ENDIAN
-  for (p = valaddr;
-       p < valaddr + len;
-       p++)
-#else /* Little endian.  */
-  for (p = valaddr + len - 1;
-       p >= valaddr;
-       p--)
-#endif
-    {
-      fprintf_filtered (stream, "%02x", *p);
-    }
-}
-\f
-/* Print the value VAL in C-ish syntax on stream STREAM.
-   FORMAT is a format-letter, or 0 for print in natural format of data type.
-   If the object printed is a string pointer, returns
-   the number of string bytes printed.  */
-
-int
-value_print (val, stream, format, pretty)
-     value val;
-     FILE *stream;
-     char format;
-     enum val_prettyprint pretty;
-{
-  register unsigned int i, n, typelen;
 
-  if (val == 0)
-    {
-      printf_filtered ("<address of value unknown>");
-      return 0;
-    }
-  if (VALUE_OPTIMIZED_OUT (val))
-    {
-      printf_filtered ("<value optimized out>");
-      return 0;
-    }
-  
-  /* A "repeated" value really contains several values in a row.
-     They are made by the @ operator.
-     Print such values as if they were arrays.  */
+  /* FIXME: We should be not printing leading zeroes in most cases.  */
 
-  else if (VALUE_REPEATED (val))
+  fprintf_filtered (stream, local_hex_format_prefix ());
+  if (TARGET_BYTE_ORDER == BIG_ENDIAN)
     {
-      n = VALUE_REPETITIONS (val);
-      typelen = TYPE_LENGTH (VALUE_TYPE (val));
-      fprintf_filtered (stream, "{");
-      /* Print arrays of characters using string syntax.  */
-      if (typelen == 1 && TYPE_CODE (VALUE_TYPE (val)) == TYPE_CODE_INT
-         && format == 0)
-       print_string (stream, VALUE_CONTENTS (val), n, 0);
-      else
+      for (p = valaddr;
+          p < valaddr + len;
+          p++)
        {
-         unsigned int things_printed = 0;
-         
-         for (i = 0; i < n && things_printed < print_max; i++)
-           {
-             /* Position of the array element we are examining to see
-                whether it is repeated.  */
-             unsigned int rep1;
-             /* Number of repetitions we have detected so far.  */
-             unsigned int reps;
-
-             if (i != 0)
-               fprintf_filtered (stream, ", ");
-             wrap_here ("");
-
-             rep1 = i + 1;
-             reps = 1;
-             while (rep1 < n
-                    && !bcmp (VALUE_CONTENTS (val) + typelen * i,
-                              VALUE_CONTENTS (val) + typelen * rep1, typelen))
-               {
-                 ++reps;
-                 ++rep1;
-               }
-
-             if (reps > REPEAT_COUNT_THRESHOLD)
-               {
-                 val_print (VALUE_TYPE (val),
-                            VALUE_CONTENTS (val) + typelen * i,
-                            VALUE_ADDRESS (val) + typelen * i,
-                            stream, format, 1, 0, pretty);
-                 fprintf (stream, " <repeats %u times>", reps);
-                 i = rep1 - 1;
-                 things_printed += REPEAT_COUNT_THRESHOLD;
-               }
-             else
-               {
-                 val_print (VALUE_TYPE (val),
-                            VALUE_CONTENTS (val) + typelen * i,
-                            VALUE_ADDRESS (val) + typelen * i,
-                            stream, format, 1, 0, pretty);
-                 things_printed++;
-               }
-           }
-         if (i < n)
-           fprintf_filtered (stream, "...");
+         fprintf_filtered (stream, "%02x", *p);
        }
-      fprintf_filtered (stream, "}");
-      return n * typelen;
     }
   else
     {
-      /* If it is a pointer, indicate what it points to.
-
-        Print type also if it is a reference.
-
-         C++: if it is a member pointer, we will take care
-        of that when we print it.  */
-      if (TYPE_CODE (VALUE_TYPE (val)) == TYPE_CODE_PTR
-         || TYPE_CODE (VALUE_TYPE (val)) == TYPE_CODE_REF)
+      for (p = valaddr + len - 1;
+          p >= valaddr;
+          p--)
        {
-         /* Hack:  remove (char *) for char strings.  Their
-            type is indicated by the quoted string anyway. */
-          if (TYPE_CODE (VALUE_TYPE (val)) == TYPE_CODE_PTR
-           && TYPE_LENGTH   (TYPE_TARGET_TYPE (VALUE_TYPE (val)))
-                        == sizeof(char)
-           && TYPE_CODE     (TYPE_TARGET_TYPE (VALUE_TYPE (val)))
-                        == TYPE_CODE_INT
-           && !TYPE_UNSIGNED (TYPE_TARGET_TYPE (VALUE_TYPE (val))))
-           {
-               /* Print nothing */
-           }
-         else
-           {
-             fprintf_filtered (stream, "(");
-             type_print (VALUE_TYPE (val), "", stream, -1);
-             fprintf_filtered (stream, ") ");
-           }
+         fprintf_filtered (stream, "%02x", *p);
        }
-      return val_print (VALUE_TYPE (val), VALUE_CONTENTS (val),
-                       VALUE_ADDRESS (val), stream, format, 1, 0, pretty);
     }
+  fprintf_filtered (stream, local_hex_format_suffix ());
 }
 
-/* Return truth value for assertion that TYPE is of the type
-   "pointer to virtual function".  */
-static int
-is_vtbl_ptr_type(type)
-     struct type *type;
-{
-  char *typename = TYPE_NAME(type);
-  static const char vtbl_ptr_name[] =
-    { CPLUS_MARKER,'v','t','b','l','_','p','t','r','_','t','y','p','e' };
-
-  return (typename != NULL && !strcmp(typename, vtbl_ptr_name));
-}
-
-/* Return truth value for the assertion that TYPE is of the type
-   "pointer to virtual function table".  */
-static int
-is_vtbl_member(type)
-     struct type *type;
-{
-  if (TYPE_CODE (type) == TYPE_CODE_PTR
-      && TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_ARRAY
-      && TYPE_CODE (TYPE_TARGET_TYPE (TYPE_TARGET_TYPE (type))) == TYPE_CODE_STRUCT)
-    /* Virtual functions tables are full of pointers to virtual functions.  */
-    return is_vtbl_ptr_type (TYPE_TARGET_TYPE (TYPE_TARGET_TYPE (type)));
-  return 0;
-}
-\f
-/* Define a mess of print controls.  */
-
-int prettyprint;       /* Controls pretty printing of structures */
-int vtblprint;         /* Controls printing of vtbl's */
-int unionprint;                /* Controls printing of nested unions.  */
-int arrayprint;                /* Controls pretty printing of arrays.  */
-int addressprint;      /* Controls pretty printing of addresses.  */
-
-struct obstack dont_print_obstack;
+/*  Called by various <lang>_val_print routines to print elements of an
+    array in the form "<elem1>, <elem2>, <elem3>, ...".
 
-static void cplus_val_print ();
+    (FIXME?)  Assumes array element separator is a comma, which is correct
+    for all languages currently handled.
+    (FIXME?)  Some languages have a notation for repeated array elements,
+    perhaps we should try to use that notation when appropriate.
+    */
 
-/* Mutually recursive subroutines of cplus_val_print and val_print to print out
-   a structure's fields: val_print_fields and cplus_val_print.
-
-   TYPE, VALADDR, STREAM, RECURSE, and PRETTY have the
-   same meanings as in cplus_val_print and val_print.
-
-   DONT_PRINT is an array of baseclass types that we
-   should not print, or zero if called from top level.  */
-static void
-val_print_fields (type, valaddr, stream, format, recurse, pretty, dont_print)
+void
+val_print_array_elements (type, valaddr, address, stream, format, deref_ref,
+                         recurse, pretty, i)
      struct type *type;
      char *valaddr;
-     FILE *stream;
-     char format;
+     CORE_ADDR address;
+     GDB_FILE *stream;
+     int format;
+     int deref_ref;
      int recurse;
      enum val_prettyprint pretty;
-     struct type **dont_print;
+     unsigned int i;
 {
-  int i, len, n_baseclasses;
-
-  fprintf_filtered (stream, "{");
-  len = TYPE_NFIELDS (type);
-  n_baseclasses = TYPE_N_BASECLASSES (type);
+  unsigned int things_printed = 0;
+  unsigned len;
+  struct type *elttype;
+  unsigned eltlen;
+  /* Position of the array element we are examining to see
+     whether it is repeated.  */
+  unsigned int rep1;
+  /* Number of repetitions we have detected so far.  */
+  unsigned int reps;
+      
+  elttype = TYPE_TARGET_TYPE (type);
+  eltlen = TYPE_LENGTH (check_typedef (elttype));
+  len = TYPE_LENGTH (type) / eltlen;
 
-  /* Print out baseclasses such that we don't print
-     duplicates of virtual baseclasses.  */
-  if (n_baseclasses > 0)
-    cplus_val_print (type, valaddr, stream, format, recurse+1, pretty, dont_print);
+  annotate_array_section_begin (i, elttype);
 
-  if (!len && n_baseclasses == 1)
-    fprintf_filtered (stream, "<No data fields>");
-  else
+  for (; i < len && things_printed < print_max; i++)
     {
-      extern int inspect_it;
-      int fields_seen = 0;
-
-      for (i = n_baseclasses; i < len; i++)
+      if (i != 0)
        {
-         /* Check if static field */
-         if (TYPE_FIELD_STATIC (type, i))
-           continue;
-         if (fields_seen)
-           fprintf_filtered (stream, ", ");
-         else if (n_baseclasses > 0)
-           {
-             fprintf_filtered (stream, "\n");
-             print_spaces_filtered (2 + 2 * recurse, stream);
-             fputs_filtered ("members of ", stream);
-             fputs_filtered (type_name_no_tag (type), stream);
-             fputs_filtered (": ", stream);
-           }
-         fields_seen = 1;
-
-         if (pretty)
+         if (prettyprint_arrays)
            {
-             fprintf_filtered (stream, "\n");
+             fprintf_filtered (stream, ",\n");
              print_spaces_filtered (2 + 2 * recurse, stream);
            }
-         else 
-           {
-             wrap_here (n_spaces (2 + 2 * recurse));
-           }
-         if (inspect_it)
-           {
-             if (TYPE_CODE (TYPE_FIELD_TYPE (type, i)) == TYPE_CODE_PTR)
-               fputs_filtered ("\"( ptr \"", stream);
-             else
-               fputs_filtered ("\"( nodef \"", stream);
-             fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
-             fputs_filtered ("\" \"", stream);
-             fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
-             fputs_filtered ("\") \"", stream);
-           }
          else
            {
-             fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
-             fputs_filtered (" = ", stream);
+             fprintf_filtered (stream, ", ");
            }
-         if (TYPE_FIELD_PACKED (type, i))
-           {
-             LONGEST val;
-             char *valp = (char *) & val;
+       }
+      wrap_here (n_spaces (2 + 2 * recurse));
 
-             val = unpack_field_as_long (type, valaddr, i);
+      rep1 = i + 1;
+      reps = 1;
+      while ((rep1 < len) && 
+            !memcmp (valaddr + i * eltlen, valaddr + rep1 * eltlen, eltlen))
+       {
+         ++reps;
+         ++rep1;
+       }
 
-             /* Since we have moved the bitfield into a long,
-                if it is declared with a smaller type, we need to
-                offset its address *in gdb* to match the type we
-                are passing to val_print.  */
-#if HOST_BYTE_ORDER == BIG_ENDIAN
-             valp += sizeof val - TYPE_LENGTH (TYPE_FIELD_TYPE (type, i));
-#endif
-             val_print (TYPE_FIELD_TYPE (type, i), valp, 0,
-                        stream, format, 0, recurse + 1, pretty);
-           }
-         else
-           {
-             val_print (TYPE_FIELD_TYPE (type, i), 
-                        valaddr + TYPE_FIELD_BITPOS (type, i) / 8,
-                        0, stream, format, 0, recurse + 1, pretty);
-           }
+      if (reps > repeat_count_threshold)
+       {
+         val_print (elttype, valaddr + i * eltlen, 0, stream, format,
+                    deref_ref, recurse + 1, pretty);
+         annotate_elt_rep (reps);
+         fprintf_filtered (stream, " <repeats %u times>", reps);
+         annotate_elt_rep_end ();
+
+         i = rep1 - 1;
+         things_printed += repeat_count_threshold;
        }
-      if (pretty)
+      else
        {
-         fprintf_filtered (stream, "\n");
-         print_spaces_filtered (2 * recurse, stream);
+         val_print (elttype, valaddr + i * eltlen, 0, stream, format,
+                    deref_ref, recurse + 1, pretty);
+         annotate_elt ();
+         things_printed++;
        }
     }
-  fprintf_filtered (stream, "}");
+  annotate_array_section_end ();
+  if (i < len)
+    {
+      fprintf_filtered (stream, "...");
+    }
 }
 
-/* Special val_print routine to avoid printing multiple copies of virtual
-   baseclasses.  */
+/*  Print a string from the inferior, starting at ADDR and printing up to LEN
+    characters, to STREAM.  If LEN is zero, printing stops at the first null
+    byte, otherwise printing proceeds (including null bytes) until either
+    print_max or LEN characters have been printed, whichever is smaller. */
 
-static void
-cplus_val_print (type, valaddr, stream, format, recurse, pretty, dont_print)
-     struct type *type;
-     char *valaddr;
-     FILE *stream;
-     char format;
-     int recurse;
-     enum val_prettyprint pretty;
-     struct type **dont_print;
-{
-  struct obstack tmp_obstack;
-  struct type **last_dont_print
-    = (struct type **)obstack_next_free (&dont_print_obstack);
-  int i, n_baseclasses = TYPE_N_BASECLASSES (type);
+/* FIXME: All callers supply LEN of zero.  Supplying a non-zero LEN is
+   pointless, this routine just then becomes a convoluted version of
+   target_read_memory_partial.  Removing all the LEN stuff would simplify
+   this routine enormously.
 
-  if (dont_print == 0)
-    {
-      /* If we're at top level, carve out a completely fresh
-        chunk of the obstack and use that until this particular
-        invocation returns.  */
-      tmp_obstack = dont_print_obstack;
-      /* Bump up the high-water mark.  Now alpha is omega.  */
-      obstack_finish (&dont_print_obstack);
-    }
+   FIXME: Use target_read_string.  */
 
-  for (i = 0; i < n_baseclasses; i++)
-    {
-      char *baddr;
+int
+val_print_string (addr, len, stream)
+    CORE_ADDR addr;
+    unsigned int len;
+    GDB_FILE *stream;
+{
+  int force_ellipsis = 0;      /* Force ellipsis to be printed if nonzero. */
+  int errcode;                 /* Errno returned from bad reads. */
+  unsigned int fetchlimit;     /* Maximum number of bytes to fetch. */
+  unsigned int nfetch;         /* Bytes to fetch / bytes fetched. */
+  unsigned int chunksize;      /* Size of each fetch, in bytes. */
+  unsigned int bufsize;                /* Size of current fetch buffer. */
+  char *buffer = NULL;         /* Dynamically growable fetch buffer. */
+  char *bufptr;                        /* Pointer to next available byte in buffer. */
+  char *limit;                 /* First location past end of fetch buffer. */
+  struct cleanup *old_chain = NULL; /* Top of the old cleanup chain. */
+  char peekchar;               /* Place into which we can read one char. */
+
+  /* First we need to figure out the limit on the number of characters we are
+     going to attempt to fetch and print.  This is actually pretty simple.  If
+     LEN is nonzero, then the limit is the minimum of LEN and print_max.  If
+     LEN is zero, then the limit is print_max.  This is true regardless of
+     whether print_max is zero, UINT_MAX (unlimited), or something in between,
+     because finding the null byte (or available memory) is what actually
+     limits the fetch. */
+
+  fetchlimit = (len == 0 ? print_max : min (len, print_max));
+
+  /* Now decide how large of chunks to try to read in one operation.  This
+     is also pretty simple.  If LEN is nonzero, then we want fetchlimit bytes,
+     so we might as well read them all in one operation.  If LEN is zero, we
+     are looking for a null terminator to end the fetching, so we might as
+     well read in blocks that are large enough to be efficient, but not so
+     large as to be slow if fetchlimit happens to be large.  So we choose the
+     minimum of 8 and fetchlimit.  We used to use 200 instead of 8 but
+     200 is way too big for remote debugging over a serial line.  */
+
+  chunksize = (len == 0 ? min (8, fetchlimit) : fetchlimit);
+
+  /* Loop until we either have all the characters to print, or we encounter
+     some error, such as bumping into the end of the address space. */
+
+  bufsize = 0;
+  do {
+    QUIT;
+    /* Figure out how much to fetch this time, and grow the buffer to fit. */
+    nfetch = min (chunksize, fetchlimit - bufsize);
+    bufsize += nfetch;
+    if (buffer == NULL)
+      {
+       buffer = (char *) xmalloc (bufsize);
+       bufptr = buffer;
+      }
+    else
+      {
+       discard_cleanups (old_chain);
+       buffer = (char *) xrealloc (buffer, bufsize);
+       bufptr = buffer + bufsize - nfetch;
+      }
+    old_chain = make_cleanup (free, buffer);
 
-      if (BASETYPE_VIA_VIRTUAL (type, i))
-       {
-         struct type **first_dont_print
-           = (struct type **)obstack_base (&dont_print_obstack);
+    /* Read as much as we can. */
+    nfetch = target_read_memory_partial (addr, bufptr, nfetch, &errcode);
+    if (len != 0)
+      {
+       addr += nfetch;
+       bufptr += nfetch;
+      }
+    else
+      {
+       /* Scan this chunk for the null byte that terminates the string
+          to print.  If found, we don't need to fetch any more.  Note
+          that bufptr is explicitly left pointing at the next character
+          after the null byte, or at the next character after the end of
+          the buffer. */
+       limit = bufptr + nfetch;
+       while (bufptr < limit)
+         {
+           ++addr;
+           ++bufptr;
+           if (bufptr[-1] == '\0')
+             {
+               /* We don't care about any error which happened after
+                  the NULL terminator.  */
+               errcode = 0;
+               break;
+             }
+         }
+      }
+  } while (errcode == 0                                        /* no error */
+          && bufsize < fetchlimit                      /* no overrun */
+          && !(len == 0 && *(bufptr - 1) == '\0'));    /* no null term */
 
-         int j = (struct type **)obstack_next_free (&dont_print_obstack)
-           - first_dont_print;
+  /* bufptr and addr now point immediately beyond the last byte which we
+     consider part of the string (including a '\0' which ends the string).  */
 
-         while (--j >= 0)
-           if (TYPE_BASECLASS (type, i) == first_dont_print[j])
-             goto flush_it;
+  /* We now have either successfully filled the buffer to fetchlimit, or
+     terminated early due to an error or finding a null byte when LEN is
+     zero.  */
 
-         obstack_ptr_grow (&dont_print_obstack, TYPE_BASECLASS (type, i));
+  if (len == 0 && bufptr > buffer && *(bufptr - 1) != '\0')
+    {
+      /* We didn't find a null terminator we were looking for.  Attempt
+        to peek at the next character.  If not successful, or it is not
+        a null byte, then force ellipsis to be printed.  */
+      if (target_read_memory (addr, &peekchar, 1) != 0 || peekchar != '\0')
+       {
+         force_ellipsis = 1;
        }
-
-      baddr = baseclass_addr (type, i, valaddr, 0);
-      if (baddr == 0)
-       error ("could not find virtual baseclass `%s'\n",
-              type_name_no_tag (TYPE_BASECLASS (type, i)));
-
-      fprintf_filtered (stream, "\n");
-      if (pretty)
-       print_spaces_filtered (2 + 2 * recurse, stream);
-      fputs_filtered ("<", stream);
-      fputs_filtered (type_name_no_tag (TYPE_BASECLASS (type, i)), stream);
-      fputs_filtered ("> = ", stream);
-      val_print_fields (TYPE_BASECLASS (type, i), baddr, stream, format,
-                       recurse, pretty,
-                       (struct type **)obstack_base (&dont_print_obstack));
-    flush_it:
-      ;
     }
-
-  if (dont_print == 0)
+  else if ((len != 0 && errcode != 0) || (len > bufptr - buffer))
     {
-      /* Free the space used to deal with the printing
-        of this type from top level.  */
-      obstack_free (&dont_print_obstack, last_dont_print);
-      /* Reset watermark so that we can continue protecting
-        ourselves from whatever we were protecting ourselves.  */
-      dont_print_obstack = tmp_obstack;
+      /* Getting an error when we have a requested length, or fetching less
+        than the number of characters actually requested, always make us
+        print ellipsis. */
+      force_ellipsis = 1;
     }
-}
 
-/* Print data of type TYPE located at VALADDR (within GDB),
-   which came from the inferior at address ADDRESS,
-   onto stdio stream STREAM according to FORMAT
-   (a letter or 0 for natural format).  The data at VALADDR
-   is in target byte order.
-
-   If the data are a string pointer, returns the number of
-   sting characters printed.
-
-   if DEREF_REF is nonzero, then dereference references,
-   otherwise just print them like pointers.
-
-   The PRETTY parameter controls prettyprinting.  */
-
-int
-val_print (type, valaddr, address, stream, format,
-          deref_ref, recurse, pretty)
-     struct type *type;
-     char *valaddr;
-     CORE_ADDR address;
-     FILE *stream;
-     char format;
-     int deref_ref;
-     int recurse;
-     enum val_prettyprint pretty;
-{
-  register unsigned int i;
-  unsigned len;
-  struct type *elttype;
-  unsigned eltlen;
-  LONGEST val;
-  unsigned char c;
-
-  if (pretty == Val_pretty_default)
-    {
-      pretty = prettyprint ? Val_prettyprint : Val_no_prettyprint;
-    }
-  
   QUIT;
 
-  check_stub_type (type);
-  
-  if (TYPE_FLAGS (type) & TYPE_FLAG_STUB)
+  /* If we get an error before fetching anything, don't print a string.
+     But if we fetch something and then get an error, print the string
+     and then the error message.  */
+  if (errcode == 0 || bufptr > buffer)
     {
-      fprintf_filtered (stream, "<unknown struct>");
-      fflush (stream);
-      return 0;
-    }
-  
-  switch (TYPE_CODE (type))
-    {
-    case TYPE_CODE_ARRAY:
-      if (TYPE_LENGTH (type) >= 0
-         && TYPE_LENGTH (TYPE_TARGET_TYPE (type)) > 0)
-       {
-         elttype = TYPE_TARGET_TYPE (type);
-         eltlen = TYPE_LENGTH (elttype);
-         len = TYPE_LENGTH (type) / eltlen;
-         if (arrayprint)
-           print_spaces_filtered (2 + 2 * recurse, stream);
-         fprintf_filtered (stream, "{");
-         /* For an array of chars, print with string syntax.  */
-         if (eltlen == 1 && TYPE_CODE (elttype) == TYPE_CODE_INT
-             && (format == 0 || format == 's') )
-           print_string (stream, valaddr, len, 0);
-         else
-           {
-             unsigned int things_printed = 0;
-             
-             for (i = 0; i < len && things_printed < print_max; i++)
-               {
-                 /* Position of the array element we are examining to see
-                    whether it is repeated.  */
-                 unsigned int rep1;
-                 /* Number of repetitions we have detected so far.  */
-                 unsigned int reps;
-                 
-                 if (i != 0)
-                   if (arrayprint)
-                     {
-                       fprintf_filtered (stream, ",\n");
-                       print_spaces_filtered (2 + 2 * recurse, stream);
-                     }
-                   else
-                     fprintf_filtered (stream, ", ");
-                   wrap_here (n_spaces (2 + 2 * recurse));
-                 
-                 rep1 = i + 1;
-                 reps = 1;
-                 while (rep1 < len
-                        && !bcmp (valaddr + i * eltlen,
-                                  valaddr + rep1 * eltlen, eltlen))
-                   {
-                     ++reps;
-                     ++rep1;
-                   }
-
-                 if (reps > REPEAT_COUNT_THRESHOLD)
-                   {
-                     val_print (elttype, valaddr + i * eltlen,
-                                0, stream, format, deref_ref,
-                                recurse + 1, pretty);
-                     fprintf_filtered (stream, " <repeats %u times>", reps);
-                     i = rep1 - 1;
-                     things_printed += REPEAT_COUNT_THRESHOLD;
-                   }
-                 else
-                   {
-                     val_print (elttype, valaddr + i * eltlen,
-                                0, stream, format, deref_ref,
-                                recurse + 1, pretty);
-                     things_printed++;
-                   }
-               }
-             if (i < len)
-               fprintf_filtered (stream, "...");
-           }
-         fprintf_filtered (stream, "}");
-         break;
-       }
-      /* Array of unspecified length: treat like pointer to first elt.  */
-      valaddr = (char *) &address;
-
-    case TYPE_CODE_PTR:
-      if (format && format != 's')
-       {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
-       }
-      if (TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_METHOD)
-       {
-         struct type *domain = TYPE_DOMAIN_TYPE (TYPE_TARGET_TYPE (type));
-         struct fn_field *f;
-         int j, len2;
-         char *kind = "";
-
-         val = unpack_long (builtin_type_int, valaddr);
-         if (val < 128)
-           {
-             len = TYPE_NFN_FIELDS (domain);
-             for (i = 0; i < len; i++)
-               {
-                 f = TYPE_FN_FIELDLIST1 (domain, i);
-                 len2 = TYPE_FN_FIELDLIST_LENGTH (domain, i);
-
-                 for (j = 0; j < len2; j++)
-                   {
-                     QUIT;
-                     if (TYPE_FN_FIELD_VOFFSET (f, j) == val)
-                       {
-                         kind = "virtual";
-                         goto common;
-                       }
-                   }
-               }
-           }
-         else
-           {
-             struct symbol *sym = find_pc_function ((CORE_ADDR) val);
-             if (sym == 0)
-               error ("invalid pointer to member function");
-             len = TYPE_NFN_FIELDS (domain);
-             for (i = 0; i < len; i++)
-               {
-                 f = TYPE_FN_FIELDLIST1 (domain, i);
-                 len2 = TYPE_FN_FIELDLIST_LENGTH (domain, i);
-
-                 for (j = 0; j < len2; j++)
-                   {
-                     QUIT;
-                     if (!strcmp (SYMBOL_NAME (sym), TYPE_FN_FIELD_PHYSNAME (f, j)))
-                       goto common;
-                   }
-               }
-           }
-       common:
-         if (i < len)
-           {
-             fprintf_filtered (stream, "&");
-             type_print_varspec_prefix (TYPE_FN_FIELD_TYPE (f, j), stream, 0, 0);
-             fprintf (stream, kind);
-             if (TYPE_FN_FIELD_PHYSNAME (f, j)[0] == '_'
-                 && TYPE_FN_FIELD_PHYSNAME (f, j)[1] == CPLUS_MARKER)
-               type_print_method_args
-                 (TYPE_FN_FIELD_ARGS (f, j) + 1, "~",
-                  TYPE_FN_FIELDLIST_NAME (domain, i), 0, stream);
-             else
-               type_print_method_args
-                 (TYPE_FN_FIELD_ARGS (f, j), "",
-                  TYPE_FN_FIELDLIST_NAME (domain, i), 0, stream);
-             break;
-           }
-         fprintf_filtered (stream, "(");
-         type_print (type, "", stream, -1);
-         fprintf_filtered (stream, ") %d", (int) val >> 3);
-       }
-      else if (TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_MEMBER)
-       {
-         struct type *domain = TYPE_DOMAIN_TYPE (TYPE_TARGET_TYPE (type));
-
-         /* VAL is a byte offset into the structure type DOMAIN.
-            Find the name of the field for that offset and
-            print it.  */
-         int extra = 0;
-         int bits = 0;
-         len = TYPE_NFIELDS (domain);
-         /* @@ Make VAL into bit offset */
-         val = unpack_long (builtin_type_int, valaddr) << 3;
-         for (i = TYPE_N_BASECLASSES (domain); i < len; i++)
-           {
-             int bitpos = TYPE_FIELD_BITPOS (domain, i);
-             QUIT;
-             if (val == bitpos)
-               break;
-             if (val < bitpos && i != 0)
-               {
-                 /* Somehow pointing into a field.  */
-                 i -= 1;
-                 extra = (val - TYPE_FIELD_BITPOS (domain, i));
-                 if (extra & 0x3)
-                   bits = 1;
-                 else
-                   extra >>= 3;
-                 break;
-               }
-           }
-         if (i < len)
-           {
-             fprintf_filtered (stream, "&");
-             type_print_base (domain, stream, 0, 0);
-             fprintf_filtered (stream, "::");
-             fputs_filtered (TYPE_FIELD_NAME (domain, i), stream);
-             if (extra)
-               fprintf_filtered (stream, " + %d bytes", extra);
-             if (bits)
-               fprintf_filtered (stream, " (offset in bits)");
-             break;
-           }
-         fprintf_filtered (stream, "%d", val >> 3);
-       }
-      else
-       {
-         CORE_ADDR addr = (CORE_ADDR) unpack_long (type, valaddr);
-         elttype = TYPE_TARGET_TYPE (type);
-
-         if (TYPE_CODE (elttype) == TYPE_CODE_FUNC)
-           {
-             /* Try to print what function it points to.  */
-             print_address_demangle (addr, stream, demangle);
-             /* Return value is irrelevant except for string pointers.  */
-             return 0;
-           }
-
-         if (addressprint && format != 's')
-           fprintf_filtered (stream, "0x%x", addr);
-
-         /* For a pointer to char or unsigned char,
-            also print the string pointed to, unless pointer is null.  */
-         i = 0;                /* Number of characters printed.  */
-         if (TYPE_LENGTH (elttype) == 1 
-             && TYPE_CODE (elttype) == TYPE_CODE_INT
-             && (format == 0 || format == 's')
-             && addr != 0
-             /* If print_max is UINT_MAX, the alloca below will fail.
-                In that case don't try to print the string.  */
-             && print_max < UINT_MAX)
-           {
-             int first_addr_err = 0;
-             int errcode = 0;
-             
-             /* Get first character.  */
-             errcode = target_read_memory (addr, (char *)&c, 1);
-             if (errcode != 0)
-               {
-                 /* First address out of bounds.  */
-                 first_addr_err = 1;
-               }
-             else
-               {
-                 /* A real string.  */
-                 char *string = (char *) alloca (print_max);
-
-                 /* If the loop ends by us hitting print_max characters,
-                    we need to have elipses at the end.  */
-                 int force_ellipses = 1;
-
-                 /* This loop always fetches print_max characters, even
-                    though print_string might want to print more or fewer
-                    (with repeated characters).  This is so that
-                    we don't spend forever fetching if we print
-                    a long string consisting of the same character
-                    repeated.  Also so we can do it all in one memory
-                    operation, which is faster.  However, this will be
-                    slower if print_max is set high, e.g. if you set
-                    print_max to 1000, not only will it take a long
-                    time to fetch short strings, but if you are near
-                    the end of the address space, it might not work.
-                    FIXME.  */
-                 QUIT;
-                 errcode = target_read_memory (addr, string, print_max);
-                 if (errcode != 0)
-                     force_ellipses = 0;
-                 else 
-                   for (i = 0; i < print_max; i++)
-                     if (string[i] == '\0')
-                       {
-                         force_ellipses = 0;
-                         break;
-                       }
-                 QUIT;
-
-                 if (addressprint)
-                   fputs_filtered (" ", stream);
-                 print_string (stream, string, i, force_ellipses);
-               }
-
-             if (errcode != 0)
-               {
-                 if (errcode == EIO)
-                   {
-                     fprintf_filtered (stream,
-                                       (" <Address 0x%x out of bounds>"
-                                        + first_addr_err),
-                                       addr + i);
-                   }
-                 else
-                   {
-                     if (errcode >= sys_nerr || errcode < 0)
-                       error ("Error reading memory address 0x%x: unknown error (%d).",
-                              addr + i, errcode);
-                     else
-                       error ("Error reading memory address 0x%x: %s.",
-                              addr + i, sys_errlist[errcode]);
-                   }
-               }
-
-             fflush (stream);
-           }
-         else /* print vtbl's nicely */
-         if (is_vtbl_member(type))
-           {
-             CORE_ADDR vt_address = (CORE_ADDR) unpack_long (type, valaddr);
-
-             int vt_index = find_pc_misc_function (vt_address);
-             if (vt_index >= 0
-                 && vt_address == misc_function_vector[vt_index].address)
-               {
-                 fputs_filtered (" <", stream);
-                 fputs_demangled (misc_function_vector[vt_index].name,
-                                  stream, 1);
-                 fputs_filtered (">", stream);
-               }
-             if (vtblprint)
-               {
-                 value val;
-
-                 val = value_at (TYPE_TARGET_TYPE (type), vt_address);
-                 val_print (VALUE_TYPE (val), VALUE_CONTENTS (val),
-                            VALUE_ADDRESS (val), stream, format,
-                            deref_ref, recurse + 1, pretty);
-                 if (pretty)
-                   {
-                     fprintf_filtered (stream, "\n");
-                     print_spaces_filtered (2 + 2 * recurse, stream);
-                   }
-               }
-             }
-
-         /* Return number of characters printed, plus one for the
-            terminating null if we have "reached the end".  */
-         return i + (print_max && i != print_max);
-       }
-      break;
-
-    case TYPE_CODE_MEMBER:
-      error ("not implemented: member type in val_print");
-      break;
-
-    case TYPE_CODE_REF:
       if (addressprint)
-        {
-         fprintf_filtered (stream, "@0x%lx",
-                           unpack_long (builtin_type_int, valaddr));
-         if (deref_ref)
-           fputs_filtered (": ", stream);
-        }
-      /* De-reference the reference.  */
-      if (deref_ref)
-       {
-         if (TYPE_CODE (TYPE_TARGET_TYPE (type)) != TYPE_CODE_UNDEF)
-           {
-             value val = value_at (TYPE_TARGET_TYPE (type),
-                                   (CORE_ADDR) unpack_long (builtin_type_int,
-                                                            valaddr));
-             val_print (VALUE_TYPE (val), VALUE_CONTENTS (val),
-                        VALUE_ADDRESS (val), stream, format,
-                        deref_ref, recurse + 1, pretty);
-           }
-         else
-           fputs_filtered ("???", stream);
-       }
-      break;
-
-    case TYPE_CODE_UNION:
-      if (recurse && !unionprint)
        {
-         fprintf_filtered (stream, "{...}");
-         break;
-       }
-      /* Fall through.  */
-    case TYPE_CODE_STRUCT:
-      if (vtblprint && is_vtbl_ptr_type(type))
-       {
-          /* Print the unmangled name if desired.  */
-         print_address_demangle(*((int *) (valaddr +   /* FIXME bytesex */
-             TYPE_FIELD_BITPOS (type, VTBL_FNADDR_OFFSET) / 8)),
-             stream, demangle);
-         break;
+         fputs_filtered (" ", stream);
        }
-      val_print_fields (type, valaddr, stream, format, recurse, pretty, 0);
-      break;
+      LA_PRINT_STRING (stream, buffer, bufptr - buffer, force_ellipsis);
+    }
 
-    case TYPE_CODE_ENUM:
-      if (format)
-       {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
-       }
-      len = TYPE_NFIELDS (type);
-      val = unpack_long (builtin_type_int, valaddr);
-      for (i = 0; i < len; i++)
+  if (errcode != 0)
+    {
+      if (errcode == EIO)
        {
-         QUIT;
-         if (val == TYPE_FIELD_BITPOS (type, i))
-           break;
+         fprintf_filtered (stream, " <Address ");
+         print_address_numeric (addr, 1, stream);
+         fprintf_filtered (stream, " out of bounds>");
        }
-      if (i < len)
-       fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
       else
-       fprintf_filtered (stream, "%d", (int) val);
-      break;
-
-    case TYPE_CODE_FUNC:
-      if (format)
        {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
+         fprintf_filtered (stream, " <Error reading address ");
+         print_address_numeric (addr, 1, stream);
+         fprintf_filtered (stream, ": %s>", safe_strerror (errcode));
        }
-      fprintf_filtered (stream, "{");
-      type_print (type, "", stream, -1);
-      fprintf_filtered (stream, "} ");
-      if (addressprint)
-       fprintf_filtered (stream, "0x%x", address);
-      break;
-
-    case TYPE_CODE_INT:
-      if (format || output_format)
-       {
-         print_scalar_formatted (valaddr, type,
-                                 format? format: output_format,
-                                 0, stream);
-         break;
-       }
-      if (TYPE_LENGTH (type) > sizeof (LONGEST))
-       {
-         if (TYPE_UNSIGNED (type))
-           {
-             /* First figure out whether the number in fact has zeros
-                in all its bytes more significant than least significant
-                sizeof (LONGEST) ones.  */
-             char *p;
-             /* Pointer to first (i.e. lowest address) nonzero character.  */
-             char *first_addr;
-             unsigned len = TYPE_LENGTH (type);
-
-#if TARGET_BYTE_ORDER == BIG_ENDIAN
-             for (p = valaddr;
-                  len > sizeof (LONGEST)
-                  && p < valaddr + TYPE_LENGTH (type);
-                  p++)
-#else /* Little endian.  */
-             first_addr = valaddr;
-             for (p = valaddr + TYPE_LENGTH (type);
-                  len > sizeof (LONGEST) && p >= valaddr;
-                  p--)
-#endif /* Little endian.  */
-               {
-                 if (*p == 0)
-                   len--;
-                 else
-                   break;
-               }
-#if TARGET_BYTE_ORDER == BIG_ENDIAN
-             first_addr = p;
-#endif
-             
-             if (len <= sizeof (LONGEST))
-               {
-                 /* We can print it in decimal.  */
-                 fprintf_filtered
-                   (stream, 
-#if defined (LONG_LONG)
-                    "%llu",
-#else
-                    "%lu",
-#endif
-                    unpack_long (BUILTIN_TYPE_LONGEST, first_addr));
-               }
-             else
-               {
-                 /* It is big, so print it in hex.  */
-                 print_hex_chars (stream, (unsigned char *)first_addr, len);
-               }
-           }
-         else
-           {
-             /* Signed.  One could assume two's complement (a reasonable
-                assumption, I think) and do better than this.  */
-             print_hex_chars (stream, (unsigned char *)valaddr,
-                              TYPE_LENGTH (type));
-           }
-         break;
-       }
-#ifdef PRINT_TYPELESS_INTEGER
-      PRINT_TYPELESS_INTEGER (stream, type, unpack_long (type, valaddr));
-#else
-#ifndef LONG_LONG
-      fprintf_filtered (stream,
-                       TYPE_UNSIGNED (type) ? "%u" : "%d",
-                       unpack_long (type, valaddr));
-#else
-      fprintf_filtered (stream,
-                       TYPE_UNSIGNED (type) ? "%llu" : "%lld",
-                       unpack_long (type, valaddr));
-#endif
-#endif
-                       
-      if (TYPE_LENGTH (type) == 1)
-       {
-         fprintf_filtered (stream, " '");
-         printchar ((unsigned char) unpack_long (type, valaddr), 
-                    stream, '\'');
-         fprintf_filtered (stream, "'");
-       }
-      break;
-
-    case TYPE_CODE_FLT:
-      if (format)
-       print_scalar_formatted (valaddr, type, format, 0, stream);
-      else
-       print_floating (valaddr, type, stream);
-      break;
-
-    case TYPE_CODE_VOID:
-      fprintf_filtered (stream, "void");
-      break;
-
-    case TYPE_CODE_UNDEF:
-      /* This happens (without TYPE_FLAG_STUB set) on systems which don't use
-        dbx xrefs (NO_DBX_XREFS in gcc) if a file has a "struct foo *bar"
-        and no complete type for struct foo in that file.  */
-      fprintf_filtered (stream, "<unknown struct>");
-      break;
-
-    case TYPE_CODE_ERROR:
-      fprintf_filtered (stream, "?");
-      break;
-
-    default:
-      error ("Invalid type code in symbol table.");
     }
-  fflush (stream);
-  return 0;
-}
-\f
-/* Print a description of a type TYPE
-   in the form of a declaration of a variable named VARSTRING.
-   (VARSTRING is demangled if necessary.)
-   Output goes to STREAM (via stdio).
-   If SHOW is positive, we show the contents of the outermost level
-   of structure even if there is a type name that could be used instead.
-   If SHOW is negative, we never show the details of elements' types.  */
-
-void
-type_print (type, varstring, stream, show)
-     struct type *type;
-     char *varstring;
-     FILE *stream;
-     int show;
-{
-  type_print_1 (type, varstring, stream, show, 0);
+  gdb_flush (stream);
+  do_cleanups (old_chain);
+  return (bufptr - buffer);
 }
 
-/* LEVEL is the depth to indent lines by.  */
+\f
+/* Validate an input or output radix setting, and make sure the user
+   knows what they really did here.  Radix setting is confusing, e.g.
+   setting the input radix to "10" never changes it!  */
 
-void
-type_print_1 (type, varstring, stream, show, level)
-     struct type *type;
-     char *varstring;
-     FILE *stream;
-     int show;
-     int level;
+/* ARGSUSED */
+static void
+set_input_radix (args, from_tty, c)
+     char *args;
+     int from_tty;
+     struct cmd_list_element *c;
 {
-  register enum type_code code;
-  type_print_base (type, stream, show, level);
-  code = TYPE_CODE (type);
-  if ((varstring && *varstring)
-      ||
-      /* 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_MEMBER
-       || code == TYPE_CODE_REF)))
-    fprintf_filtered (stream, " ");
-  type_print_varspec_prefix (type, stream, show, 0);
-  fputs_demangled (varstring, stream, -1);     /* Print demangled name
-                                                  without arguments */
-  type_print_varspec_suffix (type, stream, show, 0);
+  set_input_radix_1 (from_tty, *(unsigned *)c->var);
 }
 
-/* Print the method arguments ARGS to the file STREAM.  */
+/* ARGSUSED */
 static void
-type_print_method_args (args, prefix, varstring, staticp, stream)
-     struct type **args;
-     char *prefix, *varstring;
-     int staticp;
-     FILE *stream;
+set_input_radix_1 (from_tty, radix)
+     int from_tty;
+     unsigned radix;
 {
-  int i;
-
-  fputs_filtered (" ", stream);
-  fputs_demangled (prefix, stream, 1);
-  fputs_demangled (varstring, stream, 1);
-  fputs_filtered (" (", stream);
-  if (args && args[!staticp] && args[!staticp]->code != TYPE_CODE_VOID)
+  /* We don't currently disallow any input radix except 0 or 1, which don't
+     make any mathematical sense.  In theory, we can deal with any input
+     radix greater than 1, even if we don't have unique digits for every
+     value from 0 to radix-1, but in practice we lose on large radix values.
+     We should either fix the lossage or restrict the radix range more.
+     (FIXME). */
+
+  if (radix < 2)
     {
-      i = !staticp;            /* skip the class variable */
-      while (1)
-       {
-         type_print (args[i++], "", stream, 0);
-         if (!args[i]) 
-           {
-             fprintf_filtered (stream, " ...");
-             break;
-           }
-         else if (args[i]->code != TYPE_CODE_VOID)
-           {
-             fprintf_filtered (stream, ", ");
-           }
-         else break;
-       }
+      error ("Nonsense input radix ``decimal %u''; input radix unchanged.",
+            radix);
     }
-  fprintf_filtered (stream, ")");
-}
-  
-/* If TYPE is a derived type, then print out derivation
-   information.  Print out all layers of the type heirarchy
-   until we encounter one with multiple inheritance.
-   At that point, print out that ply, and return.  */
-static void
-type_print_derivation_info (stream, type)
-     FILE *stream;
-     struct type *type;
-{
-  char *name;
-  int i, n_baseclasses = TYPE_N_BASECLASSES (type);
-  struct type *basetype = 0;
-
-  while (type && n_baseclasses > 0)
+  input_radix = radix;
+  if (from_tty)
     {
-      /* Not actually sure about this one -- Bryan. */
-      check_stub_type (type);
-      
-      fprintf_filtered (stream, ": ");
-      for (i = 0; ;)
-       {
-         basetype = TYPE_BASECLASS (type, i);
-         if (name = type_name_no_tag (basetype))
-           {
-             fprintf_filtered (stream, "%s%s ",
-                      BASETYPE_VIA_PUBLIC(type, i) ? "public" : "private",
-                      BASETYPE_VIA_VIRTUAL(type, i) ? " virtual" : "");
-             fputs_filtered (name, stream);
-           }
-         i++;
-         if (i >= n_baseclasses)
-             break;
-         fprintf_filtered (stream, ", ");
-       }
-
-      fprintf_filtered (stream, " ");
-      if (n_baseclasses != 1)
-       break;
-      n_baseclasses = TYPE_N_BASECLASSES (basetype);
-      type = basetype;
+      printf_filtered ("Input radix now set to decimal %u, hex %x, octal %o.\n",
+                      radix, radix, radix);
     }
 }
 
-/* Print any asterisks or open-parentheses needed before the
-   variable name (to describe its type).
-
-   On outermost call, pass 0 for PASSED_A_PTR.
-   On outermost call, SHOW > 0 means should ignore
-   any typename for TYPE and show its details.
-   SHOW is always zero on recursive calls.  */
-
+/* ARGSUSED */
 static void
-type_print_varspec_prefix (type, stream, show, passed_a_ptr)
-     struct type *type;
-     FILE *stream;
-     int show;
-     int passed_a_ptr;
+set_output_radix (args, from_tty, c)
+     char *args;
+     int from_tty;
+     struct cmd_list_element *c;
 {
-  if (type == 0)
-    return;
-
-  if (TYPE_NAME (type) && show <= 0)
-    return;
-
-  QUIT;
-
-  switch (TYPE_CODE (type))
-    {
-    case TYPE_CODE_PTR:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      fprintf_filtered (stream, "*");
-      break;
-
-    case TYPE_CODE_MEMBER:
-      if (passed_a_ptr)
-       fprintf_filtered (stream, "(");
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      fprintf_filtered (stream, " ");
-      type_print_base (TYPE_DOMAIN_TYPE (type), stream, 0,
-                      passed_a_ptr);
-      fprintf_filtered (stream, "::");
-      break;
-
-    case TYPE_CODE_METHOD:
-      if (passed_a_ptr)
-       fprintf (stream, "(");
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      fprintf_filtered (stream, " ");
-      type_print_base (TYPE_DOMAIN_TYPE (type), stream, 0,
-                      passed_a_ptr);
-      fprintf_filtered (stream, "::");
-      break;
-
-    case TYPE_CODE_REF:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      fprintf_filtered (stream, "&");
-      break;
-
-    case TYPE_CODE_FUNC:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      if (passed_a_ptr)
-       fprintf_filtered (stream, "(");
-      break;
-
-    case TYPE_CODE_ARRAY:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      if (passed_a_ptr)
-       fprintf_filtered (stream, "(");
-
-    case TYPE_CODE_UNDEF:
-    case TYPE_CODE_STRUCT:
-    case TYPE_CODE_UNION:
-    case TYPE_CODE_ENUM:
-    case TYPE_CODE_INT:
-    case TYPE_CODE_FLT:
-    case TYPE_CODE_VOID:
-    case TYPE_CODE_ERROR:
-      /* These types need no prefix.  They are listed here so that
-        gcc -Wall will reveal any types that haven't been handled.  */
-      break;
-    }
+  set_output_radix_1 (from_tty, *(unsigned *)c->var);
 }
 
-/* Print any array sizes, function arguments or close parentheses
-   needed after the variable name (to describe its type).
-   Args work like type_print_varspec_prefix.  */
-
 static void
-type_print_varspec_suffix (type, stream, show, passed_a_ptr)
-     struct type *type;
-     FILE *stream;
-     int show;
-     int passed_a_ptr;
+set_output_radix_1 (from_tty, radix)
+     int from_tty;
+     unsigned radix;
 {
-  if (type == 0)
-    return;
-
-  if (TYPE_NAME (type) && show <= 0)
-    return;
-
-  QUIT;
-
-  switch (TYPE_CODE (type))
+  /* Validate the radix and disallow ones that we aren't prepared to
+     handle correctly, leaving the radix unchanged. */
+  switch (radix)
     {
-    case TYPE_CODE_ARRAY:
-      if (passed_a_ptr)
-       fprintf_filtered (stream, ")");
-      
-      fprintf_filtered (stream, "[");
-      if (TYPE_LENGTH (type) >= 0
-         && TYPE_LENGTH (TYPE_TARGET_TYPE (type)) > 0)
-       fprintf_filtered (stream, "%d",
-                         (TYPE_LENGTH (type)
-                          / TYPE_LENGTH (TYPE_TARGET_TYPE (type))));
-      fprintf_filtered (stream, "]");
-      
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      break;
-
-    case TYPE_CODE_MEMBER:
-      if (passed_a_ptr)
-       fprintf_filtered (stream, ")");
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 0);
-      break;
-
-    case TYPE_CODE_METHOD:
-      if (passed_a_ptr)
-       fprintf_filtered (stream, ")");
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 0);
-      if (passed_a_ptr)
-       {
-         int i;
-         struct type **args = TYPE_ARG_TYPES (type);
-
-         fprintf_filtered (stream, "(");
-         if (args[1] == 0)
-           fprintf_filtered (stream, "...");
-         else for (i = 1; args[i] != 0 && args[i]->code != TYPE_CODE_VOID; i++)
-           {
-             type_print_1 (args[i], "", stream, -1, 0);
-             if (args[i+1] == 0)
-               fprintf_filtered (stream, "...");
-             else if (args[i+1]->code != TYPE_CODE_VOID)
-               fprintf_filtered (stream, ",");
-           }
-         fprintf_filtered (stream, ")");
-       }
-      break;
-
-    case TYPE_CODE_PTR:
-    case TYPE_CODE_REF:
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 1);
+    case 16:
+      output_format = 'x';             /* hex */
       break;
-
-    case TYPE_CODE_FUNC:
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0,
-                                passed_a_ptr);
-      if (passed_a_ptr)
-       fprintf_filtered (stream, ")");
-      fprintf_filtered (stream, "()");
+    case 10:
+      output_format = 0;               /* decimal */
       break;
-
-    case TYPE_CODE_UNDEF:
-    case TYPE_CODE_STRUCT:
-    case TYPE_CODE_UNION:
-    case TYPE_CODE_ENUM:
-    case TYPE_CODE_INT:
-    case TYPE_CODE_FLT:
-    case TYPE_CODE_VOID:
-    case TYPE_CODE_ERROR:
-      /* These types do not need a suffix.  They are listed so that
-        gcc -Wall will report types that may not have been considered.  */
+    case 8:
+      output_format = 'o';             /* octal */
       break;
+    default:
+      error ("Unsupported output radix ``decimal %u''; output radix unchanged.",
+            radix);
+    }
+  output_radix = radix;
+  if (from_tty)
+    {
+      printf_filtered ("Output radix now set to decimal %u, hex %x, octal %o.\n",
+                      radix, radix, radix);
     }
 }
 
-/* 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 nonzero means don't print this type as just its name;
-   show its real definition even if it has a name.
-   SHOW zero means print just typename or struct tag if there is one
-   SHOW negative means abbreviate structure elements.
-   SHOW is decremented for printing of structure elements.
+/* Set both the input and output radix at once.  Try to set the output radix
+   first, since it has the most restrictive range.  An radix that is valid as
+   an output radix is also valid as an input radix.
 
-   LEVEL is the depth to indent by.
-   We increase it for some recursive calls.  */
+   It may be useful to have an unusual input radix.  If the user wishes to
+   set an input radix that is not valid as an output radix, he needs to use
+   the 'set input-radix' command. */
 
 static void
-type_print_base (type, stream, show, level)
-     struct type *type;
-     FILE *stream;
-     int show;
-     int level;
+set_radix (arg, from_tty)
+     char *arg;
+     int from_tty;
 {
-  char *name;
-  register int i;
-  register int len;
-  register int lastval;
+  unsigned radix;
 
-  QUIT;
-
-  if (type == 0)
+  radix = (arg == NULL) ? 10 : parse_and_eval_address (arg);
+  set_output_radix_1 (0, radix);
+  set_input_radix_1 (0, radix);
+  if (from_tty)
     {
-      fprintf_filtered (stream, "type unknown");
-      return;
+      printf_filtered ("Input and output radices now set to decimal %u, hex %x, octal %o.\n",
+                      radix, radix, radix);
     }
+}
 
-  if (TYPE_NAME (type) && show <= 0)
-    {
-      fputs_filtered (TYPE_NAME (type), stream);
-      return;
-    }
+/* Show both the input and output radices. */
 
-  switch (TYPE_CODE (type))
+/*ARGSUSED*/
+static void
+show_radix (arg, from_tty)
+     char *arg;
+     int from_tty;
+{
+  if (from_tty)
     {
-    case TYPE_CODE_ARRAY:
-    case TYPE_CODE_PTR:
-    case TYPE_CODE_MEMBER:
-    case TYPE_CODE_REF:
-    case TYPE_CODE_FUNC:
-    case TYPE_CODE_METHOD:
-      type_print_base (TYPE_TARGET_TYPE (type), stream, show, level);
-      break;
-
-    case TYPE_CODE_STRUCT:
-      fprintf_filtered (stream, "struct ");
-      goto struct_union;
-
-    case TYPE_CODE_UNION:
-      fprintf_filtered (stream, "union ");
-    struct_union:
-      if (name = type_name_no_tag (type))
-       {
-         fputs_filtered (name, stream);
-         fputs_filtered (" ", stream);
-       }
-      if (show < 0)
-       fprintf_filtered (stream, "{...}");
-      else
-       {
-         check_stub_type (type);
-         
-         type_print_derivation_info (stream, type);
-         
-         fprintf_filtered (stream, "{");
-         len = TYPE_NFIELDS (type);
-         if (len)
-           fprintf_filtered (stream, "\n");
-         else
-           {
-             if (TYPE_FLAGS (type) & TYPE_FLAG_STUB)
-               fprintf_filtered (stream, "<incomplete type>\n");
-             else
-               fprintf_filtered (stream, "<no data fields>\n");
-           }
-
-         /* If there is a base class for this type,
-            do not print the field that it occupies.  */
-         for (i = TYPE_N_BASECLASSES (type); i < len; i++)
-           {
-             QUIT;
-             /* Don't print out virtual function table.  */
-             if ((TYPE_FIELD_NAME (type, i))[5] == CPLUS_MARKER &&
-                 !strncmp (TYPE_FIELD_NAME (type, i), "_vptr", 5))
-               continue;
-
-             print_spaces_filtered (level + 4, stream);
-             if (TYPE_FIELD_STATIC (type, i))
-               {
-                 fprintf_filtered (stream, "static ");
-               }
-             type_print_1 (TYPE_FIELD_TYPE (type, i),
-                           TYPE_FIELD_NAME (type, i),
-                           stream, show - 1, level + 4);
-             if (!TYPE_FIELD_STATIC (type, i)
-                 && TYPE_FIELD_PACKED (type, i))
-               {
-                 /* It is a bitfield.  This code does not attempt
-                    to look at the bitpos and reconstruct filler,
-                    unnamed fields.  This would lead to misleading
-                    results if the compiler does not put out fields
-                    for such things (I don't know what it does).  */
-                 fprintf_filtered (stream, " : %d",
-                                   TYPE_FIELD_BITSIZE (type, i));
-               }
-             fprintf_filtered (stream, ";\n");
-           }
-
-         /* C++: print out the methods */
-         len = TYPE_NFN_FIELDS (type);
-         if (len) fprintf_filtered (stream, "\n");
-         for (i = 0; i < len; i++)
-           {
-             struct fn_field *f = TYPE_FN_FIELDLIST1 (type, i);
-             int j, len2 = TYPE_FN_FIELDLIST_LENGTH (type, i);
-
-             for (j = 0; j < len2; j++)
-               {
-                 QUIT;
-                 print_spaces_filtered (level + 4, stream);
-                 if (TYPE_FN_FIELD_VIRTUAL_P (f, j))
-                   fprintf_filtered (stream, "virtual ");
-                 else if (TYPE_FN_FIELD_STATIC_P (f, j))
-                   fprintf_filtered (stream, "static ");
-                 type_print (TYPE_TARGET_TYPE (TYPE_FN_FIELD_TYPE (f, j)), "", stream, 0);
-                 if (TYPE_FLAGS (TYPE_FN_FIELD_TYPE (f, j)) & TYPE_FLAG_STUB)
-                   {
-                     /* Build something we can demangle.  */
-                     char *strchr (), *gdb_mangle_typename ();
-                     char *inner_name = gdb_mangle_typename (type);
-                     char *mangled_name
-                       = (char *)xmalloc (strlen (TYPE_FN_FIELDLIST_NAME (type, i))
-                                         + strlen (inner_name)
-                                         + strlen (TYPE_FN_FIELD_PHYSNAME (f, j))
-                                         + 1);
-                     char *demangled_name, *cplus_demangle ();
-                     strcpy (mangled_name, TYPE_FN_FIELDLIST_NAME (type, i));
-                     strcat (mangled_name, inner_name);
-                     strcat (mangled_name, TYPE_FN_FIELD_PHYSNAME (f, j));
-                     demangled_name = cplus_demangle (mangled_name, 1);
-                     if (demangled_name == 0)
-                       fprintf_filtered (stream, " <badly mangled name %s>",
-                           mangled_name);
-                     else 
-                       {
-                         fprintf_filtered (stream, " %s",
-                             strchr (demangled_name, ':') + 2);
-                         free (demangled_name);
-                       }
-                     free (mangled_name);
-                   }
-                 else if (TYPE_FN_FIELD_PHYSNAME (f, j)[0] == '_'
-                       && TYPE_FN_FIELD_PHYSNAME (f, j)[1] == CPLUS_MARKER)
-                   type_print_method_args
-                     (TYPE_FN_FIELD_ARGS (f, j) + 1, "~",
-                      TYPE_FN_FIELDLIST_NAME (type, i), 0, stream);
-                 else
-                   type_print_method_args
-                     (TYPE_FN_FIELD_ARGS (f, j), "",
-                      TYPE_FN_FIELDLIST_NAME (type, i),
-                      TYPE_FN_FIELD_STATIC_P (f, j), stream);
-
-                 fprintf_filtered (stream, ";\n");
-               }
-           }
-
-         print_spaces_filtered (level, stream);
-         fprintf_filtered (stream, "}");
-       }
-      break;
-
-    case TYPE_CODE_ENUM:
-      fprintf_filtered (stream, "enum ");
-      if (name = type_name_no_tag (type))
+      if (input_radix == output_radix)
        {
-         fputs_filtered (name, stream);
-         fputs_filtered (" ", stream);
+         printf_filtered ("Input and output radices set to decimal %u, hex %x, octal %o.\n",
+                          input_radix, input_radix, input_radix);
        }
-      if (show < 0)
-       fprintf_filtered (stream, "{...}");
       else
        {
-         fprintf_filtered (stream, "{");
-         len = TYPE_NFIELDS (type);
-         lastval = 0;
-         for (i = 0; i < len; i++)
-           {
-             QUIT;
-             if (i) fprintf_filtered (stream, ", ");
-             fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
-             if (lastval != TYPE_FIELD_BITPOS (type, i))
-               {
-                 fprintf_filtered (stream, " = %d", TYPE_FIELD_BITPOS (type, i));
-                 lastval = TYPE_FIELD_BITPOS (type, i);
-               }
-             lastval++;
-           }
-         fprintf_filtered (stream, "}");
+         printf_filtered ("Input radix set to decimal %u, hex %x, octal %o.\n",
+                          input_radix, input_radix, input_radix);
+         printf_filtered ("Output radix set to decimal %u, hex %x, octal %o.\n",
+                          output_radix, output_radix, output_radix);
        }
-      break;
-
-    case TYPE_CODE_INT:
-      if (TYPE_LENGTH (type) > sizeof (LONGEST))
-       {
-         fprintf_filtered (stream, "<%d bit integer>",
-                           TYPE_LENGTH (type) * TARGET_CHAR_BIT);
-       }
-      else
-       {
-         if (TYPE_UNSIGNED (type))
-           name = unsigned_type_table[TYPE_LENGTH (type)];
-         else
-           name = signed_type_table[TYPE_LENGTH (type)];
-       }
-      fputs_filtered (name, stream);
-      break;
-
-    case TYPE_CODE_FLT:
-      name = float_type_table[TYPE_LENGTH (type)];
-      fputs_filtered (name, stream);
-      break;
-
-    case TYPE_CODE_VOID:
-      fprintf_filtered (stream, "void");
-      break;
-
-    case 0:
-      fprintf_filtered (stream, "struct unknown");
-      break;
-
-    case TYPE_CODE_ERROR:
-      fprintf_filtered (stream, "<unknown type>");
-      break;
-
-    default:
-      error ("Invalid type code in symbol table.");
     }
 }
-\f
-/* Validate an input or output radix setting, and make sure the user
-   knows what they really did here.  Radix setting is confusing, e.g.
-   setting the input radix to "10" never changes it!  */
 
+\f
+/*ARGSUSED*/
 static void
-set_input_radix (args, from_tty, c)
-     char *args;
+set_print (arg, from_tty)
+     char *arg;
      int from_tty;
-     struct cmd_list_element *c;
 {
-  unsigned radix = *(unsigned *)c->var;
-
-  if (from_tty)
-    printf_filtered ("Input radix set to decimal %d, hex %x, octal %o\n",
-       radix, radix, radix);
+  printf_unfiltered (
+"\"set print\" must be followed by the name of a print subcommand.\n");
+  help_list (setprintlist, "set print ", -1, gdb_stdout);
 }
 
+/*ARGSUSED*/
 static void
-set_output_radix (args, from_tty, c)
+show_print (args, from_tty)
      char *args;
      int from_tty;
-     struct cmd_list_element *c;
 {
-  unsigned radix = *(unsigned *)c->var;
-
-  if (from_tty)
-    printf_filtered ("Output radix set to decimal %d, hex %x, octal %o\n",
-       radix, radix, radix);
-
-  /* FIXME, we really should be able to validate the setting BEFORE
-     it takes effect.  */
-  switch (radix)
-    {
-    case 16:
-      output_format = 'x';
-      break;
-    case 10:
-      output_format = 0;
-      break;
-    case 8:
-      output_format = 'o';             /* octal */
-      break;
-    default:
-      output_format = 0;
-      error ("Unsupported radix ``decimal %d''; using decimal output",
-             radix);
-    }
-}
-
-/* Both at once */
-static void
-set_radix (arg, from_tty, c)
-     char *arg;
-     int from_tty;
-     struct cmd_list_element *c;
-{
-  unsigned radix = *(unsigned *)c->var;
-
-  if (from_tty)
-    printf_filtered ("Radix set to decimal %d, hex %x, octal %o\n",
-       radix, radix, radix);
-
-  input_radix = radix;
-  output_radix = radix;
-
-  set_output_radix (arg, 0, c);
+  cmd_show_list (showprintlist, from_tty, "");
 }
 \f
 void
@@ -1810,106 +964,100 @@ _initialize_valprint ()
 {
   struct cmd_list_element *c;
 
+  add_prefix_cmd ("print", no_class, set_print,
+                 "Generic command for setting how things print.",
+                 &setprintlist, "set print ", 0, &setlist);
+  add_alias_cmd ("p", "print", no_class, 1, &setlist); 
+  /* prefer set print to set prompt */ 
+  add_alias_cmd ("pr", "print", no_class, 1, &setlist);
+
+  add_prefix_cmd ("print", no_class, show_print,
+                 "Generic command for showing print settings.",
+                 &showprintlist, "show print ", 0, &showlist);
+  add_alias_cmd ("p", "print", no_class, 1, &showlist); 
+  add_alias_cmd ("pr", "print", no_class, 1, &showlist); 
+
   add_show_from_set
-    (add_set_cmd ("array-max", class_vars, var_uinteger, (char *)&print_max,
+    (add_set_cmd ("elements", no_class, var_uinteger, (char *)&print_max,
                  "Set limit on string chars or array elements to print.\n\
-\"set array-max 0\" causes there to be no limit.",
-                 &setlist),
-     &showlist);
+\"set print elements 0\" causes there to be no limit.",
+                 &setprintlist),
+     &showprintlist);
 
   add_show_from_set
-    (add_set_cmd ("prettyprint", class_support, var_boolean, (char *)&prettyprint,
-                 "Set prettyprinting of structures.",
-                 &setlist),
-     &showlist);
+    (add_set_cmd ("null-stop", no_class, var_boolean,
+                 (char *)&stop_print_at_null,
+                 "Set printing of char arrays to stop at first null char.",
+                 &setprintlist),
+     &showprintlist);
+
+  add_show_from_set
+    (add_set_cmd ("repeats", no_class, var_uinteger,
+                 (char *)&repeat_count_threshold,
+                 "Set threshold for repeated print elements.\n\
+\"set print repeats 0\" causes all elements to be individually printed.",
+                 &setprintlist),
+     &showprintlist);
 
-  add_alias_cmd ("pp", "prettyprint", class_support, 1, &setlist);
+  add_show_from_set
+    (add_set_cmd ("pretty", class_support, var_boolean,
+                 (char *)&prettyprint_structs,
+                 "Set prettyprinting of structures.",
+                 &setprintlist),
+     &showprintlist);
 
   add_show_from_set
-    (add_set_cmd ("unionprint", class_support, var_boolean, (char *)&unionprint,
+    (add_set_cmd ("union", class_support, var_boolean, (char *)&unionprint,
                  "Set printing of unions interior to structures.",
-                 &setlist),
-     &showlist);
+                 &setprintlist),
+     &showprintlist);
   
   add_show_from_set
-    (add_set_cmd ("vtblprint", class_support, var_boolean, (char *)&vtblprint,
-                 "Set printing of C++ virtual function tables.",
-                 &setlist),
-     &showlist);
-
-  add_show_from_set
-    (add_set_cmd ("arrayprint", class_support, var_boolean, (char *)&arrayprint,
+    (add_set_cmd ("array", class_support, var_boolean,
+                 (char *)&prettyprint_arrays,
                  "Set prettyprinting of arrays.",
-                 &setlist),
-     &showlist);
+                 &setprintlist),
+     &showprintlist);
 
   add_show_from_set
-    (add_set_cmd ("addressprint", class_support, var_boolean, (char *)&addressprint,
+    (add_set_cmd ("address", class_support, var_boolean, (char *)&addressprint,
                  "Set printing of addresses.",
-                 &setlist),
-     &showlist);
+                 &setprintlist),
+     &showprintlist);
 
-#if 0
-  /* The "show radix" cmd isn't good enough to show two separate values.
-     The rest of the code works, but the show part is confusing, so don't
-     let them be set separately 'til we work out "show".  */
   c = add_set_cmd ("input-radix", class_support, var_uinteger,
                   (char *)&input_radix,
                  "Set default input radix for entering numbers.",
                  &setlist);
   add_show_from_set (c, &showlist);
-  c->function = set_input_radix;
+  c->function.sfunc = set_input_radix;
 
   c = add_set_cmd ("output-radix", class_support, var_uinteger,
                   (char *)&output_radix,
                  "Set default output radix for printing of values.",
                  &setlist);
   add_show_from_set (c, &showlist);
-  c->function = set_output_radix;
-#endif 
-
-  c = add_set_cmd ("radix", class_support, var_uinteger,
-                  (char *)&output_radix,
-                 "Set default input and output number radix.",
-                 &setlist);
-  add_show_from_set (c, &showlist);
-  c->function = set_radix;
+  c->function.sfunc = set_output_radix;
+
+  /* The "set radix" and "show radix" commands are special in that they are
+     like normal set and show commands but allow two normally independent
+     variables to be either set or shown with a single command.  So the
+     usual add_set_cmd() and add_show_from_set() commands aren't really
+     appropriate. */
+  add_cmd ("radix", class_support, set_radix,
+          "Set default input and output number radices.\n\
+Use 'set input-radix' or 'set output-radix' to independently set each.\n\
+Without an argument, sets both radices back to the default value of 10.",
+          &setlist);
+  add_cmd ("radix", class_support, show_radix,
+          "Show the default input and output number radices.\n\
+Use 'show input-radix' or 'show output-radix' to independently show each.",
+          &showlist);
 
   /* Give people the defaults which they are used to.  */
-  prettyprint = 0;
+  prettyprint_structs = 0;
+  prettyprint_arrays = 0;
   unionprint = 1;
-  vtblprint = 0;
-  arrayprint = 0;
   addressprint = 1;
-
-  print_max = 200;
-
-  unsigned_type_table
-    = (char **) xmalloc ((1 + sizeof (unsigned LONGEST)) * sizeof (char *));
-  bzero (unsigned_type_table, (1 + sizeof (unsigned LONGEST)));
-  unsigned_type_table[sizeof (unsigned char)] = "unsigned char";
-  unsigned_type_table[sizeof (unsigned short)] = "unsigned short";
-  unsigned_type_table[sizeof (unsigned long)] = "unsigned long";
-  unsigned_type_table[sizeof (unsigned int)] = "unsigned int";
-#ifdef LONG_LONG
-  unsigned_type_table[sizeof (unsigned long long)] = "unsigned long long";
-#endif
-
-  signed_type_table
-    = (char **) xmalloc ((1 + sizeof (LONGEST)) * sizeof (char *));
-  bzero (signed_type_table, (1 + sizeof (LONGEST)));
-  signed_type_table[sizeof (char)] = "char";
-  signed_type_table[sizeof (short)] = "short";
-  signed_type_table[sizeof (long)] = "long";
-  signed_type_table[sizeof (int)] = "int";
-#ifdef LONG_LONG
-  signed_type_table[sizeof (long long)] = "long long";
-#endif
-
-  float_type_table
-    = (char **) xmalloc ((1 + sizeof (double)) * sizeof (char *));
-  bzero (float_type_table, (1 + sizeof (double)));
-  float_type_table[sizeof (float)] = "float";
-  float_type_table[sizeof (double)] = "double";
-  obstack_begin (&dont_print_obstack, 32 * sizeof (struct type *));
+  print_max = PRINT_MAX_DEFAULT;
 }
This page took 0.050145 seconds and 4 git commands to generate.