2005-01-28 Andrew Cagney <cagney@gnu.org>
[deliverable/binutils-gdb.git] / gdb / cp-valprint.c
index 58bbc7ca390677afc0fbe3175324473bab5d05f4..a4aef464a9f9f49ecca3c620d719cefa6edf0473 100644 (file)
@@ -1,6 +1,7 @@
 /* Support for printing C++ values for GDB, the GNU debugger.
-   Copyright 1986, 1988, 1989, 1991, 1994-1996, 2000
-   Free Software Foundation, Inc.
+
+   Copyright 1986, 1988, 1989, 1991, 1992, 1993, 1994, 1995, 1996,
+   1997, 2000, 2001, 2002, 2003, 2005 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -20,7 +21,7 @@
    Boston, MA 02111-1307, USA.  */
 
 #include "defs.h"
-#include "obstack.h"
+#include "gdb_obstack.h"
 #include "symtab.h"
 #include "gdbtypes.h"
 #include "expression.h"
 #include "gdb_string.h"
 #include "c-lang.h"
 #include "target.h"
-
-/* Indication of presence of HP-compiled object files */
-extern int hp_som_som_object_present;  /* defined in symtab.c */
-
+#include "cp-abi.h"
+#include "valprint.h"
 
 int vtblprint;                 /* Controls printing of vtbl's */
 int objectprint;               /* Controls looking up an object's derived type
@@ -47,26 +46,25 @@ static struct obstack dont_print_statmem_obstack;
 
 extern void _initialize_cp_valprint (void);
 
-static void cp_print_static_field (struct type *, value_ptr,
+static void cp_print_static_field (struct type *, struct value *,
                                   struct ui_file *, int, int,
                                   enum val_prettyprint);
 
-static void cp_print_value (struct type *, struct type *, char *, int,
-                           CORE_ADDR, struct ui_file *, int, int,
+static void cp_print_value (struct type *, struct type *, const bfd_byte *,
+                           int, CORE_ADDR, struct ui_file *, int, int,
                            enum val_prettyprint, struct type **);
 
 static void cp_print_hpacc_virtual_table_entries (struct type *, int *,
-                                                 value_ptr,
+                                                 struct value *,
                                                  struct ui_file *, int,
                                                  int,
                                                  enum val_prettyprint);
 
 
 void
-cp_print_class_method (valaddr, type, stream)
-     char *valaddr;
-     struct type *type;
-     struct ui_file *stream;
+cp_print_class_method (const bfd_byte *valaddr,
+                      struct type *type,
+                      struct ui_file *stream)
 {
   struct type *domain;
   struct fn_field *f = NULL;
@@ -86,7 +84,7 @@ cp_print_class_method (valaddr, type, stream)
       fprintf_filtered (stream, "<unknown>");
       return;
     }
-  addr = unpack_pointer (lookup_pointer_type (builtin_type_void), valaddr);
+  addr = unpack_pointer (type, valaddr);
   if (METHOD_PTR_IS_VIRTUAL (addr))
     {
       offset = METHOD_PTR_TO_VOFFSET (addr);
@@ -96,13 +94,11 @@ cp_print_class_method (valaddr, type, stream)
          f = TYPE_FN_FIELDLIST1 (domain, i);
          len2 = TYPE_FN_FIELDLIST_LENGTH (domain, i);
 
+         check_stub_method_group (domain, i);
          for (j = 0; j < len2; j++)
            {
-             QUIT;
              if (TYPE_FN_FIELD_VOFFSET (f, j) == offset)
                {
-                 if (TYPE_FN_FIELD_STUB (f, j))
-                   check_stub_method (domain, i, j);
                  kind = "virtual ";
                  goto common;
                }
@@ -115,7 +111,7 @@ cp_print_class_method (valaddr, type, stream)
       if (sym == 0)
        {
          /* 1997-08-01 Currently unsupported with HP aCC */
-         if (hp_som_som_object_present)
+         if (deprecated_hp_som_som_object_present)
            {
              fputs_filtered ("?? <not supported with HP aCC>", stream);
              return;
@@ -128,25 +124,22 @@ cp_print_class_method (valaddr, type, stream)
          f = TYPE_FN_FIELDLIST1 (domain, i);
          len2 = TYPE_FN_FIELDLIST_LENGTH (domain, i);
 
+         check_stub_method_group (domain, i);
          for (j = 0; j < len2; j++)
            {
-             QUIT;
-             if (TYPE_FN_FIELD_STUB (f, j))
-               check_stub_method (domain, i, j);
-             if (STREQ (SYMBOL_NAME (sym), TYPE_FN_FIELD_PHYSNAME (f, j)))
-               {
-                 goto common;
-               }
+             if (strcmp (DEPRECATED_SYMBOL_NAME (sym), TYPE_FN_FIELD_PHYSNAME (f, j))
+                 == 0)
+               goto common;
            }
        }
     }
-common:
+ common:
   if (i < len)
     {
       char *demangled_name;
 
       fprintf_filtered (stream, "&");
-      fprintf_filtered (stream, kind);
+      fputs_filtered (kind, stream);
       demangled_name = cplus_demangle (TYPE_FN_FIELD_PHYSNAME (f, j),
                                       DMGL_ANSI | DMGL_PARAMS);
       if (demangled_name == NULL)
@@ -155,7 +148,7 @@ common:
       else
        {
          fputs_filtered (demangled_name, stream);
-         free (demangled_name);
+         xfree (demangled_name);
        }
     }
   else
@@ -166,41 +159,32 @@ common:
     }
 }
 
-/* This was what it was for gcc 2.4.5 and earlier.  */
-static const char vtbl_ptr_name_old[] =
-{CPLUS_MARKER, 'v', 't', 'b', 'l', '_', 'p', 't', 'r', '_', 't', 'y', 'p', 'e', 0};
-/* It was changed to this after 2.4.5.  */
-const char vtbl_ptr_name[] =
-{'_', '_', 'v', 't', 'b', 'l', '_', 'p', 't', 'r', '_', 't', 'y', 'p', 'e', 0};
-
-/* HP aCC uses different names */
-const char hpacc_vtbl_ptr_name[] =
-{'_', '_', 'v', 'f', 'p', 0};
-const char hpacc_vtbl_ptr_type_name[] =
-{'_', '_', 'v', 'f', 't', 'y', 'p', 0};
+/* GCC versions after 2.4.5 use this.  */
+const char vtbl_ptr_name[] = "__vtbl_ptr_type";
 
+/* HP aCC uses different names.  */
+const char hpacc_vtbl_ptr_name[] = "__vfp";
+const char hpacc_vtbl_ptr_type_name[] = "__vftyp";
 
 /* Return truth value for assertion that TYPE is of the type
    "pointer to virtual function".  */
 
 int
-cp_is_vtbl_ptr_type (type)
-     struct type *type;
+cp_is_vtbl_ptr_type (struct type *type)
 {
   char *typename = type_name_no_tag (type);
 
-  return (typename != NULL
-         && (STREQ (typename, vtbl_ptr_name)
-             || STREQ (typename, vtbl_ptr_name_old)));
+  return (typename != NULL && !strcmp (typename, vtbl_ptr_name));
 }
 
 /* Return truth value for the assertion that TYPE is of the type
    "pointer to virtual function table".  */
 
 int
-cp_is_vtbl_member (type)
-     struct type *type;
+cp_is_vtbl_member (struct type *type)
 {
+  /* With older versions of g++, the vtbl field pointed to an array
+     of structures.  Nowadays it points directly to the structure. */
   if (TYPE_CODE (type) == TYPE_CODE_PTR)
     {
       type = TYPE_TARGET_TYPE (type);
@@ -215,6 +199,17 @@ cp_is_vtbl_member (type)
              return cp_is_vtbl_ptr_type (type);
            }
        }
+      else if (TYPE_CODE (type) == TYPE_CODE_STRUCT)  /* if not using thunks */
+       {
+         return cp_is_vtbl_ptr_type (type);
+       }
+      else if (TYPE_CODE (type) == TYPE_CODE_PTR)     /* if using thunks */
+       {
+         /* The type name of the thunk pointer is NULL when using dwarf2.
+            We could test for a pointer to a function, but there is
+            no type info for the virtual table either, so it wont help.  */
+         return cp_is_vtbl_ptr_type (type);
+       }
     }
   return 0;
 }
@@ -232,19 +227,11 @@ cp_is_vtbl_member (type)
    should not print, or zero if called from top level.  */
 
 void
-cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format, recurse, pretty,
-                      dont_print_vb, dont_print_statmem)
-     struct type *type;
-     struct type *real_type;
-     char *valaddr;
-     int offset;
-     CORE_ADDR address;
-     struct ui_file *stream;
-     int format;
-     int recurse;
-     enum val_prettyprint pretty;
-     struct type **dont_print_vb;
-     int dont_print_statmem;
+cp_print_value_fields (struct type *type, struct type *real_type,
+                      const bfd_byte *valaddr, int offset, CORE_ADDR address,
+                      struct ui_file *stream, int format, int recurse,
+                      enum val_prettyprint pretty,
+                      struct type **dont_print_vb,int dont_print_statmem)
 {
   int i, len, n_baseclasses;
   struct obstack tmp_obstack;
@@ -268,16 +255,15 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
 
   /* If there are no data fields, or if the only field is the
    * vtbl pointer, skip this part */
-  if ((len == n_baseclasses) ||
-      ((len - n_baseclasses == 1) &&
-       TYPE_HAS_VTABLE (type) &&
-  STREQN (TYPE_FIELD_NAME (type, n_baseclasses), hpacc_vtbl_ptr_name, 5)) ||
-      !len)
+  if ((len == n_baseclasses)
+      || ((len - n_baseclasses == 1)
+         && TYPE_HAS_VTABLE (type)
+         && strncmp (TYPE_FIELD_NAME (type, n_baseclasses),
+                     hpacc_vtbl_ptr_name, 5) == 0)
+      || !len)
     fprintf_filtered (stream, "<No data fields>");
   else
     {
-      extern int inspect_it;
-
       if (dont_print_statmem == 0)
        {
          /* If we're at top level, carve out a completely fresh
@@ -294,7 +280,9 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
            continue;
 
          /* If a vtable pointer appears, we'll print it out later */
-         if (TYPE_HAS_VTABLE (type) && STREQN (TYPE_FIELD_NAME (type, i), hpacc_vtbl_ptr_name, 5))
+         if (TYPE_HAS_VTABLE (type)
+             && strncmp (TYPE_FIELD_NAME (type, i), hpacc_vtbl_ptr_name,
+                         5) == 0)
            continue;
 
          if (fields_seen)
@@ -356,7 +344,7 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
 
          if (!TYPE_FIELD_STATIC (type, i) && TYPE_FIELD_PACKED (type, i))
            {
-             value_ptr v;
+             struct value *v;
 
              /* Bitfields require special handling, especially due to byte
                 order problems.  */
@@ -366,11 +354,12 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
                }
              else
                {
-                 v = value_from_longest (TYPE_FIELD_TYPE (type, i),
-                         unpack_field_as_long (type, valaddr + offset, i));
+                 v = value_from_longest
+                   (TYPE_FIELD_TYPE (type, i), 
+                    unpack_field_as_long (type, valaddr + offset, i));
 
-                 val_print (TYPE_FIELD_TYPE (type, i), VALUE_CONTENTS (v), 0, 0,
-                            stream, format, 0, recurse + 1, pretty);
+                 val_print (TYPE_FIELD_TYPE (type, i), VALUE_CONTENTS (v),
+                            0, 0, stream, format, 0, recurse + 1, pretty);
                }
            }
          else
@@ -381,7 +370,7 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
                }
              else if (TYPE_FIELD_STATIC (type, i))
                {
-                 value_ptr v = value_static_field (type, i);
+                 struct value *v = value_static_field (type, i);
                  if (v == NULL)
                    fputs_filtered ("<optimized out>", stream);
                  else
@@ -392,7 +381,7 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
              else
                {
                  val_print (TYPE_FIELD_TYPE (type, i),
-                         valaddr, offset + TYPE_FIELD_BITPOS (type, i) / 8,
+                            valaddr, offset + TYPE_FIELD_BITPOS (type, i) / 8,
                             address + TYPE_FIELD_BITPOS (type, i) / 8,
                             stream, format, 0, recurse + 1, pretty);
                }
@@ -415,9 +404,11 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
        }
     }                          /* if there are data fields */
   /* Now print out the virtual table pointer if there is one */
-  if (TYPE_HAS_VTABLE (type) && STREQN (TYPE_FIELD_NAME (type, n_baseclasses), hpacc_vtbl_ptr_name, 5))
+  if (TYPE_HAS_VTABLE (type)
+      && strncmp (TYPE_FIELD_NAME (type, n_baseclasses),
+                 hpacc_vtbl_ptr_name, 5) == 0)
     {
-      value_ptr v;
+      struct value *v;
       /* First get the virtual table pointer and print it out */
 
 #if 0
@@ -432,7 +423,7 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
       v = value_from_pointer (lookup_pointer_type (builtin_type_unsigned_long),
                              *(unsigned long *) (valaddr + offset));
 
-      val_print (VALUE_TYPE (v), VALUE_CONTENTS (v), 0, 0,
+      val_print (value_type (v), VALUE_CONTENTS (v), 0, 0,
                 stream, format, 0, recurse + 1, pretty);
       fields_seen = 1;
 
@@ -449,7 +440,8 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
 
          if (1)
            {
-             /* no RRBC support; function pointers embedded directly in vtable */
+             /* no RRBC support; function pointers embedded directly
+                 in vtable */
 
              int vfuncs = count_virtual_fns (real_type);
 
@@ -457,7 +449,8 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
 
              /* FIXME : doesn't work at present */
 #if 0
-             fprintf_filtered (stream, "%d entr%s: ", vfuncs, vfuncs == 1 ? "y" : "ies");
+             fprintf_filtered (stream, "%d entr%s: ", vfuncs,
+                               vfuncs == 1 ? "y" : "ies");
 #else
              fputs_filtered ("not implemented", stream);
 
@@ -466,13 +459,15 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
 
              /* recursive function that prints all virtual function entries */
 #if 0
-             cp_print_hpacc_virtual_table_entries (real_type, &vfuncs, v, stream, format, recurse, pretty);
+             cp_print_hpacc_virtual_table_entries (real_type, &vfuncs, v,
+                                                   stream, format, recurse,
+                                                   pretty);
 #endif
              fputs_filtered ("}", stream);
            }                   /* non-RRBC case */
          else
            {
-             /* FIXME -- seem comments above */
+             /* FIXME -- see comments above */
              /* RRBC support present; function pointers are found
               * by indirection through the class segment entries. */
 
@@ -495,23 +490,17 @@ cp_print_value_fields (type, real_type, valaddr, offset, address, stream, format
    baseclasses.  */
 
 static void
-cp_print_value (type, real_type, valaddr, offset, address, stream, format, recurse, pretty,
-               dont_print_vb)
-     struct type *type;
-     struct type *real_type;
-     char *valaddr;
-     int offset;
-     CORE_ADDR address;
-     struct ui_file *stream;
-     int format;
-     int recurse;
-     enum val_prettyprint pretty;
-     struct type **dont_print_vb;
+cp_print_value (struct type *type, struct type *real_type,
+               const bfd_byte *valaddr, int offset, CORE_ADDR address,
+               struct ui_file *stream, int format, int recurse,
+               enum val_prettyprint pretty, struct type **dont_print_vb)
 {
   struct obstack tmp_obstack;
   struct type **last_dont_print
-  = (struct type **) obstack_next_free (&dont_print_vb_obstack);
+    = (struct type **) obstack_next_free (&dont_print_vb_obstack);
   int i, n_baseclasses = TYPE_N_BASECLASSES (type);
+  int thisoffset;
+  struct type *thistype;
 
   if (dont_print_vb == 0)
     {
@@ -529,15 +518,15 @@ cp_print_value (type, real_type, valaddr, offset, address, stream, format, recur
       int skip;
       struct type *baseclass = check_typedef (TYPE_BASECLASS (type, i));
       char *basename = TYPE_NAME (baseclass);
-      char *base_valaddr;
+      const bfd_byte *base_valaddr;
 
       if (BASETYPE_VIA_VIRTUAL (type, i))
        {
          struct type **first_dont_print
-         = (struct type **) obstack_base (&dont_print_vb_obstack);
+           = (struct type **) obstack_base (&dont_print_vb_obstack);
 
          int j = (struct type **) obstack_next_free (&dont_print_vb_obstack)
-         - first_dont_print;
+           - first_dont_print;
 
          while (--j >= 0)
            if (baseclass == first_dont_print[j])
@@ -546,32 +535,45 @@ cp_print_value (type, real_type, valaddr, offset, address, stream, format, recur
          obstack_ptr_grow (&dont_print_vb_obstack, baseclass);
        }
 
+      thisoffset = offset;
+      thistype = real_type;
       if (TYPE_HAS_VTABLE (type) && BASETYPE_VIA_VIRTUAL (type, i))
        {
          /* Assume HP/Taligent runtime convention */
          find_rt_vbase_offset (type, TYPE_BASECLASS (type, i),
                                valaddr, offset, &boffset, &skip);
          if (skip >= 0)
-           error ("Virtual base class offset not found from vtable while printing");
+           error ("Virtual base class offset not found from vtable while"
+                  " printing");
          base_valaddr = valaddr;
        }
       else
        {
-         boffset = baseclass_offset (type, i, valaddr + offset, address + offset);
+         boffset = baseclass_offset (type, i,
+                                     valaddr + offset,
+                                     address);
          skip = ((boffset == -1) || (boffset + offset) < 0) ? 1 : -1;
 
          if (BASETYPE_VIA_VIRTUAL (type, i))
            {
-             /* The virtual base class pointer might have been clobbered by the
-                user program. Make sure that it still points to a valid memory
-                location.  */
+             /* The virtual base class pointer might have been
+                clobbered by the user program. Make sure that it
+                still points to a valid memory location.  */
 
-             if (boffset != -1 && ((boffset + offset) < 0 || (boffset + offset) >= TYPE_LENGTH (type)))
+             if (boffset != -1
+                 && ((boffset + offset) < 0
+                     || (boffset + offset) >= TYPE_LENGTH (type)))
                {
-                 base_valaddr = (char *) alloca (TYPE_LENGTH (baseclass));
-                 if (target_read_memory (address + boffset, base_valaddr,
+                 /* FIXME (alloca): unsafe if baseclass is really really large. */
+                 bfd_byte *buf = alloca (TYPE_LENGTH (baseclass));
+                 base_valaddr = buf;
+                 if (target_read_memory (address + boffset, buf,
                                          TYPE_LENGTH (baseclass)) != 0)
                    skip = 1;
+                 address = address + boffset;
+                 thisoffset = 0;
+                 boffset = 0;
+                 thistype = baseclass;
                }
              else
                base_valaddr = valaddr;
@@ -596,9 +598,12 @@ cp_print_value (type, real_type, valaddr, offset, address, stream, format, recur
       if (skip >= 1)
        fprintf_filtered (stream, "<invalid address>");
       else
-       cp_print_value_fields (baseclass, real_type, base_valaddr, offset + boffset, address,
-                              stream, format, recurse, pretty,
-                    (struct type **) obstack_base (&dont_print_vb_obstack),
+       cp_print_value_fields (baseclass, thistype, base_valaddr,
+                              thisoffset + boffset, address + boffset,
+                              stream, format,
+                              recurse, pretty,
+                              ((struct type **)
+                               obstack_base (&dont_print_vb_obstack)),
                               0);
       fputs_filtered (", ", stream);
 
@@ -627,13 +632,12 @@ cp_print_value (type, real_type, valaddr, offset, address, stream, format, recur
    have the same meanings as in c_val_print.  */
 
 static void
-cp_print_static_field (type, val, stream, format, recurse, pretty)
-     struct type *type;
-     value_ptr val;
-     struct ui_file *stream;
-     int format;
-     int recurse;
-     enum val_prettyprint pretty;
+cp_print_static_field (struct type *type,
+                      struct value *val,
+                      struct ui_file *stream,
+                      int format,
+                      int recurse,
+                      enum val_prettyprint pretty)
 {
   if (TYPE_CODE (type) == TYPE_CODE_STRUCT)
     {
@@ -649,7 +653,8 @@ cp_print_static_field (type, val, stream, format, recurse, pretty)
        {
          if (VALUE_ADDRESS (val) == first_dont_print[i])
            {
-             fputs_filtered ("<same as static member of an already seen type>",
+             fputs_filtered ("<same as static member of an already"
+                             " seen type>",
                              stream);
              return;
            }
@@ -659,20 +664,19 @@ cp_print_static_field (type, val, stream, format, recurse, pretty)
                    sizeof (CORE_ADDR));
 
       CHECK_TYPEDEF (type);
-      cp_print_value_fields (type, type, VALUE_CONTENTS_ALL (val), VALUE_EMBEDDED_OFFSET (val), VALUE_ADDRESS (val),
+      cp_print_value_fields (type, type, VALUE_CONTENTS_ALL (val),
+                            VALUE_EMBEDDED_OFFSET (val), VALUE_ADDRESS (val),
                             stream, format, recurse, pretty, NULL, 1);
       return;
     }
-  val_print (type, VALUE_CONTENTS_ALL (val), VALUE_EMBEDDED_OFFSET (val), VALUE_ADDRESS (val),
+  val_print (type, VALUE_CONTENTS_ALL (val), 
+            VALUE_EMBEDDED_OFFSET (val), VALUE_ADDRESS (val),
             stream, format, 0, recurse, pretty);
 }
 
 void
-cp_print_class_member (valaddr, domain, stream, prefix)
-     char *valaddr;
-     struct type *domain;
-     struct ui_file *stream;
-     char *prefix;
+cp_print_class_member (const bfd_byte *valaddr, struct type *domain,
+                      struct ui_file *stream, char *prefix)
 {
 
   /* VAL is a byte offset into the structure type DOMAIN.
@@ -680,7 +684,7 @@ cp_print_class_member (valaddr, domain, stream, prefix)
      print it.  */
   int extra = 0;
   int bits = 0;
-  register unsigned int i;
+  unsigned int i;
   unsigned len = TYPE_NFIELDS (domain);
 
   /* @@ Make VAL into bit offset */
@@ -714,7 +718,7 @@ cp_print_class_member (valaddr, domain, stream, prefix)
   if (i < len)
     {
       char *name;
-      fprintf_filtered (stream, prefix);
+      fputs_filtered (prefix, stream);
       name = type_name_no_tag (domain);
       if (name)
        fputs_filtered (name, stream);
@@ -746,14 +750,10 @@ cp_print_class_member (valaddr, domain, stream, prefix)
 
 
 static void
-cp_print_hpacc_virtual_table_entries (type, vfuncs, v, stream, format, recurse, pretty)
-     struct type *type;
-     int *vfuncs;
-     value_ptr v;
-     struct ui_file *stream;
-     int format;
-     int recurse;
-     enum val_prettyprint pretty;
+cp_print_hpacc_virtual_table_entries (struct type *type, int *vfuncs,
+                                     struct value *v, struct ui_file *stream,
+                                     int format, int recurse,
+                                     enum val_prettyprint pretty)
 {
   int fn, oi;
 
@@ -764,7 +764,8 @@ cp_print_hpacc_virtual_table_entries (type, vfuncs, v, stream, format, recurse,
   /* Recursion on other classes that can share the same vtable */
   struct type *pbc = primary_base_class (type);
   if (pbc)
-    cp_print_hpacc_virtual_table_entries (pbc, vfuncs, v, stream, format, recurse, pretty);
+    cp_print_hpacc_virtual_table_entries (pbc, vfuncs, v, stream, format,
+                                         recurse, pretty);
 
   /* Now deal with vfuncs declared in this class */
   for (fn = 0; fn < TYPE_NFN_FIELDS (type); fn++)
@@ -772,23 +773,28 @@ cp_print_hpacc_virtual_table_entries (type, vfuncs, v, stream, format, recurse,
       if (TYPE_FN_FIELD_VIRTUAL_P (TYPE_FN_FIELDLIST1 (type, fn), oi))
        {
          char *vf_name;
+         const char *field_physname;
 
          /* virtual function offset */
-         int vx = TYPE_FN_FIELD_VOFFSET (TYPE_FN_FIELDLIST1 (type, fn), oi) - 1;
+         int vx = (TYPE_FN_FIELD_VOFFSET (TYPE_FN_FIELDLIST1 (type, fn), oi)
+                   - 1);
 
          /* Get the address of the vfunction entry */
-         value_ptr vf = value_copy (v);
+         struct value *vf = value_copy (v);
          if (VALUE_LAZY (vf))
            (void) value_fetch_lazy (vf);
-         vf->aligner.contents[0] += 4 * (HP_ACC_VFUNC_START + vx);     /* adjust by offset */
+         /* adjust by offset */
+         vf->aligner.contents[0] += 4 * (HP_ACC_VFUNC_START + vx);
          vf = value_ind (vf);  /* get the entry */
-         VALUE_TYPE (vf) = VALUE_TYPE (v);     /* make it a pointer */
+         vf->type = value_type (v);    /* make it a pointer */
 
          /* print out the entry */
-         val_print (VALUE_TYPE (vf), VALUE_CONTENTS (vf), 0, 0,
+         val_print (value_type (vf), VALUE_CONTENTS (vf), 0, 0,
                     stream, format, 0, recurse + 1, pretty);
-         vf_name = cplus_demangle (TYPE_FN_FIELD_PHYSNAME (TYPE_FN_FIELDLIST1 (type, fn), oi),
-                                   DMGL_ARM);  /* pai: (temp) FIXME Maybe this should be DMGL_ANSI */
+         field_physname
+           = TYPE_FN_FIELD_PHYSNAME (TYPE_FN_FIELDLIST1 (type, fn), oi);
+         /* pai: (temp) FIXME Maybe this should be DMGL_ANSI */
+         vf_name = cplus_demangle (field_physname, DMGL_ARM);
          fprintf_filtered (stream, " %s", vf_name);
          if (--(*vfuncs) > 0)
            fputs_filtered (", ", stream);
@@ -798,9 +804,9 @@ cp_print_hpacc_virtual_table_entries (type, vfuncs, v, stream, format, recurse,
 
 
 void
-_initialize_cp_valprint ()
+_initialize_cp_valprint (void)
 {
-  add_show_from_set
+  deprecated_add_show_from_set
     (add_set_cmd ("static-members", class_support, var_boolean,
                  (char *) &static_field_print,
                  "Set printing of C++ static members.",
@@ -809,13 +815,13 @@ _initialize_cp_valprint ()
   /* Turn on printing of static fields.  */
   static_field_print = 1;
 
-  add_show_from_set
+  deprecated_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
+  deprecated_add_show_from_set
     (add_set_cmd ("object", class_support, var_boolean, (char *) &objectprint,
              "Set printing of object's derived type based on vtable info.",
                  &setprintlist),
@@ -827,5 +833,5 @@ _initialize_cp_valprint ()
   obstack_begin (&dont_print_vb_obstack, 32 * sizeof (struct type *));
   obstack_specify_allocation (&dont_print_statmem_obstack,
                              32 * sizeof (CORE_ADDR), sizeof (CORE_ADDR),
-                             xmalloc, free);
+                             xmalloc, xfree);
 }
This page took 0.033898 seconds and 4 git commands to generate.