* dummy-frame.c (deprecated_pc_in_call_dummy): Add GDBARCH parameter,
[deliverable/binutils-gdb.git] / gdb / valprint.c
index 58d53690c872ea750baaf9aeac50ea6a4d5e5e27..756dc50a57ff585c5855a6696221ac102a148ad4 100644 (file)
-/* Print values for GNU debugger gdb.
-   Copyright (C) 1986, 1988, 1989 Free Software Foundation, Inc.
+/* Print values for GDB, the GNU debugger.
 
-This file is part of GDB.
+   Copyright (C) 1986, 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995, 1996,
+   1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008,
+   2009 Free Software Foundation, Inc.
 
-GDB 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.
+   This file is part of GDB.
 
-GDB 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.
+   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 3 of the License, or
+   (at your option) any later version.
 
-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.  */
+   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, see <http://www.gnu.org/licenses/>.  */
 
-#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 "annotate.h"
+#include "valprint.h"
+#include "floatformat.h"
+#include "doublest.h"
+#include "exceptions.h"
+#include "dfp.h"
+#include "python/python.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 int partial_memory_read (CORE_ADDR memaddr, gdb_byte *myaddr,
+                               int len, int *errnoptr);
 
-static unsigned int print_max;
+static void show_print (char *, int);
 
-static void type_print_varspec_suffix ();
-static void type_print_varspec_prefix ();
-static void type_print_base ();
-static void type_print_method_args ();
+static void set_print (char *, int);
 
-/* Default input and output radixes, and output format letter.  */
+static void set_radix (char *, int);
 
-unsigned input_radix = 10;
-unsigned output_radix = 10;
-int output_format = 0;
+static void show_radix (char *, int);
 
+static void set_input_radix (char *, int, struct cmd_list_element *);
 
-char **unsigned_type_table;
-char **signed_type_table;
-char **float_type_table;
+static void set_input_radix_1 (int, unsigned);
 
+static void set_output_radix (char *, int, struct cmd_list_element *);
 
-/* 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.  */
+static void set_output_radix_1 (int, unsigned);
+
+void _initialize_valprint (void);
 
+#define PRINT_MAX_DEFAULT 200  /* Start print_max off at this value. */
+
+struct value_print_options user_print_options =
+{
+  Val_pretty_default,          /* pretty */
+  0,                           /* prettyprint_arrays */
+  0,                           /* prettyprint_structs */
+  0,                           /* vtblprint */
+  1,                           /* unionprint */
+  1,                           /* addressprint */
+  0,                           /* objectprint */
+  PRINT_MAX_DEFAULT,           /* print_max */
+  10,                          /* repeat_count_threshold */
+  0,                           /* output_format */
+  0,                           /* format */
+  0,                           /* stop_print_at_null */
+  0,                           /* inspect_it */
+  0,                           /* print_array_indexes */
+  0,                           /* deref_ref */
+  1,                           /* static_field_print */
+  1,                           /* pascal_static_field_print */
+  0,                           /* raw */
+  0                            /* summary */
+};
+
+/* Initialize *OPTS to be a copy of the user print options.  */
 void
-print_string (stream, string, length, force_ellipses)
-     FILE *stream;
-     char *string;
-     unsigned int length;
-     int force_ellipses;
+get_user_print_options (struct value_print_options *opts)
 {
-  register unsigned int i;
-  unsigned int things_printed = 0;
-  int in_quotes = 0;
-  int need_comma = 0;
-  extern int inspect_it;
+  *opts = user_print_options;
+}
 
-  if (length == 0)
-    {
-      fputs_filtered ("\"\"", stdout);
-      return;
-    }
+/* Initialize *OPTS to be a copy of the user print options, but with
+   pretty-printing disabled.  */
+void
+get_raw_print_options (struct value_print_options *opts)
+{  
+  *opts = user_print_options;
+  opts->pretty = Val_no_prettyprint;
+}
 
-  for (i = 0; i < length && things_printed < print_max; ++i)
-    {
-      /* 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;
+/* Initialize *OPTS to be a copy of the user print options, but using
+   FORMAT as the formatting option.  */
+void
+get_formatted_print_options (struct value_print_options *opts,
+                            char format)
+{
+  *opts = user_print_options;
+  opts->format = format;
+}
 
-      QUIT;
+static void
+show_print_max (struct ui_file *file, int from_tty,
+               struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("\
+Limit on string chars or array elements to print is %s.\n"),
+                   value);
+}
 
-      if (need_comma)
-       {
-         fputs_filtered (", ", stream);
-         need_comma = 0;
-       }
 
-      rep1 = i + 1;
-      reps = 1;
-      while (rep1 < length && string[rep1] == string[i])
-       {
-         ++rep1;
-         ++reps;
-       }
+/* Default input and output radixes, and output format letter.  */
 
-      if (reps > REPEAT_COUNT_THRESHOLD)
-       {
-         if (in_quotes)
-           {
-             if (inspect_it)
-               fputs_filtered ("\\\", ", stream);
-             else
-               fputs_filtered ("\", ", stream);
-             in_quotes = 0;
-           }
-         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
-       {
-         if (!in_quotes)
-           {
-             if (inspect_it)
-               fputs_filtered ("\\\"", stream);
-             else
-               fputs_filtered ("\"", stream);
-             in_quotes = 1;
-           }
-         printchar (string[i], stream, '"');
-         ++things_printed;
-       }
-    }
+unsigned input_radix = 10;
+static void
+show_input_radix (struct ui_file *file, int from_tty,
+                 struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("\
+Default input radix for entering numbers is %s.\n"),
+                   value);
+}
 
-  /* Terminate the quotes if necessary.  */
-  if (in_quotes)
-    {
-      if (inspect_it)
-       fputs_filtered ("\\\"", stream);
-      else
-       fputs_filtered ("\"", stream);
-    }
+unsigned output_radix = 10;
+static void
+show_output_radix (struct ui_file *file, int from_tty,
+                  struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("\
+Default output radix for printing of values is %s.\n"),
+                   value);
+}
+
+/* By default we print arrays without printing the index of each element in
+   the array.  This behavior can be changed by setting PRINT_ARRAY_INDEXES.  */
 
-  if (force_ellipses || i < length)
-    fputs_filtered ("...", stream);
+static void
+show_print_array_indexes (struct ui_file *file, int from_tty,
+                         struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("Printing of array indexes is %s.\n"), value);
 }
 
-/* Print a floating point value of type TYPE, pointed to in GDB by VALADDR,
-   on STREAM.  */
+/* 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. */
 
-void
-print_floating (valaddr, type, stream)
-     char *valaddr;
-     struct type *type;
-     FILE *stream;
+static void
+show_repeat_count_threshold (struct ui_file *file, int from_tty,
+                            struct cmd_list_element *c, const char *value)
 {
-  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. */
-       bcopy (valaddr, &low, 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
-         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;
-       }
+  fprintf_filtered (file, _("Threshold for repeated print elements is %s.\n"),
+                   value);
+}
 
-    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.  */
+/* If nonzero, stops printing of char arrays at first null. */
 
-  doub = unpack_double (type, valaddr, &inv);
-  if (inv)
-    fprintf_filtered (stream, "<invalid float value>");
-  else
-    fprintf_filtered (stream, len <= sizeof(float) ? "%.6g" : "%.17g", doub);
+static void
+show_stop_print_at_null (struct ui_file *file, int from_tty,
+                        struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("\
+Printing of char arrays to stop at first null char is %s.\n"),
+                   value);
 }
 
-/* VALADDR points to an integer of LEN bytes.  Print it in hex on stream.  */
+/* Controls pretty printing of structures. */
+
 static void
-print_hex_chars (stream, valaddr, len)
-     FILE *stream;
-     unsigned char *valaddr;
-     unsigned len;
+show_prettyprint_structs (struct ui_file *file, int from_tty,
+                         struct cmd_list_element *c, const char *value)
 {
-  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);
-    }
+  fprintf_filtered (file, _("Prettyprinting of structures is %s.\n"), value);
 }
-\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;
+/* Controls pretty printing of arrays.  */
+
+static void
+show_prettyprint_arrays (struct ui_file *file, int from_tty,
+                        struct cmd_list_element *c, const char *value)
 {
-  register unsigned int i, n, typelen;
+  fprintf_filtered (file, _("Prettyprinting of arrays is %s.\n"), value);
+}
 
-  if (val == 0)
+/* If nonzero, causes unions inside structures or other unions to be
+   printed. */
+
+static void
+show_unionprint (struct ui_file *file, int from_tty,
+                struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("\
+Printing of unions interior to structures is %s.\n"),
+                   value);
+}
+
+/* If nonzero, causes machine addresses to be printed in certain contexts. */
+
+static void
+show_addressprint (struct ui_file *file, int from_tty,
+                  struct cmd_list_element *c, const char *value)
+{
+  fprintf_filtered (file, _("Printing of addresses is %s.\n"), value);
+}
+\f
+
+/* A helper function for val_print.  When printing in "summary" mode,
+   we want to print scalar arguments, but not aggregate arguments.
+   This function distinguishes between the two.  */
+
+static int
+scalar_type_p (struct type *type)
+{
+  CHECK_TYPEDEF (type);
+  while (TYPE_CODE (type) == TYPE_CODE_REF)
     {
-      printf_filtered ("<address of value unknown>");
-      return 0;
+      type = TYPE_TARGET_TYPE (type);
+      CHECK_TYPEDEF (type);
     }
-  if (VALUE_OPTIMIZED_OUT (val))
+  switch (TYPE_CODE (type))
     {
-      printf_filtered ("<value optimized out>");
+    case TYPE_CODE_ARRAY:
+    case TYPE_CODE_STRUCT:
+    case TYPE_CODE_UNION:
+    case TYPE_CODE_SET:
+    case TYPE_CODE_STRING:
+    case TYPE_CODE_BITSTRING:
       return 0;
+    default:
+      return 1;
     }
-  
-  /* A "repeated" value really contains several values in a row.
-     They are made by the @ operator.
-     Print such values as if they were arrays.  */
+}
 
-  else if (VALUE_REPEATED (val))
-    {
-      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
-       {
-         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;
-               }
+/* Print using the given LANGUAGE the data of type TYPE located at VALADDR
+   (within GDB), which came from the inferior at address ADDRESS, onto
+   stdio stream STREAM according to OPTIONS.
 
-             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, "}");
-      return n * typelen;
-    }
-  else
+   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 (struct type *type, const gdb_byte *valaddr, int embedded_offset,
+          CORE_ADDR address, struct ui_file *stream, int recurse,
+          const struct value_print_options *options,
+          const struct language_defn *language)
+{
+  volatile struct gdb_exception except;
+  int ret = 0;
+  struct value_print_options local_opts = *options;
+  struct type *real_type = check_typedef (type);
+
+  if (local_opts.pretty == Val_pretty_default)
+    local_opts.pretty = (local_opts.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 (TYPE_STUB (real_type))
     {
-      /* If it is a pointer, indicate what it points to.
+      fprintf_filtered (stream, "<incomplete type>");
+      gdb_flush (stream);
+      return (0);
+    }
 
-        Print type also if it is a reference.
+  if (!options->raw)
+    {
+      ret = apply_val_pretty_printer (type, valaddr, embedded_offset,
+                                     address, stream, recurse, options,
+                                     language);
+      if (ret)
+       return ret;
+    }
 
-         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)
-       {
-         /* 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, ") ");
-           }
-       }
-      return val_print (VALUE_TYPE (val), VALUE_CONTENTS (val),
-                       VALUE_ADDRESS (val), stream, format, 1, 0, pretty);
+  /* Handle summary mode.  If the value is a scalar, print it;
+     otherwise, print an ellipsis.  */
+  if (options->summary && !scalar_type_p (type))
+    {
+      fprintf_filtered (stream, "...");
+      return 0;
     }
-}
 
-/* 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' };
+  TRY_CATCH (except, RETURN_MASK_ERROR)
+    {
+      ret = language->la_val_print (type, valaddr, embedded_offset, address,
+                                   stream, recurse, &local_opts);
+    }
+  if (except.reason < 0)
+    fprintf_filtered (stream, _("<error reading variable>"));
 
-  return (typename != NULL && !strcmp(typename, vtbl_ptr_name));
+  return ret;
 }
 
-/* Return truth value for the assertion that TYPE is of the type
-   "pointer to virtual function table".  */
+/* Check whether the value VAL is printable.  Return 1 if it is;
+   return 0 and print an appropriate error message to STREAM if it
+   is not.  */
+
 static int
-is_vtbl_member(type)
-     struct type *type;
+value_check_printable (struct value *val, struct ui_file *stream)
 {
-  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.  */
+  if (val == 0)
+    {
+      fprintf_filtered (stream, _("<address of value unknown>"));
+      return 0;
+    }
+
+  if (value_optimized_out (val))
+    {
+      fprintf_filtered (stream, _("<value optimized out>"));
+      return 0;
+    }
 
-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.  */
+  if (TYPE_CODE (value_type (val)) == TYPE_CODE_INTERNAL_FUNCTION)
+    {
+      fprintf_filtered (stream, _("<internal function %s>"),
+                       value_internal_function_name (val));
+      return 0;
+    }
 
-struct obstack dont_print_obstack;
+  return 1;
+}
 
-static void cplus_val_print ();
+/* Print using the given LANGUAGE the value VAL onto stream STREAM according
+   to OPTIONS.
 
-/* Mutually recursive subroutines of cplus_val_print and val_print to print out
-   a structure's fields: val_print_fields and cplus_val_print.
+   If the data are a string pointer, returns the number of string characters
+   printed.
 
-   TYPE, VALADDR, STREAM, RECURSE, and PRETTY have the
-   same meanings as in cplus_val_print and val_print.
+   This is a preferable interface to val_print, above, because it uses
+   GDB's value mechanism.  */
 
-   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)
-     struct type *type;
-     char *valaddr;
-     FILE *stream;
-     char format;
-     int recurse;
-     enum val_prettyprint pretty;
-     struct type **dont_print;
+int
+common_val_print (struct value *val, struct ui_file *stream, int recurse,
+                 const struct value_print_options *options,
+                 const struct language_defn *language)
 {
-  int i, len, n_baseclasses;
+  if (!value_check_printable (val, stream))
+    return 0;
 
-  fprintf_filtered (stream, "{");
-  len = TYPE_NFIELDS (type);
-  n_baseclasses = TYPE_N_BASECLASSES (type);
+  return val_print (value_type (val), value_contents_all (val),
+                   value_embedded_offset (val), value_address (val),
+                   stream, recurse, options, language);
+}
 
-  /* 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);
+/* Print the value VAL in C-ish syntax on stream STREAM according to
+   OPTIONS.
+   If the object printed is a string pointer, returns
+   the number of string bytes printed.  */
 
-  if (!len && n_baseclasses == 1)
-    fprintf_filtered (stream, "<No data fields>");
-  else
+int
+value_print (struct value *val, struct ui_file *stream,
+            const struct value_print_options *options)
+{
+  if (!value_check_printable (val, stream))
+    return 0;
+
+  if (!options->raw)
     {
-      extern int inspect_it;
-      int fields_seen = 0;
+      int r = apply_val_pretty_printer (value_type (val),
+                                       value_contents_all (val),
+                                       value_embedded_offset (val),
+                                       value_address (val),
+                                       stream, 0, options,
+                                       current_language);
+      if (r)
+       return r;
+    }
 
-      for (i = n_baseclasses; i < len; i++)
-       {
-         /* 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;
+  return LA_VALUE_PRINT (val, stream, options);
+}
 
-         if (pretty)
-           {
-             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);
-           }
-         if (TYPE_FIELD_PACKED (type, i))
-           {
-             value v;
+/* 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.  */
 
-             /* Bitfields require special handling, especially due to byte
-                order problems.  */
-             v = value_from_long (TYPE_FIELD_TYPE (type, i),
-                                  unpack_field_as_long (type, valaddr, i));
+void
+val_print_type_code_int (struct type *type, const gdb_byte *valaddr,
+                        struct ui_file *stream)
+{
+  enum bfd_endian byte_order = gdbarch_byte_order (current_gdbarch);
 
-             val_print (TYPE_FIELD_TYPE (type, i), VALUE_CONTENTS (v), 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 (TYPE_LENGTH (type) > sizeof (LONGEST))
+    {
+      LONGEST val;
+
+      if (TYPE_UNSIGNED (type)
+         && extract_long_unsigned_integer (valaddr, TYPE_LENGTH (type),
+                                           &val))
+       {
+         print_longest (stream, 'u', 0, val);
        }
-      if (pretty)
+      else
        {
-         fprintf_filtered (stream, "\n");
-         print_spaces_filtered (2 * recurse, stream);
+         /* 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), byte_order);
        }
     }
-  fprintf_filtered (stream, "}");
+  else
+    {
+      print_longest (stream, TYPE_UNSIGNED (type) ? 'u' : 'd', 0,
+                    unpack_long (type, valaddr));
+    }
 }
 
-/* Special val_print routine to avoid printing multiple copies of virtual
-   baseclasses.  */
-
-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;
+void
+val_print_type_code_flags (struct type *type, const gdb_byte *valaddr,
+                          struct ui_file *stream)
 {
-  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);
+  ULONGEST val = unpack_long (type, valaddr);
+  int bitpos, nfields = TYPE_NFIELDS (type);
 
-  if (dont_print == 0)
+  fputs_filtered ("[ ", stream);
+  for (bitpos = 0; bitpos < nfields; bitpos++)
     {
-      /* 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);
+      if (TYPE_FIELD_BITPOS (type, bitpos) != -1
+         && (val & ((ULONGEST)1 << bitpos)))
+       {
+         if (TYPE_FIELD_NAME (type, bitpos))
+           fprintf_filtered (stream, "%s ", TYPE_FIELD_NAME (type, bitpos));
+         else
+           fprintf_filtered (stream, "#%d ", bitpos);
+       }
     }
+  fputs_filtered ("]", stream);
+}
+
+/* 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. The format chars b,h,w,g are 
+   from print_scalar_formatted().  Numbers are printed using C
+   format. 
+
+   USE_C_FORMAT means to use C format in all cases.  Without it, 
+   'o' and 'x' format do not include the standard C radix prefix
+   (leading 0 or 0x). 
+   
+   Hilfinger/2004-09-09: USE_C_FORMAT was originally called USE_LOCAL
+   and was intended to request formating according to the current
+   language and would be used for most integers that GDB prints.  The
+   exceptional cases were things like protocols where the format of
+   the integer is a protocol thing, not a user-visible thing).  The
+   parameter remains to preserve the information of what things might
+   be printed with language-specific format, should we ever resurrect
+   that capability. */
 
-  for (i = 0; i < n_baseclasses; i++)
+void
+print_longest (struct ui_file *stream, int format, int use_c_format,
+              LONGEST val_long)
+{
+  const char *val;
+
+  switch (format)
     {
-      char *baddr;
+    case 'd':
+      val = int_string (val_long, 10, 1, 0, 1); break;
+    case 'u':
+      val = int_string (val_long, 10, 0, 0, 1); break;
+    case 'x':
+      val = int_string (val_long, 16, 0, 0, use_c_format); break;
+    case 'b':
+      val = int_string (val_long, 16, 0, 2, 1); break;
+    case 'h':
+      val = int_string (val_long, 16, 0, 4, 1); break;
+    case 'w':
+      val = int_string (val_long, 16, 0, 8, 1); break;
+    case 'g':
+      val = int_string (val_long, 16, 0, 16, 1); break;
+      break;
+    case 'o':
+      val = int_string (val_long, 8, 0, 0, use_c_format); break;
+    default:
+      internal_error (__FILE__, __LINE__, _("failed internal consistency check"));
+    } 
+  fputs_filtered (val, stream);
+}
 
-      if (BASETYPE_VIA_VIRTUAL (type, i))
+/* 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 (LONGEST arg)
+{
+  /* Let the compiler do the work */
+  int rtnval = (int) arg;
+
+  /* Check for overflows or underflows */
+  if (sizeof (LONGEST) > sizeof (int))
+    {
+      if (rtnval != arg)
        {
-         struct type **first_dont_print
-           = (struct type **)obstack_base (&dont_print_obstack);
+         error (_("Value out of range."));
+       }
+    }
+  return (rtnval);
+}
 
-         int j = (struct type **)obstack_next_free (&dont_print_obstack)
-           - first_dont_print;
+/* Print a floating point value of type TYPE (not always a
+   TYPE_CODE_FLT), pointed to in GDB by VALADDR, on STREAM.  */
 
-         while (--j >= 0)
-           if (TYPE_BASECLASS (type, i) == first_dont_print[j])
-             goto flush_it;
+void
+print_floating (const gdb_byte *valaddr, struct type *type,
+               struct ui_file *stream)
+{
+  DOUBLEST doub;
+  int inv;
+  const struct floatformat *fmt = NULL;
+  unsigned len = TYPE_LENGTH (type);
+  enum float_kind kind;
 
-         obstack_ptr_grow (&dont_print_obstack, TYPE_BASECLASS (type, i));
+  /* If it is a floating-point, check for obvious problems.  */
+  if (TYPE_CODE (type) == TYPE_CODE_FLT)
+    fmt = floatformat_from_type (type);
+  if (fmt != NULL)
+    {
+      kind = floatformat_classify (fmt, valaddr);
+      if (kind == float_nan)
+       {
+         if (floatformat_is_negative (fmt, valaddr))
+           fprintf_filtered (stream, "-");
+         fprintf_filtered (stream, "nan(");
+         fputs_filtered ("0x", stream);
+         fputs_filtered (floatformat_mantissa (fmt, valaddr), stream);
+         fprintf_filtered (stream, ")");
+         return;
+       }
+      else if (kind == float_infinite)
+       {
+         if (floatformat_is_negative (fmt, valaddr))
+           fputs_filtered ("-", stream);
+         fputs_filtered ("inf", stream);
+         return;
        }
-
-      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)
+  /* NOTE: cagney/2002-01-15: The TYPE passed into print_floating()
+     isn't necessarily a TYPE_CODE_FLT.  Consequently, unpack_double
+     needs to be used as that takes care of any necessary type
+     conversions.  Such conversions are of course direct to DOUBLEST
+     and disregard any possible target floating point limitations.
+     For instance, a u64 would be converted and displayed exactly on a
+     host with 80 bit DOUBLEST but with loss of information on a host
+     with 64 bit DOUBLEST.  */
+
+  doub = unpack_double (type, valaddr, &inv);
+  if (inv)
     {
-      /* 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;
+      fprintf_filtered (stream, "<invalid float value>");
+      return;
     }
-}
 
-/* 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.
+  /* FIXME: kettenis/2001-01-20: The following code makes too much
+     assumptions about the host and target floating point format.  */
+
+  /* NOTE: cagney/2002-02-03: Since the TYPE of what was passed in may
+     not necessarily be a TYPE_CODE_FLT, the below ignores that and
+     instead uses the type's length to determine the precision of the
+     floating-point value being printed.  */
 
-   If the data are a string pointer, returns the number of
-   sting characters printed.
+  if (len < sizeof (double))
+      fprintf_filtered (stream, "%.9g", (double) doub);
+  else if (len == sizeof (double))
+      fprintf_filtered (stream, "%.17g", (double) doub);
+  else
+#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
+}
 
-   if DEREF_REF is nonzero, then dereference references,
-   otherwise just print them like pointers.
+void
+print_decimal_floating (const gdb_byte *valaddr, struct type *type,
+                       struct ui_file *stream)
+{
+  char decstr[MAX_DECIMAL_STRING];
+  unsigned len = TYPE_LENGTH (type);
 
-   The PRETTY parameter controls prettyprinting.  */
+  decimal_to_string (valaddr, len, decstr);
+  fputs_filtered (decstr, stream);
+  return;
+}
 
-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;
+void
+print_binary_chars (struct ui_file *stream, const gdb_byte *valaddr,
+                   unsigned len, enum bfd_endian byte_order)
 {
-  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;
+#define BITS_IN_BYTES 8
 
-  check_stub_type (type);
-  
-  if (TYPE_FLAGS (type) & TYPE_FLAG_STUB)
-    {
-      fprintf_filtered (stream, "<unknown struct>");
-      fflush (stream);
-      return 0;
-    }
-  
-  switch (TYPE_CODE (type))
+  const gdb_byte *p;
+  unsigned int i;
+  int b;
+
+  /* Declared "int" so it will be signed.
+   * This ensures that right shift will shift in zeros.
+   */
+  const int mask = 0x080;
+
+  /* FIXME: We should be not printing leading zeroes in most cases.  */
+
+  if (byte_order == BFD_ENDIAN_BIG)
     {
-    case TYPE_CODE_ARRAY:
-      if (TYPE_LENGTH (type) >= 0
-         && TYPE_LENGTH (TYPE_TARGET_TYPE (type)) > 0)
+      for (p = valaddr;
+          p < valaddr + len;
+          p++)
        {
-         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
+         /* Every byte has 8 binary characters; peel off
+          * and print from the MSB end.
+          */
+         for (i = 0; i < (BITS_IN_BYTES * sizeof (*p)); i++)
            {
-             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;
+             if (*p & (mask >> i))
+               b = 1;
+             else
+               b = 0;
 
-    case TYPE_CODE_PTR:
-      if (format && format != 's')
-       {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
+             fprintf_filtered (stream, "%1d", b);
+           }
        }
-      if (TYPE_CODE (TYPE_TARGET_TYPE (type)) == TYPE_CODE_METHOD)
+    }
+  else
+    {
+      for (p = valaddr + len - 1;
+          p >= valaddr;
+          p--)
        {
-         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)
+         for (i = 0; i < (BITS_IN_BYTES * sizeof (*p)); i++)
            {
-             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);
+             if (*p & (mask >> i))
+               b = 1;
              else
-               type_print_method_args
-                 (TYPE_FN_FIELD_ARGS (f, j), "",
-                  TYPE_FN_FIELDLIST_NAME (domain, i), 0, stream);
-             break;
+               b = 0;
+
+             fprintf_filtered (stream, "%1d", b);
            }
-         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)
+    }
+}
+
+/* VALADDR points to an integer of LEN bytes.
+ * Print it in octal on stream or format it in buf.
+ */
+void
+print_octal_chars (struct ui_file *stream, const gdb_byte *valaddr,
+                  unsigned len, enum bfd_endian byte_order)
+{
+  const gdb_byte *p;
+  unsigned char octa1, octa2, octa3, carry;
+  int cycle;
+
+  /* FIXME: We should be not printing leading zeroes in most cases.  */
+
+
+  /* Octal is 3 bits, which doesn't fit.  Yuk.  So we have to track
+   * the extra bits, which cycle every three bytes:
+   *
+   * Byte side:       0            1             2          3
+   *                         |             |            |            |
+   * bit number   123 456 78 | 9 012 345 6 | 78 901 234 | 567 890 12 |
+   *
+   * Octal side:   0   1   carry  3   4  carry ...
+   *
+   * Cycle number:    0             1            2
+   *
+   * But of course we are printing from the high side, so we have to
+   * figure out where in the cycle we are so that we end up with no
+   * left over bits at the end.
+   */
+#define BITS_IN_OCTAL 3
+#define HIGH_ZERO     0340
+#define LOW_ZERO      0016
+#define CARRY_ZERO    0003
+#define HIGH_ONE      0200
+#define MID_ONE       0160
+#define LOW_ONE       0016
+#define CARRY_ONE     0001
+#define HIGH_TWO      0300
+#define MID_TWO       0070
+#define LOW_TWO       0007
+
+  /* For 32 we start in cycle 2, with two bits and one bit carry;
+   * for 64 in cycle in cycle 1, with one bit and a two bit carry.
+   */
+  cycle = (len * BITS_IN_BYTES) % BITS_IN_OCTAL;
+  carry = 0;
+
+  fputs_filtered ("0", stream);
+  if (byte_order == BFD_ENDIAN_BIG)
+    {
+      for (p = valaddr;
+          p < valaddr + len;
+          p++)
        {
-         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)
+         switch (cycle)
            {
-             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)");
+           case 0:
+             /* No carry in, carry out two bits.
+              */
+             octa1 = (HIGH_ZERO & *p) >> 5;
+             octa2 = (LOW_ZERO & *p) >> 2;
+             carry = (CARRY_ZERO & *p);
+             fprintf_filtered (stream, "%o", octa1);
+             fprintf_filtered (stream, "%o", octa2);
+             break;
+
+           case 1:
+             /* Carry in two bits, carry out one bit.
+              */
+             octa1 = (carry << 1) | ((HIGH_ONE & *p) >> 7);
+             octa2 = (MID_ONE & *p) >> 4;
+             octa3 = (LOW_ONE & *p) >> 1;
+             carry = (CARRY_ONE & *p);
+             fprintf_filtered (stream, "%o", octa1);
+             fprintf_filtered (stream, "%o", octa2);
+             fprintf_filtered (stream, "%o", octa3);
+             break;
+
+           case 2:
+             /* Carry in one bit, no carry out.
+              */
+             octa1 = (carry << 2) | ((HIGH_TWO & *p) >> 6);
+             octa2 = (MID_TWO & *p) >> 3;
+             octa3 = (LOW_TWO & *p);
+             carry = 0;
+             fprintf_filtered (stream, "%o", octa1);
+             fprintf_filtered (stream, "%o", octa2);
+             fprintf_filtered (stream, "%o", octa3);
              break;
+
+           default:
+             error (_("Internal error in octal conversion;"));
            }
-         fprintf_filtered (stream, "%d", val >> 3);
+
+         cycle++;
+         cycle = cycle % BITS_IN_OCTAL;
        }
-      else
+    }
+  else
+    {
+      for (p = valaddr + len - 1;
+          p >= valaddr;
+          p--)
        {
-         CORE_ADDR addr = (CORE_ADDR) unpack_long (type, valaddr);
-         elttype = TYPE_TARGET_TYPE (type);
-
-         if (TYPE_CODE (elttype) == TYPE_CODE_FUNC)
+         switch (cycle)
            {
-             /* Try to print what function it points to.  */
-             print_address_demangle (addr, stream, demangle);
-             /* Return value is irrelevant except for string pointers.  */
-             return 0;
-           }
+           case 0:
+             /* Carry out, no carry in */
+             octa1 = (HIGH_ZERO & *p) >> 5;
+             octa2 = (LOW_ZERO & *p) >> 2;
+             carry = (CARRY_ZERO & *p);
+             fprintf_filtered (stream, "%o", octa1);
+             fprintf_filtered (stream, "%o", octa2);
+             break;
 
-         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);
-               }
+           case 1:
+             /* Carry in, carry out */
+             octa1 = (carry << 1) | ((HIGH_ONE & *p) >> 7);
+             octa2 = (MID_ONE & *p) >> 4;
+             octa3 = (LOW_ONE & *p) >> 1;
+             carry = (CARRY_ONE & *p);
+             fprintf_filtered (stream, "%o", octa1);
+             fprintf_filtered (stream, "%o", octa2);
+             fprintf_filtered (stream, "%o", octa3);
+             break;
 
-             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]);
-                   }
-               }
+           case 2:
+             /* Carry in, no carry out */
+             octa1 = (carry << 2) | ((HIGH_TWO & *p) >> 6);
+             octa2 = (MID_TWO & *p) >> 3;
+             octa3 = (LOW_TWO & *p);
+             carry = 0;
+             fprintf_filtered (stream, "%o", octa1);
+             fprintf_filtered (stream, "%o", octa2);
+             fprintf_filtered (stream, "%o", octa3);
+             break;
 
-             fflush (stream);
+           default:
+             error (_("Internal error in octal conversion;"));
            }
-         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);
+         cycle++;
+         cycle = cycle % BITS_IN_OCTAL;
        }
-      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;
-       }
-      val_print_fields (type, valaddr, stream, format, recurse, pretty, 0);
-      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_BITPOS (type, i))
-           break;
-       }
-      if (i < len)
-       fputs_filtered (TYPE_FIELD_NAME (type, i), stream);
-      else
-       fprintf_filtered (stream, "%d", (int) val);
-      break;
+/* VALADDR points to an integer of LEN bytes.
+ * Print it in decimal on stream or format it in buf.
+ */
+void
+print_decimal_chars (struct ui_file *stream, const gdb_byte *valaddr,
+                    unsigned len, enum bfd_endian byte_order)
+{
+#define TEN             10
+#define CARRY_OUT(  x ) ((x) / TEN)    /* extend char to int */
+#define CARRY_LEFT( x ) ((x) % TEN)
+#define SHIFT( x )      ((x) << 4)
+#define LOW_NIBBLE(  x ) ( (x) & 0x00F)
+#define HIGH_NIBBLE( x ) (((x) & 0x0F0) >> 4)
+
+  const gdb_byte *p;
+  unsigned char *digits;
+  int carry;
+  int decimal_len;
+  int i, j, decimal_digits;
+  int dummy;
+  int flip;
+
+  /* Base-ten number is less than twice as many digits
+   * as the base 16 number, which is 2 digits per byte.
+   */
+  decimal_len = len * 2 * 2;
+  digits = xmalloc (decimal_len);
+
+  for (i = 0; i < decimal_len; i++)
+    {
+      digits[i] = 0;
+    }
 
-    case TYPE_CODE_FUNC:
-      if (format)
+  /* Ok, we have an unknown number of bytes of data to be printed in
+   * decimal.
+   *
+   * Given a hex number (in nibbles) as XYZ, we start by taking X and
+   * decemalizing it as "x1 x2" in two decimal nibbles.  Then we multiply
+   * the nibbles by 16, add Y and re-decimalize.  Repeat with Z.
+   *
+   * The trick is that "digits" holds a base-10 number, but sometimes
+   * the individual digits are > 10. 
+   *
+   * Outer loop is per nibble (hex digit) of input, from MSD end to
+   * LSD end.
+   */
+  decimal_digits = 0;          /* Number of decimal digits so far */
+  p = (byte_order == BFD_ENDIAN_BIG) ? valaddr : valaddr + len - 1;
+  flip = 0;
+  while ((byte_order == BFD_ENDIAN_BIG) ? (p < valaddr + len) : (p >= valaddr))
+    {
+      /*
+       * Multiply current base-ten number by 16 in place.
+       * Each digit was between 0 and 9, now is between
+       * 0 and 144.
+       */
+      for (j = 0; j < decimal_digits; j++)
        {
-         print_scalar_formatted (valaddr, type, format, 0, stream);
-         break;
+         digits[j] = SHIFT (digits[j]);
        }
-      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)
+      /* Take the next nibble off the input and add it to what
+       * we've got in the LSB position.  Bottom 'digit' is now
+       * between 0 and 159.
+       *
+       * "flip" is used to run this loop twice for each byte.
+       */
+      if (flip == 0)
        {
-         fprintf_filtered (stream, " '");
-         printchar ((unsigned char) unpack_long (type, valaddr), 
-                    stream, '\'');
-         fprintf_filtered (stream, "'");
+         /* Take top nibble.
+          */
+         digits[0] += HIGH_NIBBLE (*p);
+         flip = 1;
+       }
+      else
+       {
+         /* Take low nibble and bump our pointer "p".
+          */
+         digits[0] += LOW_NIBBLE (*p);
+          if (byte_order == BFD_ENDIAN_BIG)
+           p++;
+         else
+           p--;
+         flip = 0;
        }
-      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;
+      /* Re-decimalize.  We have to do this often enough
+       * that we don't overflow, but once per nibble is
+       * overkill.  Easier this way, though.  Note that the
+       * carry is often larger than 10 (e.g. max initial
+       * carry out of lowest nibble is 15, could bubble all
+       * the way up greater than 10).  So we have to do
+       * the carrying beyond the last current digit.
+       */
+      carry = 0;
+      for (j = 0; j < decimal_len - 1; j++)
+       {
+         digits[j] += carry;
 
-    case TYPE_CODE_ERROR:
-      fprintf_filtered (stream, "?");
-      break;
+         /* "/" won't handle an unsigned char with
+          * a value that if signed would be negative.
+          * So extend to longword int via "dummy".
+          */
+         dummy = digits[j];
+         carry = CARRY_OUT (dummy);
+         digits[j] = CARRY_LEFT (dummy);
 
-    default:
-      error ("Invalid type code in symbol table.");
+         if (j >= decimal_digits && carry == 0)
+           {
+             /*
+              * All higher digits are 0 and we
+              * no longer have a carry.
+              *
+              * Note: "j" is 0-based, "decimal_digits" is
+              *       1-based.
+              */
+             decimal_digits = j + 1;
+             break;
+           }
+       }
     }
-  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);
+  /* Ok, now "digits" is the decimal representation, with
+   * the "decimal_digits" actual digits.  Print!
+   */
+  for (i = decimal_digits - 1; i >= 0; i--)
+    {
+      fprintf_filtered (stream, "%1d", digits[i]);
+    }
+  xfree (digits);
 }
 
-/* LEVEL is the depth to indent lines by.  */
+/* VALADDR points to an integer of LEN bytes.  Print it in hex on stream.  */
 
 void
-type_print_1 (type, varstring, stream, show, level)
-     struct type *type;
-     char *varstring;
-     FILE *stream;
-     int show;
-     int level;
+print_hex_chars (struct ui_file *stream, const gdb_byte *valaddr,
+                unsigned len, enum bfd_endian byte_order)
 {
-  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);
-}
+  const gdb_byte *p;
 
-/* Print the method arguments ARGS to the file STREAM.  */
-static void
-type_print_method_args (args, prefix, varstring, staticp, stream)
-     struct type **args;
-     char *prefix, *varstring;
-     int staticp;
-     FILE *stream;
-{
-  int i;
+  /* FIXME: We should be not printing leading zeroes in most cases.  */
 
-  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)
+  fputs_filtered ("0x", stream);
+  if (byte_order == BFD_ENDIAN_BIG)
     {
-      i = !staticp;            /* skip the class variable */
-      while (1)
+      for (p = valaddr;
+          p < valaddr + len;
+          p++)
        {
-         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;
+         fprintf_filtered (stream, "%02x", *p);
+       }
+    }
+  else
+    {
+      for (p = valaddr + len - 1;
+          p >= valaddr;
+          p--)
+       {
+         fprintf_filtered (stream, "%02x", *p);
        }
     }
-  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;
+
+/* VALADDR points to a char integer of LEN bytes.  Print it out in appropriate language form on stream.  
+   Omit any leading zero chars.  */
+
+void
+print_char_chars (struct ui_file *stream, struct type *type,
+                 const gdb_byte *valaddr,
+                 unsigned len, enum bfd_endian byte_order)
 {
-  char *name;
-  int i, n_baseclasses = TYPE_N_BASECLASSES (type);
-  struct type *basetype = 0;
+  const gdb_byte *p;
 
-  while (type && n_baseclasses > 0)
+  if (byte_order == BFD_ENDIAN_BIG)
     {
-      /* Not actually sure about this one -- Bryan. */
-      check_stub_type (type);
-      
-      fprintf_filtered (stream, ": ");
-      for (i = 0; ;)
+      p = valaddr;
+      while (p < valaddr + len - 1 && *p == 0)
+       ++p;
+
+      while (p < valaddr + len)
        {
-         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, ", ");
+         LA_EMIT_CHAR (*p, type, stream, '\'');
+         ++p;
        }
+    }
+  else
+    {
+      p = valaddr + len - 1;
+      while (p > valaddr && *p == 0)
+       --p;
 
-      fprintf_filtered (stream, " ");
-      if (n_baseclasses != 1)
-       break;
-      n_baseclasses = TYPE_N_BASECLASSES (basetype);
-      type = basetype;
+      while (p >= valaddr)
+       {
+         LA_EMIT_CHAR (*p, type, stream, '\'');
+         --p;
+       }
     }
 }
 
-/* Print any asterisks or open-parentheses needed before the
-   variable name (to describe its type).
+/* Assuming TYPE is a simple, non-empty array type, compute its upper
+   and lower bound.  Save the low bound into LOW_BOUND if not NULL.
+   Save the high bound into HIGH_BOUND if not NULL.
+
+   Return 1 if the operation was successful. Return zero otherwise,
+   in which case the values of LOW_BOUND and HIGH_BOUNDS are unmodified.
+   
+   Computing the array upper and lower bounds is pretty easy, but this
+   function does some additional verifications before returning them.
+   If something incorrect is detected, it is better to return a status
+   rather than throwing an error, making it easier for the caller to
+   implement an error-recovery plan.  For instance, it may decide to
+   warn the user that the bounds were not found and then use some
+   default values instead.  */
 
-   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.  */
-
-static void
-type_print_varspec_prefix (type, stream, show, passed_a_ptr)
-     struct type *type;
-     FILE *stream;
-     int show;
-     int passed_a_ptr;
+int
+get_array_bounds (struct type *type, long *low_bound, long *high_bound)
 {
-  if (type == 0)
-    return;
-
-  if (TYPE_NAME (type) && show <= 0)
-    return;
+  struct type *index = TYPE_INDEX_TYPE (type);
+  long low = 0;
+  long high = 0;
+                                  
+  if (index == NULL)
+    return 0;
+
+  if (TYPE_CODE (index) == TYPE_CODE_RANGE)
+    {
+      low = TYPE_LOW_BOUND (index);
+      high = TYPE_HIGH_BOUND (index);
+    }
+  else if (TYPE_CODE (index) == TYPE_CODE_ENUM)
+    {
+      const int n_enums = TYPE_NFIELDS (index);
 
-  QUIT;
+      low = TYPE_FIELD_BITPOS (index, 0);
+      high = TYPE_FIELD_BITPOS (index, n_enums - 1);
+    }
+  else
+    return 0;
 
-  switch (TYPE_CODE (type))
-    {
-    case TYPE_CODE_PTR:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      fprintf_filtered (stream, "*");
-      break;
+  /* Abort if the lower bound is greater than the higher bound, except
+     when low = high + 1.  This is a very common idiom used in Ada when
+     defining empty ranges (for instance "range 1 .. 0").  */
+  if (low > high + 1)
+    return 0;
 
-    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;
+  if (low_bound)
+    *low_bound = low;
 
-    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;
+  if (high_bound)
+    *high_bound = high;
 
-    case TYPE_CODE_REF:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      fprintf_filtered (stream, "&");
-      break;
+  return 1;
+}
 
-    case TYPE_CODE_FUNC:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      if (passed_a_ptr)
-       fprintf_filtered (stream, "(");
-      break;
+/* Print on STREAM using the given OPTIONS the index for the element
+   at INDEX of an array whose index type is INDEX_TYPE.  */
+    
+void  
+maybe_print_array_index (struct type *index_type, LONGEST index,
+                         struct ui_file *stream,
+                        const struct value_print_options *options)
+{
+  struct value *index_value;
 
-    case TYPE_CODE_ARRAY:
-      type_print_varspec_prefix (TYPE_TARGET_TYPE (type), stream, 0,
-                                0);
-      if (passed_a_ptr)
-       fprintf_filtered (stream, "(");
+  if (!options->print_array_indexes)
+    return; 
+    
+  index_value = value_from_longest (index_type, index);
 
-    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;
-    }
+  LA_PRINT_ARRAY_INDEX (index_value, stream, options);
 }
 
-/* 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;
-{
-  if (type == 0)
-    return;
+/*  Called by various <lang>_val_print routines to print elements of an
+   array in the form "<elem1>, <elem2>, <elem3>, ...".
 
-  if (TYPE_NAME (type) && show <= 0)
-    return;
+   (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.
+ */
 
-  QUIT;
+void
+val_print_array_elements (struct type *type, const gdb_byte *valaddr,
+                         CORE_ADDR address, struct ui_file *stream,
+                         int recurse,
+                         const struct value_print_options *options,
+                         unsigned int i)
+{
+  unsigned int things_printed = 0;
+  unsigned len;
+  struct type *elttype, *index_type;
+  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;
+  long low_bound_index = 0;
+
+  elttype = TYPE_TARGET_TYPE (type);
+  eltlen = TYPE_LENGTH (check_typedef (elttype));
+  index_type = TYPE_INDEX_TYPE (type);
+
+  /* Compute the number of elements in the array.  On most arrays,
+     the size of its elements is not zero, and so the number of elements
+     is simply the size of the array divided by the size of the elements.
+     But for arrays of elements whose size is zero, we need to look at
+     the bounds.  */
+  if (eltlen != 0)
+    len = TYPE_LENGTH (type) / eltlen;
+  else
+    {
+      long low, hi;
+      if (get_array_bounds (type, &low, &hi))
+        len = hi - low + 1;
+      else
+        {
+          warning (_("unable to get bounds of array, assuming null array"));
+          len = 0;
+        }
+    }
 
-  switch (TYPE_CODE (type))
+  /* Get the array low bound.  This only makes sense if the array
+     has one or more element in it.  */
+  if (len > 0 && !get_array_bounds (type, &low_bound_index, NULL))
     {
-    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;
+      warning (_("unable to get low bound of array, using zero as default"));
+      low_bound_index = 0;
+    }
 
-    case TYPE_CODE_MEMBER:
-      if (passed_a_ptr)
-       fprintf_filtered (stream, ")");
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 0);
-      break;
+  annotate_array_section_begin (i, elttype);
 
-    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)
+  for (; i < len && things_printed < options->print_max; i++)
+    {
+      if (i != 0)
        {
-         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++)
+         if (options->prettyprint_arrays)
            {
-             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, ",\n");
+             print_spaces_filtered (2 + 2 * recurse, stream);
+           }
+         else
+           {
+             fprintf_filtered (stream, ", ");
            }
-         fprintf_filtered (stream, ")");
        }
-      break;
+      wrap_here (n_spaces (2 + 2 * recurse));
+      maybe_print_array_index (index_type, i + low_bound_index,
+                               stream, options);
 
-    case TYPE_CODE_PTR:
-    case TYPE_CODE_REF:
-      type_print_varspec_suffix (TYPE_TARGET_TYPE (type), stream, 0, 1);
-      break;
+      rep1 = i + 1;
+      reps = 1;
+      while ((rep1 < len) &&
+            !memcmp (valaddr + i * eltlen, valaddr + rep1 * eltlen, eltlen))
+       {
+         ++reps;
+         ++rep1;
+       }
 
-    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, "()");
-      break;
+      if (reps > options->repeat_count_threshold)
+       {
+         val_print (elttype, valaddr + i * eltlen, 0, address + i * eltlen,
+                    stream, recurse + 1, options, current_language);
+         annotate_elt_rep (reps);
+         fprintf_filtered (stream, " <repeats %u times>", reps);
+         annotate_elt_rep_end ();
 
-    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.  */
-      break;
+         i = rep1 - 1;
+         things_printed += options->repeat_count_threshold;
+       }
+      else
+       {
+         val_print (elttype, valaddr + i * eltlen, 0, address + i * eltlen,
+                    stream, recurse + 1, options, current_language);
+         annotate_elt ();
+         things_printed++;
+       }
+    }
+  annotate_array_section_end ();
+  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.
+/* Read LEN bytes of target memory at address MEMADDR, placing the
+   results in GDB's memory at MYADDR.  Returns a count of the bytes
+   actually read, and optionally an errno value in the location
+   pointed to by ERRNOPTR if ERRNOPTR is non-null. */
 
-   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.
+/* FIXME: cagney/1999-10-14: Only used by val_print_string.  Can this
+   function be eliminated.  */
 
-   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;
-     FILE *stream;
-     int show;
-     int level;
+static int
+partial_memory_read (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int *errnoptr)
 {
-  char *name;
-  register int i;
-  register int len;
-  register int lastval;
-
-  QUIT;
+  int nread;                   /* Number of bytes actually read. */
+  int errcode;                 /* Error from last read. */
 
-  if (type == 0)
+  /* First try a complete read. */
+  errcode = target_read_memory (memaddr, myaddr, len);
+  if (errcode == 0)
     {
-      fprintf_filtered (stream, "type unknown");
-      return;
+      /* Got it all. */
+      nread = len;
     }
-
-  if (TYPE_NAME (type) && show <= 0)
+  else
     {
-      fputs_filtered (TYPE_NAME (type), stream);
-      return;
+      /* Loop, reading one byte at a time until we get as much as we can. */
+      for (errcode = 0, nread = 0; len > 0 && errcode == 0; nread++, len--)
+       {
+         errcode = target_read_memory (memaddr++, myaddr++, 1);
+       }
+      /* If an error, the last read was unsuccessful, so adjust count. */
+      if (errcode != 0)
+       {
+         nread--;
+       }
+    }
+  if (errnoptr != NULL)
+    {
+      *errnoptr = errcode;
     }
+  return (nread);
+}
 
-  switch (TYPE_CODE (type))
+/* Read a string from the inferior, at ADDR, with LEN characters of WIDTH bytes
+   each.  Fetch at most FETCHLIMIT characters.  BUFFER will be set to a newly
+   allocated buffer containing the string, which the caller is responsible to
+   free, and BYTES_READ will be set to the number of bytes read.  Returns 0 on
+   success, or errno on failure.
+
+   If LEN > 0, reads exactly LEN characters (including eventual NULs in
+   the middle or end of the string).  If LEN is -1, stops at the first
+   null character (not necessarily the first null byte) up to a maximum
+   of FETCHLIMIT characters.  Set FETCHLIMIT to UINT_MAX to read as many
+   characters as possible from the string.
+
+   Unless an exception is thrown, BUFFER will always be allocated, even on
+   failure.  In this case, some characters might have been read before the
+   failure happened.  Check BYTES_READ to recognize this situation.
+
+   Note: There was a FIXME asking to make this code use target_read_string,
+   but this function is more general (can read past null characters, up to
+   given LEN). Besides, it is used much more often than target_read_string
+   so it is more tested.  Perhaps callers of target_read_string should use
+   this function instead?  */
+
+int
+read_string (CORE_ADDR addr, int len, int width, unsigned int fetchlimit,
+            gdb_byte **buffer, int *bytes_read)
+{
+  int found_nul;               /* Non-zero if we found the nul char.  */
+  int errcode;                 /* Errno returned from bad reads.  */
+  unsigned int nfetch;         /* Chars to fetch / chars fetched.  */
+  unsigned int chunksize;      /* Size of each fetch, in chars.  */
+  gdb_byte *bufptr;            /* Pointer to next available byte in buffer.  */
+  gdb_byte *limit;             /* First location past end of fetch buffer.  */
+  struct cleanup *old_chain = NULL;    /* Top of the old cleanup chain.  */
+
+  /* Decide how large of chunks to try to read in one operation.  This
+     is also pretty simple.  If LEN >= zero, then we want fetchlimit chars,
+     so we might as well read them all in one operation.  If LEN is -1, we
+     are looking for a NUL 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 == -1 ? min (8, fetchlimit) : fetchlimit);
+
+  /* Loop until we either have all the characters, or we encounter
+     some error, such as bumping into the end of the address space.  */
+
+  found_nul = 0;
+  *buffer = NULL;
+
+  old_chain = make_cleanup (free_current_contents, buffer);
+
+  if (len > 0)
     {
-    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;
+      *buffer = (gdb_byte *) xmalloc (len * width);
+      bufptr = *buffer;
 
-    case TYPE_CODE_STRUCT:
-      fprintf_filtered (stream, "struct ");
-      goto struct_union;
+      nfetch = partial_memory_read (addr, bufptr, len * width, &errcode)
+       / width;
+      addr += nfetch * width;
+      bufptr += nfetch * width;
+    }
+  else if (len == -1)
+    {
+      unsigned long bufsize = 0;
 
-    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
+      do
        {
-         check_stub_type (type);
-         
-         type_print_derivation_info (stream, type);
-         
-         fprintf_filtered (stream, "{");
-         len = TYPE_NFIELDS (type);
-         if (len)
-           fprintf_filtered (stream, "\n");
+         QUIT;
+         nfetch = min (chunksize, fetchlimit - bufsize);
+
+         if (*buffer == NULL)
+           *buffer = (gdb_byte *) xmalloc (nfetch * width);
          else
-           {
-             if (TYPE_FLAGS (type) & TYPE_FLAG_STUB)
-               fprintf_filtered (stream, "<incomplete type>\n");
-             else
-               fprintf_filtered (stream, "<no data fields>\n");
-           }
+           *buffer = (gdb_byte *) xrealloc (*buffer,
+                                            (nfetch + bufsize) * width);
 
-         /* 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");
-           }
+         bufptr = *buffer + bufsize * width;
+         bufsize += nfetch;
+
+         /* Read as much as we can.  */
+         nfetch = partial_memory_read (addr, bufptr, nfetch * width, &errcode)
+                   / width;
+
+         /* Scan this chunk for the null character 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 character, or at the next character after the end
+            of the buffer.  */
 
-         /* C++: print out the methods */
-         len = TYPE_NFN_FIELDS (type);
-         if (len) fprintf_filtered (stream, "\n");
-         for (i = 0; i < len; i++)
+         limit = bufptr + nfetch * width;
+         while (bufptr < limit)
            {
-             struct fn_field *f = TYPE_FN_FIELDLIST1 (type, i);
-             int j, len2 = TYPE_FN_FIELDLIST_LENGTH (type, i);
+             unsigned long c;
 
-             for (j = 0; j < len2; j++)
+             c = extract_unsigned_integer (bufptr, width);
+             addr += width;
+             bufptr += width;
+             if (c == 0)
                {
-                 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");
+                 /* We don't care about any error which happened after
+                    the NUL terminator.  */
+                 errcode = 0;
+                 found_nul = 1;
+                 break;
                }
            }
-
-         print_spaces_filtered (level, stream);
-         fprintf_filtered (stream, "}");
        }
-      break;
+      while (errcode == 0      /* no error */
+            && bufptr - *buffer < fetchlimit * width   /* no overrun */
+            && !found_nul);    /* haven't found NUL yet */
+    }
+  else
+    {                          /* Length of string is really 0!  */
+      /* We always allocate *buffer.  */
+      *buffer = bufptr = xmalloc (1);
+      errcode = 0;
+    }
+
+  /* bufptr and addr now point immediately beyond the last byte which we
+     consider part of the string (including a '\0' which ends the string).  */
+  *bytes_read = bufptr - *buffer;
+
+  QUIT;
+
+  discard_cleanups (old_chain);
+
+  return errcode;
+}
+
+/* Print a string from the inferior, starting at ADDR and printing up to LEN
+   characters, of WIDTH bytes a piece, to STREAM.  If LEN is -1, 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.  */
+
+int
+val_print_string (struct type *elttype, CORE_ADDR addr, int len,
+                 struct ui_file *stream,
+                 const struct value_print_options *options)
+{
+  int force_ellipsis = 0;      /* Force ellipsis to be printed if nonzero.  */
+  int errcode;                 /* Errno returned from bad reads.  */
+  int found_nul;               /* Non-zero if we found the nul char */
+  unsigned int fetchlimit;     /* Maximum number of chars to print.  */
+  int bytes_read;
+  gdb_byte *buffer = NULL;     /* Dynamically growable fetch buffer.  */
+  struct cleanup *old_chain = NULL;    /* Top of the old cleanup chain.  */
+  int width = TYPE_LENGTH (elttype);
+
+  /* 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 >= zero, then the limit is the minimum of LEN and print_max.  If
+     LEN is -1, 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 == -1 ? options->print_max : min (len, options->print_max));
+
+  errcode = read_string (addr, len, width, fetchlimit, &buffer, &bytes_read);
+  old_chain = make_cleanup (xfree, buffer);
+
+  addr += bytes_read;
+
+  /* We now have either successfully filled the buffer to fetchlimit, or
+     terminated early due to an error or finding a null char when LEN is -1.  */
+
+  /* Determine found_nul by looking at the last character read.  */
+  found_nul = extract_unsigned_integer (buffer + bytes_read - width, width) == 0;
+
+  if (len == -1 && !found_nul)
+    {
+      gdb_byte *peekbuf;
+
+      /* We didn't find a NUL 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.  */
+
+      peekbuf = (gdb_byte *) alloca (width);
+
+      if (target_read_memory (addr, peekbuf, width) == 0
+         && extract_unsigned_integer (peekbuf, width) != 0)
+       force_ellipsis = 1;
+    }
+  else if ((len >= 0 && errcode != 0) || (len > bytes_read / width))
+    {
+      /* 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;
+    }
 
-    case TYPE_CODE_ENUM:
-      fprintf_filtered (stream, "enum ");
-      if (name = type_name_no_tag (type))
+  /* 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 || bytes_read > 0)
+    {
+      if (options->addressprint)
        {
-         fputs_filtered (name, stream);
          fputs_filtered (" ", stream);
        }
-      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, "}");
-       }
-      break;
+      LA_PRINT_STRING (stream, elttype, buffer, bytes_read / width, force_ellipsis, options);
+    }
 
-    case TYPE_CODE_INT:
-      if (TYPE_LENGTH (type) > sizeof (LONGEST))
+  if (errcode != 0)
+    {
+      if (errcode == EIO)
        {
-         fprintf_filtered (stream, "<%d bit integer>",
-                           TYPE_LENGTH (type) * TARGET_CHAR_BIT);
+         fprintf_filtered (stream, " <Address ");
+         fputs_filtered (paddress (addr), stream);
+         fprintf_filtered (stream, " out of bounds>");
        }
       else
        {
-         if (TYPE_UNSIGNED (type))
-           name = unsigned_type_table[TYPE_LENGTH (type)];
-         else
-           name = signed_type_table[TYPE_LENGTH (type)];
+         fprintf_filtered (stream, " <Error reading address ");
+         fputs_filtered (paddress (addr), stream);
+         fprintf_filtered (stream, ": %s>", safe_strerror (errcode));
        }
-      fputs_filtered (name, stream);
-      break;
+    }
 
-    case TYPE_CODE_FLT:
-      name = float_type_table[TYPE_LENGTH (type)];
-      fputs_filtered (name, stream);
-      break;
+  gdb_flush (stream);
+  do_cleanups (old_chain);
 
-    case TYPE_CODE_VOID:
-      fprintf_filtered (stream, "void");
-      break;
+  return (bytes_read / width);
+}
+\f
 
-    case 0:
-      fprintf_filtered (stream, "struct unknown");
-      break;
+/* The 'set input-radix' command writes to this auxiliary variable.
+   If the requested radix is valid, INPUT_RADIX is updated; otherwise,
+   it is left unchanged.  */
 
-    case TYPE_CODE_ERROR:
-      fprintf_filtered (stream, "<unknown type>");
-      break;
+static unsigned input_radix_1 = 10;
 
-    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!  */
 
 static void
-set_input_radix (args, from_tty, c)
-     char *args;
-     int from_tty;
-     struct cmd_list_element *c;
+set_input_radix (char *args, int from_tty, struct cmd_list_element *c)
 {
-  unsigned radix = *(unsigned *)c->var;
+  set_input_radix_1 (from_tty, input_radix_1);
+}
 
+static void
+set_input_radix_1 (int from_tty, unsigned radix)
+{
+  /* 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)
+    {
+      input_radix_1 = input_radix;
+      error (_("Nonsense input radix ``decimal %u''; input radix unchanged."),
+            radix);
+    }
+  input_radix_1 = input_radix = radix;
   if (from_tty)
-    printf_filtered ("Input radix set to decimal %d, hex %x, octal %o\n",
-       radix, radix, radix);
+    {
+      printf_filtered (_("Input radix now set to decimal %u, hex %x, octal %o.\n"),
+                      radix, radix, radix);
+    }
 }
 
+/* The 'set output-radix' command writes to this auxiliary variable.
+   If the requested radix is valid, OUTPUT_RADIX is updated,
+   otherwise, it is left unchanged.  */
+
+static unsigned output_radix_1 = 10;
+
 static void
-set_output_radix (args, from_tty, c)
-     char *args;
-     int from_tty;
-     struct cmd_list_element *c;
+set_output_radix (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);
+  set_output_radix_1 (from_tty, output_radix_1);
+}
 
-  /* FIXME, we really should be able to validate the setting BEFORE
-     it takes effect.  */
+static void
+set_output_radix_1 (int from_tty, unsigned radix)
+{
+  /* Validate the radix and disallow ones that we aren't prepared to
+     handle correctly, leaving the radix unchanged. */
   switch (radix)
     {
     case 16:
-      output_format = 'x';
+      user_print_options.output_format = 'x';  /* hex */
       break;
     case 10:
-      output_format = 0;
+      user_print_options.output_format = 0;    /* decimal */
       break;
     case 8:
-      output_format = 'o';             /* octal */
+      user_print_options.output_format = 'o';  /* octal */
       break;
     default:
-      output_format = 0;
-      error ("Unsupported radix ``decimal %d''; using decimal output",
-             radix);
+      output_radix_1 = output_radix;
+      error (_("Unsupported output radix ``decimal %u''; output radix unchanged."),
+            radix);
+    }
+  output_radix_1 = output_radix = radix;
+  if (from_tty)
+    {
+      printf_filtered (_("Output radix now set to decimal %u, hex %x, octal %o.\n"),
+                      radix, radix, radix);
     }
 }
 
-/* Both at once */
+/* 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.
+
+   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
-set_radix (arg, from_tty, c)
-     char *arg;
-     int from_tty;
-     struct cmd_list_element *c;
+set_radix (char *arg, int from_tty)
 {
-  unsigned radix = *(unsigned *)c->var;
+  unsigned radix;
 
+  radix = (arg == NULL) ? 10 : parse_and_eval_long (arg);
+  set_output_radix_1 (0, radix);
+  set_input_radix_1 (0, radix);
   if (from_tty)
-    printf_filtered ("Radix set to decimal %d, hex %x, octal %o\n",
-       radix, radix, radix);
+    {
+      printf_filtered (_("Input and output radices now set to decimal %u, hex %x, octal %o.\n"),
+                      radix, radix, radix);
+    }
+}
 
-  input_radix = radix;
-  output_radix = radix;
+/* Show both the input and output radices. */
 
-  set_output_radix (arg, 0, c);
+static void
+show_radix (char *arg, int from_tty)
+{
+  if (from_tty)
+    {
+      if (input_radix == output_radix)
+       {
+         printf_filtered (_("Input and output radices set to decimal %u, hex %x, octal %o.\n"),
+                          input_radix, input_radix, input_radix);
+       }
+      else
+       {
+         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);
+       }
+    }
 }
 \f
-struct cmd_list_element *setprintlist = NULL;
-struct cmd_list_element *showprintlist = NULL;
 
-/*ARGSUSED*/
 static void
-set_print (arg, from_tty)
-     char *arg;
-     int from_tty;
+set_print (char *arg, int from_tty)
 {
-  printf (
-"\"set print\" must be followed by the name of a print subcommand.\n");
-  help_list (setprintlist, "set print ", -1, stdout);
+  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
-show_print (args, from_tty)
-     char *args;
-     int from_tty;
+show_print (char *args, int from_tty)
 {
   cmd_show_list (showprintlist, from_tty, "");
 }
 \f
 void
-_initialize_valprint ()
+_initialize_valprint (void)
 {
   struct cmd_list_element *c;
 
   add_prefix_cmd ("print", no_class, set_print,
-                 "Generic command for setting how things 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.",
+                 _("Generic command for showing print settings."),
                  &showprintlist, "show print ", 0, &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 ("pretty", class_support, var_boolean, (char *)&prettyprint,
-                 "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 ("vtbl", class_support, var_boolean, (char *)&vtblprint,
-                 "Set printing of C++ virtual function tables.",
-                 &setprintlist),
-     &showprintlist);
-
-  add_show_from_set
-    (add_set_cmd ("array", class_support, var_boolean, (char *)&arrayprint,
-                 "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 = set_radix;
-
-  /* Give people the defaults which they are used to.  */
-  prettyprint = 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 *));
+  add_alias_cmd ("p", "print", no_class, 1, &showlist);
+  add_alias_cmd ("pr", "print", no_class, 1, &showlist);
+
+  add_setshow_uinteger_cmd ("elements", no_class,
+                           &user_print_options.print_max, _("\
+Set limit on string chars or array elements to print."), _("\
+Show limit on string chars or array elements to print."), _("\
+\"set print elements 0\" causes there to be no limit."),
+                           NULL,
+                           show_print_max,
+                           &setprintlist, &showprintlist);
+
+  add_setshow_boolean_cmd ("null-stop", no_class,
+                          &user_print_options.stop_print_at_null, _("\
+Set printing of char arrays to stop at first null char."), _("\
+Show printing of char arrays to stop at first null char."), NULL,
+                          NULL,
+                          show_stop_print_at_null,
+                          &setprintlist, &showprintlist);
+
+  add_setshow_uinteger_cmd ("repeats", no_class,
+                           &user_print_options.repeat_count_threshold, _("\
+Set threshold for repeated print elements."), _("\
+Show threshold for repeated print elements."), _("\
+\"set print repeats 0\" causes all elements to be individually printed."),
+                           NULL,
+                           show_repeat_count_threshold,
+                           &setprintlist, &showprintlist);
+
+  add_setshow_boolean_cmd ("pretty", class_support,
+                          &user_print_options.prettyprint_structs, _("\
+Set prettyprinting of structures."), _("\
+Show prettyprinting of structures."), NULL,
+                          NULL,
+                          show_prettyprint_structs,
+                          &setprintlist, &showprintlist);
+
+  add_setshow_boolean_cmd ("union", class_support,
+                          &user_print_options.unionprint, _("\
+Set printing of unions interior to structures."), _("\
+Show printing of unions interior to structures."), NULL,
+                          NULL,
+                          show_unionprint,
+                          &setprintlist, &showprintlist);
+
+  add_setshow_boolean_cmd ("array", class_support,
+                          &user_print_options.prettyprint_arrays, _("\
+Set prettyprinting of arrays."), _("\
+Show prettyprinting of arrays."), NULL,
+                          NULL,
+                          show_prettyprint_arrays,
+                          &setprintlist, &showprintlist);
+
+  add_setshow_boolean_cmd ("address", class_support,
+                          &user_print_options.addressprint, _("\
+Set printing of addresses."), _("\
+Show printing of addresses."), NULL,
+                          NULL,
+                          show_addressprint,
+                          &setprintlist, &showprintlist);
+
+  add_setshow_zuinteger_cmd ("input-radix", class_support, &input_radix_1,
+                            _("\
+Set default input radix for entering numbers."), _("\
+Show default input radix for entering numbers."), NULL,
+                            set_input_radix,
+                            show_input_radix,
+                            &setlist, &showlist);
+
+  add_setshow_zuinteger_cmd ("output-radix", class_support, &output_radix_1,
+                            _("\
+Set default output radix for printing of values."), _("\
+Show default output radix for printing of values."), NULL,
+                            set_output_radix,
+                            show_output_radix,
+                            &setlist, &showlist);
+
+  /* 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 deprecated_add_set_cmd() and [deleted]
+     add_show_from_set() commands aren't really appropriate. */
+  /* FIXME: i18n: With the new add_setshow_integer command, that is no
+     longer true - show can display anything.  */
+  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);
+
+  add_setshow_boolean_cmd ("array-indexes", class_support,
+                           &user_print_options.print_array_indexes, _("\
+Set printing of array indexes."), _("\
+Show printing of array indexes"), NULL, NULL, show_print_array_indexes,
+                           &setprintlist, &showprintlist);
 }
This page took 0.061092 seconds and 4 git commands to generate.