Mon Dec 21 12:40:10 1992 Ian Lance Taylor (ian@cygnus.com)
[binutils-gdb.git] / gdb / valprint.c
index 8087ef8248cd4be6e03dab3396512f068639bcf7..b2f62fd5c359b67bf38159910a184019668d070a 100644 (file)
-/* Print values for GNU debugger gdb.
-   Copyright (C) 1986, 1988 Free Software Foundation, Inc.
-
-GDB is distributed in the hope that it will be useful, but WITHOUT ANY
-WARRANTY.  No author or distributor accepts responsibility to anyone
-for the consequences of using it or for whether it serves any
-particular purpose or works at all, unless he says so in writing.
-Refer to the GDB General Public License for full details.
-
-Everyone is granted permission to copy, modify and redistribute GDB,
-but only under the conditions described in the GDB General Public
-License.  A copy of this license is supposed to have been given to you
-along with GDB so you can know your rights and responsibilities.  It
-should be in a file named COPYING.  Among other things, the copyright
-notice and this notice must be preserved on all copies.
-
-In other words, go ahead and share GDB, but don't try to stop
-anyone else from sharing it farther.  Help stamp out software hoarding!
-*/
-
-#include <stdio.h>
+/* Print values for GDB, the GNU debugger.
+   Copyright 1986, 1988, 1989, 1991 Free Software Foundation, Inc.
+
+This file is part of GDB.
+
+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 2 of the License, or
+(at your option) any later version.
+
+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 this program; if not, write to the Free Software
+Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
+
 #include "defs.h"
-#include "initialize.h"
-#include "param.h"
+#include <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 <errno.h>
+
+/* Prototypes for local functions */
+
+static void
+print_hex_chars PARAMS ((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, struct cmd_list_element *));
+
+static void
+set_output_radix PARAMS ((char *, int, struct cmd_list_element *));
+
+static void
+value_print_array_elements PARAMS ((value, FILE *, int, enum val_prettyprint));
 
 /* Maximum number of chars to print for a string pointer value
-   or vector contents.  */
+   or vector contents, or UINT_MAX for no limit.  */
+
+unsigned int print_max;
+
+/* Default input and output radixes, and output format letter.  */
+
+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;
 
-static int print_max;
+int prettyprint_structs;       /* Controls pretty printing of structures */
+int prettyprint_arrays;                /* Controls pretty printing of arrays.  */
 
-static void type_print_varspec_suffix ();
-static void type_print_varspec_prefix ();
-static void type_print_base ();
+/* If nonzero, causes unions inside structures or other unions to be
+   printed. */
 
-START_FILE
+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 */
 
-char **unsigned_type_table;
-char **signed_type_table;
-char **float_type_table;
 \f
+/* 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;
+     int format;
+     int deref_ref;
+     int recurse;
+     enum val_prettyprint pretty;
+{
+  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.  Typical types that my be stubs
+     are structs, unions, and C++ methods. */
+
+  check_stub_type (type);
+  if (TYPE_FLAGS (type) & TYPE_FLAG_STUB)
+    {
+      fprintf_filtered (stream, "<incomplete type>");
+      fflush (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.  */
 
-value_print (val, stream, format)
+int
+value_print (val, stream, format, pretty)
      value val;
      FILE *stream;
-     char format;
+     int format;
+     enum val_prettyprint pretty;
 {
-  register int i, n, typelen;
+  register unsigned int 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.
@@ -60,606 +164,568 @@ value_print (val, stream, format)
     {
       n = VALUE_REPETITIONS (val);
       typelen = TYPE_LENGTH (VALUE_TYPE (val));
-      fputc ('{', stream);
+      fprintf_filtered (stream, "{");
       /* Print arrays of characters using string syntax.  */
       if (typelen == 1 && TYPE_CODE (VALUE_TYPE (val)) == TYPE_CODE_INT
          && format == 0)
-       {
-         fputc ('"', stream);
-         for (i = 0; i < n && i < print_max; i++)
-           {
-             QUIT;
-             printchar (VALUE_CONTENTS (val)[i], stream, '"');
-           }
-         if (i < n)
-           fprintf (stream, "...");
-         fputc ('"', stream);
-       }
+       LA_PRINT_STRING (stream, VALUE_CONTENTS (val), n, 0);
       else
        {
-         for (i = 0; i < n && i < print_max; i++)
-           {
-             if (i)
-               fprintf (stream, ", ");
-             val_print (VALUE_TYPE (val), VALUE_CONTENTS (val) + typelen * i,
-                        VALUE_ADDRESS (val) + typelen * i, stream, format);
-           }
-         if (i < n)
-           fprintf (stream, "...");
+         value_print_array_elements (val, stream, format, pretty);
        }
-      fputc ('}', stream);
+      fprintf_filtered (stream, "}");
+      return (n * typelen);
     }
   else
     {
-      /* A simple (nonrepeated) value */
-      /* If it is a pointer, indicate what it points to.  */
-      if (TYPE_CODE (VALUE_TYPE (val)) == TYPE_CODE_PTR)
+      struct type *type = VALUE_TYPE (val);
+
+      /* 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 (type) == TYPE_CODE_PTR ||
+         TYPE_CODE (type) == TYPE_CODE_REF)
        {
-         fprintf (stream, "(");
-         type_print (VALUE_TYPE (val), "", stream, -1);
-         fprintf (stream, ") ");
+         /* Hack:  remove (char *) for char strings.  Their
+            type is indicated by the quoted string anyway. */
+          if (TYPE_CODE (type) == TYPE_CODE_PTR &&
+             TYPE_LENGTH (TYPE_TARGET_TYPE (type)) == sizeof(char) &&
+             TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_INT &&
+             !TYPE_UNSIGNED (TYPE_TARGET_TYPE (type)))
+           {
+               /* Print nothing */
+           }
+         else
+           {
+             fprintf_filtered (stream, "(");
+             type_print (type, "", stream, -1);
+             fprintf_filtered (stream, ") ");
+           }
        }
-      return val_print (VALUE_TYPE (val), VALUE_CONTENTS (val),
-                       VALUE_ADDRESS (val), stream, format);
+      return (val_print (type, VALUE_CONTENTS (val),
+                        VALUE_ADDRESS (val), stream, format, 1, 0, pretty));
     }
 }
-\f
-/* 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).
 
-   If the data are a string pointer, returns the number of
-   sting characters printed.  */
+/*  Called by various <lang>_val_print routines to print TYPE_CODE_INT's */
 
-int
-val_print (type, valaddr, address, stream, format)
+void
+val_print_type_code_int (type, valaddr, stream)
      struct type *type;
      char *valaddr;
-     CORE_ADDR address;
      FILE *stream;
-     char format;
 {
-  register int i;
-  int len;
-  struct type *elttype;
-  int eltlen;
-  int val;
-  unsigned char c;
-
-  QUIT;
+  char *p;
+  /* Pointer to first (i.e. lowest address) nonzero character.  */
+  char *first_addr;
+  unsigned int len;
 
-  switch (TYPE_CODE (type))
+  if (TYPE_LENGTH (type) > sizeof (LONGEST))
     {
-    case TYPE_CODE_ARRAY:
-      if (TYPE_LENGTH (type) >= 0)
+      if (TYPE_UNSIGNED (type))
        {
-         elttype = TYPE_TARGET_TYPE (type);
-         eltlen = TYPE_LENGTH (elttype);
-         len = TYPE_LENGTH (type) / eltlen;
-         fprintf (stream, "{");
-         /* For an array of chars, print with string syntax.  */
-         if (eltlen == 1 && TYPE_CODE (elttype) == TYPE_CODE_INT
-             && format == 0)
+         /* First figure out whether the number in fact has zeros
+            in all its bytes more significant than least significant
+            sizeof (LONGEST) ones.  */
+         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.  */
            {
-             fputc ('"', stream);
-             for (i = 0; i < len && i < print_max; i++)
+             if (*p == 0)
                {
-                 QUIT;
-                 printchar (valaddr[i], stream, '"');
+                 len--;
                }
-             if (i < len)
-               fprintf (stream, "...");
-             fputc ('"', stream);
-           }
-         else
-           {
-             for (i = 0; i < len && i < print_max; i++)
+             else
                {
-                 if (i) fprintf (stream, ", ");
-                 val_print (elttype, valaddr + i * eltlen,
-                            0, stream, format);
+                 break;
                }
-             if (i < len)
-               fprintf (stream, "...");
            }
-         fprintf (stream, "}");
-         break;
-       }
-      /* Array of unspecified length: treat like pointer.  */
-
-    case TYPE_CODE_PTR:
-      if (format)
-       {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
-       }
-      fprintf (stream, "0x%x", * (int *) valaddr);
-      /* For a pointer to char or unsigned char,
-        also print the string pointed to, unless pointer is null.  */
-
-      /* For an array of chars, print with string syntax.  */
-      elttype = TYPE_TARGET_TYPE (type);
-      if (TYPE_LENGTH (elttype) == 1 && TYPE_CODE (elttype) == TYPE_CODE_INT
-         && format == 0
-         && unpack_long (type, valaddr) != 0)
-       {
-         fputc (' ', stream);
-         fputc ('"', stream);
-         for (i = 0; i < print_max; i++)
+#if TARGET_BYTE_ORDER == BIG_ENDIAN
+         first_addr = p;
+#endif
+         if (len <= sizeof (LONGEST))
            {
-             QUIT;
-             read_memory (unpack_long (type, valaddr) + i, &c, 1);
-             if (c == 0)
-               break;
-             printchar (c, stream, '"');
+             /* We can print it in decimal.  */
+             fprintf_filtered
+               (stream, 
+#if defined (LONG_LONG)
+                "%llu",
+#else
+                "%lu",
+#endif
+                unpack_long (BUILTIN_TYPE_LONGEST, first_addr));
            }
-         fputc ('"', stream);
-         if (i == print_max)
-           fprintf (stream, "...");
-         fflush (stream);
-         /* Return number of characters printed, plus one for the
-            terminating null if we have "reached the end".  */
-         return i + (i != print_max);
-       }
-      break;
-
-    case TYPE_CODE_STRUCT:
-    case TYPE_CODE_UNION:
-      fprintf (stream, "{");
-      len = TYPE_NFIELDS (type);
-      for (i = 0; i < len; i++)
-       {
-         if (i) fprintf (stream, ", ");
-         fprintf (stream, "%s = ", TYPE_FIELD_NAME (type, i));
-         if (TYPE_FIELD_PACKED (type, i))
+         else
            {
-             val = unpack_field_as_long (type, valaddr, i);
-             val_print (TYPE_FIELD_TYPE (type, i), &val, 0, stream, format);
+             /* It is big, so print it in hex.  */
+             print_hex_chars (stream, (unsigned char *) first_addr, len);
            }
-         else
-           val_print (TYPE_FIELD_TYPE (type, i), 
-                      valaddr + TYPE_FIELD_BITPOS (type, i) / 8,
-                      0, stream, format);
        }
-      fprintf (stream, "}");
-      break;
-
-    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++)
-       {
-         QUIT;
-         if (val == TYPE_FIELD_VALUE (type, i))
-           break;
-       }
-      if (i < len)
-       fprintf (stream, "%s", TYPE_FIELD_NAME (type, i));
       else
-       fprintf (stream, "%d", val);
-      break;
-
-    case TYPE_CODE_FUNC:
-      if (format)
        {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
+         /* 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));
        }
-      fprintf (stream, "{");
-      type_print (type, "", stream, -1);
-      fprintf (stream, "} ");
-      fprintf (stream, "0x%x", address);
-      break;
-
-    case TYPE_CODE_INT:
-      if (format)
-       {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
-       }
-      fprintf (stream,
-              TYPE_UNSIGNED (type) ? "%u" : "%d",
-              unpack_long (type, valaddr));
-      if (TYPE_LENGTH (type) == 1)
-       {
-         fprintf (stream, " '");
-         printchar (unpack_long (type, valaddr), stream, '\'');
-         fputc ('\'', stream);
-       }
-      break;
-
-    case TYPE_CODE_FLT:
-      if (format)
-       {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
-       }
-#ifdef IEEE_FLOAT
-      if (is_nan (unpack_double (type, valaddr)))
-       {
-         fprintf (stream, "Nan");
-         break;
-       }
-#endif
-      fprintf (stream, "%g", unpack_double (type, valaddr));
-      break;
-
-    case TYPE_CODE_VOID:
-      fprintf (stream, "void");
-      break;
-
-    default:
-      error ("Invalid type code in symbol table.");
     }
-  fflush (stream);
-}
-\f
-#ifdef IEEE_FLOAT
-
-union ieee {
-  int i[2];
-  double d;
-};
-
-/* Nonzero if ARG (a double) is a NAN.  */
-
-int
-is_nan (arg)
-     union ieee arg;
-{
-  int lowhalf, highhalf;
-  union { int i; char c; } test;
-
-  /* Separate the high and low words of the double.
-     Distinguish big and little-endian machines.  */
-  test.i = 1;
-  if (test.c != 1)
-    /* Big-endian machine */
-    lowhalf = arg.i[1], highhalf = arg.i[0];
   else
-    lowhalf = arg.i[0], highhalf = arg.i[1];
-
-  /* Nan: exponent is the maximum possible, and fraction is nonzero.  */
-  return (((highhalf>>20) & 0x7ff) == 0x7ff
-         &&
-         ! ((highhalf & 0xfffff == 0) && (lowhalf == 0)));
-}
+    {
+#ifdef PRINT_TYPELESS_INTEGER
+      PRINT_TYPELESS_INTEGER (stream, type, unpack_long (type, valaddr));
+#else
+      fprintf_filtered (stream, TYPE_UNSIGNED (type) ?
+#if defined (LONG_LONG)
+                       "%llu" : "%lld",
+#else
+                       "%u" : "%d",
 #endif
-\f
-/* Print a description of a type TYPE
-   in the form of a declaration of a variable named VARSTRING.
-   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.  */
-
-type_print (type, varstring, stream, show)
-     struct type *type;
-     char *varstring;
-     FILE *stream;
-     int show;
-{
-  type_print_1 (type, varstring, stream, show, 0);
-}
+                       unpack_long (type, valaddr));
+#endif
+    }
+}                      
 
-/* LEVEL is the depth to indent lines by.  */
+/* Print a floating point value of type TYPE, pointed to in GDB by VALADDR,
+   on STREAM.  */
 
-type_print_1 (type, varstring, stream, show, level)
+void
+print_floating (valaddr, type, stream)
+     char *valaddr;
      struct type *type;
-     char *varstring;
      FILE *stream;
-     int show;
-     int level;
 {
-  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_ARRAY)))
-    fprintf (stream, " ");
-  type_print_varspec_prefix (type, stream, show, 0);
-  fprintf (stream, "%s", varstring);
-  type_print_varspec_suffix (type, stream, show, 0);
+  double doub;
+  int inv;
+  unsigned len = TYPE_LENGTH (type);
+  
+#if defined (IEEE_FLOAT)
+
+  /* Check for NaN's.  Note that this code does not depend on us being
+     on an IEEE conforming system.  It only depends on the target
+     machine using IEEE representation.  This means (a)
+     cross-debugging works right, and (2) IEEE_FLOAT can (and should)
+     be defined for systems like the 68881, which uses IEEE
+     representation, but is not IEEE conforming.  */
+
+  {
+    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))
+      {
+       /* It's single precision. */
+       memcpy ((char *) &low, valaddr, sizeof (low));
+       /* target -> host.  */
+       SWAP_TARGET_AND_HOST (&low, sizeof (float));
+       nonnegative = low >= 0;
+       is_nan = ((((low >> 23) & 0xFF) == 0xFF) 
+                 && 0 != (low & 0x7FFFFF));
+       low &= 0x7fffff;
+       high = 0;
+      }
+    else
+      {
+       /* It's double precision.  Get the high and low words.  */
+
+#if TARGET_BYTE_ORDER == BIG_ENDIAN
+       memcpy (&low, valaddr+4,  sizeof (low));
+       memcpy (&high, valaddr+0, sizeof (high));
+#else
+       memcpy (&low, valaddr+0,  sizeof (low));
+       memcpy (&high, valaddr+4, 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;
+      }
+
+    if (is_nan)
+      {
+       /* The meaning of the sign and fraction is not defined by IEEE.
+          But the user might know what they mean.  For example, they
+          (in an implementation-defined manner) distinguish between
+          signaling and quiet NaN's.  */
+       if (high)
+         fprintf_filtered (stream, "-NaN(0x%lx%.8lx)" + nonnegative,
+                           high, low);
+       else
+         fprintf_filtered (stream, "-NaN(0x%lx)" + nonnegative, low);
+       return;
+      }
+  }
+#endif /* IEEE_FLOAT.  */
+
+  doub = unpack_double (type, valaddr, &inv);
+  if (inv)
+    fprintf_filtered (stream, "<invalid float value>");
+  else
+    fprintf_filtered (stream, len <= sizeof(float) ? "%.9g" : "%.17g", doub);
 }
 
-/* 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.  */
+/* VALADDR points to an integer of LEN bytes.  Print it in hex on stream.  */
 
 static void
-type_print_varspec_prefix (type, stream, show, passed_a_ptr)
-     struct type *type;
+print_hex_chars (stream, valaddr, len)
      FILE *stream;
-     int show;
-     int passed_a_ptr;
+     unsigned char *valaddr;
+     unsigned len;
 {
-  if (type == 0)
-    return;
-
-  if (TYPE_NAME (type) && show <= 0)
-    return;
-
-  QUIT;
-
-  switch (TYPE_CODE (type))
+  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
     {
-    case TYPE_CODE_PTR:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      fputc ('*', stream);
-      break;
-
-    case TYPE_CODE_FUNC:
-    case TYPE_CODE_ARRAY:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0, 0);
-      if (passed_a_ptr)
-       fputc ('(', stream);
-      break;
+      fprintf_filtered (stream, "%02x", *p);
     }
 }
 
-/* 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.  */
+/*  Called by various <lang>_val_print routines to print elements of an
+    array in the form "<elem1>, <elem2>, <elem3>, ...".
 
-static void
-type_print_varspec_suffix (type, stream, show, passed_a_ptr)
+    (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.
+    */
+
+void
+val_print_array_elements (type, valaddr, address, stream, format, deref_ref,
+                         recurse, pretty, i)
      struct type *type;
+     char *valaddr;
+     CORE_ADDR address;
      FILE *stream;
-     int show;
-     int passed_a_ptr;
+     int format;
+     int deref_ref;
+     int recurse;
+     enum val_prettyprint pretty;
+     unsigned int i;
 {
-  if (type == 0)
-    return;
-
-  if (TYPE_NAME (type) && show <= 0)
-    return;
-
-  QUIT;
-
-  switch (TYPE_CODE (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 (elttype);
+  len = TYPE_LENGTH (type) / eltlen;
+             
+  for (; i < len && things_printed < print_max; i++)
     {
-    case TYPE_CODE_ARRAY:
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 0);
-      if (passed_a_ptr)
-       fprintf (stream, ")");
-      fprintf (stream, "[");
-      if (TYPE_LENGTH (type) >= 0)
-       fprintf (stream, "%d",
-                TYPE_LENGTH (type) / TYPE_LENGTH (TYPE_TARGET_TYPE (type)));
-      fprintf (stream, "]");
-      break;
-
-    case TYPE_CODE_PTR:
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      break;
-
-    case TYPE_CODE_FUNC:
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 0);
-      if (passed_a_ptr)
-       fprintf (stream, ")");
-      fprintf (stream, "()");
-      break;
+      if (i != 0)
+       {
+         if (prettyprint_arrays)
+           {
+             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) && 
+            !memcmp (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, "...");
     }
 }
 
-/* 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.
-
-   LEVEL is the depth to indent by.
-   We increase it for some recursive calls.  */
-
 static void
-type_print_base (type, stream, show, level)
-     struct type *type;
+value_print_array_elements (val, stream, format, pretty)
+     value val;
      FILE *stream;
-     int show;
-     int level;
+     int format;
+     enum val_prettyprint pretty;
 {
-  char *name;
-  register int i;
-  register int len;
-  register int lastval;
-
-  QUIT;
-
-  if (type == 0)
+  unsigned int things_printed = 0;
+  register unsigned int i, n, typelen;
+  /* Position of the array elem we are examining to see if it is repeated.  */
+  unsigned int rep1;
+  /* Number of repetitions we have detected so far.  */
+  unsigned int reps;
+    
+  n = VALUE_REPETITIONS (val);
+  typelen = TYPE_LENGTH (VALUE_TYPE (val));
+  for (i = 0; i < n && things_printed < print_max; i++)
     {
-      fprintf (stream, "type unknown");
-      return;
-    }
-
-  if (TYPE_NAME (type) && show <= 0)
-    {
-      fprintf (stream, TYPE_NAME (type));
-      return;
-    }
-
-  switch (TYPE_CODE (type))
-    {
-    case TYPE_CODE_ARRAY:
-    case TYPE_CODE_PTR:
-    case TYPE_CODE_FUNC:
-      type_print_base (TYPE_TARGET_TYPE (type), stream, show, level);
-      break;
-
-    case TYPE_CODE_STRUCT:
-      fprintf (stream, "struct ");
-      goto struct_union;
-
-    case TYPE_CODE_UNION:
-      fprintf (stream, "union ");
-    struct_union:
-      if (TYPE_NAME (type) && (name = TYPE_NAME (type)))
+      if (i != 0)
+       {
+         fprintf_filtered (stream, ", ");
+       }
+      wrap_here ("");
+      
+      rep1 = i + 1;
+      reps = 1;
+      while (rep1 < n && !memcmp (VALUE_CONTENTS (val) + typelen * i,
+                                 VALUE_CONTENTS (val) + typelen * rep1,
+                                 typelen))
+       {
+         ++reps;
+         ++rep1;
+       }
+      
+      if (reps > repeat_count_threshold)
        {
-         while (*name != ' ') name++;
-         fprintf (stream, "%s ", name + 1);
+         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;
        }
-      if (show < 0)
-       fprintf (stream, "{...}");
       else
        {
-         fprintf (stream, "{");
-         len = TYPE_NFIELDS (type);
-         fprintf (stream, "\n");
-         for (i = 0; i < len; i++)
-           {
-             QUIT;
-             print_spaces (level + 4, stream);
-
-             /* If this is a bit-field and there is a gap before it,
-                print a nameless field to account for the gap.  */
+         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, "...");
+    }
+}
 
-             if (TYPE_FIELD_PACKED (type, i))
-               {
-                 int gap = (TYPE_FIELD_BITPOS (type, i)
-                            - (i > 0
-                               ? (TYPE_FIELD_BITPOS (type, i - 1)
-                                  + (TYPE_FIELD_PACKED (type, i - 1)
-                                     ? TYPE_FIELD_BITSIZE (type, i - 1)
-                                     : TYPE_LENGTH (TYPE_FIELD_TYPE (type, i - 1)) * 8))
-                               : 0));
-                 if (gap != 0)
-                   {
-                     fprintf (stream, "int : %d;\n", gap);
-                     print_spaces (level + 4, stream);
-                   }
-               }
+\f
+#if 0
+/* 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!  */
 
-             /* Print the declaration of this field.  */
+/* ARGSUSED */
+static void
+set_input_radix (args, from_tty, c)
+     char *args;
+     int from_tty;
+     struct cmd_list_element *c;
+{
+  unsigned radix = *(unsigned *)c->var;
 
-             type_print_1 (TYPE_FIELD_TYPE (type, i),
-                           TYPE_FIELD_NAME (type, i),
-                           stream, show - 1, level + 4);
+  if (from_tty)
+    printf_filtered ("Input radix set to decimal %d, hex %x, octal %o\n",
+       radix, radix, radix);
+}
+#endif
 
-             /* Print the field width.  */
+/* ARGSUSED */
+static void
+set_output_radix (args, from_tty, c)
+     char *args;
+     int from_tty;
+     struct cmd_list_element *c;
+{
+  unsigned radix = *(unsigned *)c->var;
 
-             if (TYPE_FIELD_PACKED (type, i))
-               fprintf (stream, " : %d", TYPE_FIELD_BITSIZE (type, i));
+  if (from_tty)
+    printf_filtered ("Output radix set to decimal %d, hex %x, octal %o\n",
+       radix, radix, radix);
 
-             fprintf (stream, ";\n");
-           }
-         print_spaces (level, stream);
-         fputc ('}', stream);
-       }
+  /* FIXME, we really should be able to validate the setting BEFORE
+     it takes effect.  */
+  switch (radix)
+    {
+    case 16:
+      output_format = 'x';
       break;
-
-    case TYPE_CODE_ENUM:
-      fprintf (stream, "enum ");
-      if (TYPE_NAME (type))
-       {
-         name = TYPE_NAME (type);
-         while (*name != ' ') name++;
-         fprintf (stream, "%s ", name + 1);
-       }
-      if (show < 0)
-       fprintf (stream, "{...}");
-      else
-       {
-         fprintf (stream, "{");
-         len = TYPE_NFIELDS (type);
-         lastval = 0;
-         for (i = 0; i < len; i++)
-           {
-             QUIT;
-             if (i) fprintf (stream, ", ");
-             fprintf (stream, "%s", TYPE_FIELD_NAME (type, i));
-             if (lastval != TYPE_FIELD_VALUE (type, i))
-               {
-                 fprintf (stream, " : %d", TYPE_FIELD_VALUE (type, i));
-                 lastval = TYPE_FIELD_VALUE (type, i);
-               }
-             lastval++;
-           }
-         fprintf (stream, "}");
-       }
+    case 10:
+      output_format = 0;
       break;
-
-    case TYPE_CODE_INT:
-      if (TYPE_UNSIGNED (type))
-       name = unsigned_type_table[TYPE_LENGTH (type)];
-      else
-       name = signed_type_table[TYPE_LENGTH (type)];
-      fprintf (stream, "%s", name);
+    case 8:
+      output_format = 'o';             /* octal */
       break;
+    default:
+      output_format = 0;
+      error ("Unsupported radix ``decimal %d''; using decimal output",
+             radix);
+    }
+}
 
-    case TYPE_CODE_FLT:
-      name = float_type_table[TYPE_LENGTH (type)];
-      fprintf (stream, "%s", name);
-      break;
+/* 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;
 
-    case TYPE_CODE_VOID:
-      fprintf (stream, "void");
-      break;
+  if (from_tty)
+    printf_filtered ("Radix set to decimal %d, hex %x, octal %o\n",
+       radix, radix, radix);
 
-    case 0:
-      fprintf (stream, "struct unknown");
-      break;
+  input_radix = radix;
+  output_radix = radix;
 
-    default:
-      error ("Invalid type code in symbol table.");
-    }
+  set_output_radix (arg, 0, c);
 }
 \f
+/*ARGSUSED*/
 static void
-set_maximum_command (arg)
+set_print (arg, from_tty)
      char *arg;
+     int from_tty;
 {
-  if (!arg) error_no_arg ("value for maximum elements to print");
-  print_max = atoi (arg);
+  printf (
+"\"set print\" must be followed by the name of a print subcommand.\n");
+  help_list (setprintlist, "set print ", -1, stdout);
 }
 
-static
-initialize ()
+/*ARGSUSED*/
+static void
+show_print (args, from_tty)
+     char *args;
+     int from_tty;
 {
-  add_com ("set-maximum", class_vars, set_maximum_command,
-          "Set NUMBER as limit on string chars or array elements to print.");
-
+  cmd_show_list (showprintlist, from_tty, "");
+}
+\f
+void
+_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); 
+  add_alias_cmd ("pr", "print", no_class, 1, &setlist); /* prefer set print
+                                                                                                                  to     set prompt */
+  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 ("elements", no_class, var_uinteger, (char *)&print_max,
+                 "Set limit on string chars or array elements to print.\n\
+\"set print elements 0\" causes there to be no limit.",
+                 &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_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 ("union", class_support, var_boolean, (char *)&unionprint,
+                 "Set printing of unions interior to structures.",
+                 &setprintlist),
+     &showprintlist);
+  
+  add_show_from_set
+    (add_set_cmd ("array", class_support, var_boolean,
+                 (char *)&prettyprint_arrays,
+                 "Set prettyprinting of arrays.",
+                 &setprintlist),
+     &showprintlist);
+
+  add_show_from_set
+    (add_set_cmd ("address", class_support, var_boolean, (char *)&addressprint,
+                 "Set printing of addresses.",
+                 &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 = 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.sfunc = set_radix;
+
+  /* Give people the defaults which they are used to.  */
+  prettyprint_structs = 0;
+  prettyprint_arrays = 0;
+  unionprint = 1;
+  addressprint = 1;
   print_max = 200;
-
-  unsigned_type_table
-    = (char **) xmalloc ((1 + sizeof (unsigned long)) * sizeof (char *));
-  bzero (unsigned_type_table, (1 + sizeof (unsigned long)));
-  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";
-
-  signed_type_table
-    = (char **) xmalloc ((1 + sizeof (long)) * sizeof (char *));
-  bzero (signed_type_table, (1 + sizeof (long)));
-  signed_type_table[sizeof (char)] = "char";
-  signed_type_table[sizeof (short)] = "short";
-  signed_type_table[sizeof (long)] = "long";
-  signed_type_table[sizeof (int)] = "int";
-
-  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";
 }
-
-END_FILE