PR 6878
[deliverable/binutils-gdb.git] / gdb / ada-valprint.c
index 648cd3605e76f68a79e04951ea48f70667791e4e..e2f7740e2891188c524ef6be9f2f4502fb05ca06 100644 (file)
@@ -33,6 +33,7 @@
 #include "c-lang.h"
 #include "infcall.h"
 #include "exceptions.h"
+#include "objfiles.h"
 
 /* Encapsulates arguments to ada_val_print.  */
 struct ada_val_print_args
@@ -71,24 +72,31 @@ adjust_type_signedness (struct type *type)
 {
   if (type != NULL && TYPE_CODE (type) == TYPE_CODE_RANGE
       && TYPE_LOW_BOUND (type) >= 0)
-    TYPE_FLAGS (type) |= TYPE_FLAG_UNSIGNED;
+    TYPE_UNSIGNED (type) = 1;
 }
 
-/* Assuming TYPE is a simple, non-empty array type, prints its lower bound 
-   on STREAM, if non-standard (i.e., other than 1 for numbers, other
-   than lower bound of index type for enumerated type).  Returns 1 
-   if something printed, otherwise 0.  */
+/* Assuming TYPE is a simple array type, prints its lower bound on STREAM,
+   if non-standard (i.e., other than 1 for numbers, other than lower bound
+   of index type for enumerated type).  Returns 1 if something printed,
+   otherwise 0.  */
 
 static int
 print_optional_low_bound (struct ui_file *stream, struct type *type)
 {
   struct type *index_type;
   long low_bound;
+  long high_bound;
 
   if (print_array_indexes_p ())
     return 0;
 
-  if (!get_array_low_bound (type, &low_bound))
+  if (!get_array_bounds (type, &low_bound, &high_bound))
+    return 0;
+
+  /* If this is an empty array, then don't print the lower bound.
+     That would be confusing, because we would print the lower bound,
+     followed by... nothing!  */
+  if (low_bound > high_bound)
     return 0;
 
   index_type = TYPE_INDEX_TYPE (type);
@@ -105,12 +113,16 @@ print_optional_low_bound (struct ui_file *stream, struct type *type)
 
   switch (TYPE_CODE (index_type))
     {
+    case TYPE_CODE_BOOL:
+      if (low_bound == 0)
+       return 0;
+      break;
     case TYPE_CODE_ENUM:
       if (low_bound == TYPE_FIELD_BITPOS (index_type, 0))
        return 0;
       break;
     case TYPE_CODE_UNDEF:
-      index_type = builtin_type_long;
+      index_type = builtin_type_int32;
       /* FALL THROUGH */
     default:
       if (low_bound == 1)
@@ -201,7 +213,7 @@ val_print_packed_array_elements (struct type *type, const gdb_byte *valaddr,
       if (i - i0 > repeat_count_threshold)
        {
          val_print (elttype, value_contents (v0), 0, 0, stream, format,
-                    0, recurse + 1, pretty);
+                    0, recurse + 1, pretty, current_language);
          annotate_elt_rep (i - i0);
          fprintf_filtered (stream, _(" <repeats %u times>"), i - i0);
          annotate_elt_rep_end ();
@@ -228,7 +240,7 @@ val_print_packed_array_elements (struct type *type, const gdb_byte *valaddr,
                                           stream, format, pretty);
                }
              val_print (elttype, value_contents (v0), 0, 0, stream, format,
-                        0, recurse + 1, pretty);
+                        0, recurse + 1, pretty, current_language);
              annotate_elt ();
            }
        }
@@ -586,8 +598,73 @@ ada_val_print_stub (void *args0)
                          argsp->recurse, argsp->pretty);
 }
 
+/* Assuming TYPE is a simple array, print the value of this array located
+   at VALADDR.  See ada_val_print for a description of the various
+   parameters of this function; they are identical.  The semantics
+   of the return value is also identical to ada_val_print.  */
+
+static int
+ada_val_print_array (struct type *type, const gdb_byte *valaddr,
+                    CORE_ADDR address, struct ui_file *stream, int format,
+                    int deref_ref, int recurse, enum val_prettyprint pretty)
+{
+  struct type *elttype = TYPE_TARGET_TYPE (type);
+  unsigned int eltlen;
+  unsigned int len;
+  int result = 0;
+
+  if (elttype == NULL)
+    eltlen = 0;
+  else
+    eltlen = TYPE_LENGTH (elttype);
+  if (eltlen == 0)
+    len = 0;
+  else
+    len = TYPE_LENGTH (type) / eltlen;
+
+  /* For an array of chars, print with string syntax.  */
+  if (ada_is_string_type (type) && (format == 0 || format == 's'))
+    {
+      if (prettyprint_arrays)
+        print_spaces_filtered (2 + 2 * recurse, stream);
+
+      /* If requested, look for the first null char and only print
+         elements up to it.  */
+      if (stop_print_at_null)
+        {
+          int temp_len;
+
+          /* Look for a NULL char.  */
+          for (temp_len = 0;
+               (temp_len < len
+                && temp_len < print_max
+                && char_at (valaddr, temp_len, eltlen) != 0);
+               temp_len += 1);
+          len = temp_len;
+        }
+
+      printstr (stream, valaddr, len, 0, eltlen);
+      result = len;
+    }
+  else
+    {
+      fprintf_filtered (stream, "(");
+      print_optional_low_bound (stream, type);
+      if (TYPE_FIELD_BITSIZE (type, 0) > 0)
+        val_print_packed_array_elements (type, valaddr, 0, stream,
+                                         format, recurse, pretty);
+      else
+        val_print_array_elements (type, valaddr, address, stream,
+                                  format, deref_ref, recurse,
+                                  pretty, 0);
+      fprintf_filtered (stream, ")");
+    }
+
+  return result;
+}
+
 /* See the comment on ada_val_print.  This function differs in that it
* does not catch evaluation errors (leaving that to ada_val_print).  */
  does not catch evaluation errors (leaving that to ada_val_print).  */
 
 static int
 ada_val_print_1 (struct type *type, const gdb_byte *valaddr0,
@@ -675,8 +752,8 @@ ada_val_print_1 (struct type *type, const gdb_byte *valaddr0,
                parray_of_char =
                  make_pointer_type
                  (create_array_type
-                  (NULL, builtin_type_char,
-                   create_range_type (NULL, builtin_type_int, 0, 32)), NULL);
+                  (NULL, builtin_type_true_char,
+                   create_range_type (NULL, builtin_type_int32, 0, 32)), NULL);
 
              printable_val =
                value_ind (value_cast (parray_of_char,
@@ -716,18 +793,27 @@ ada_val_print_1 (struct type *type, const gdb_byte *valaddr0,
            {
              print_scalar_formatted (valaddr, type, format, 0, stream);
            }
-          else if (ada_is_system_address_type (type))
+          else if (ada_is_system_address_type (type)
+                  && TYPE_OBJFILE (type) != NULL)
             {
               /* FIXME: We want to print System.Address variables using
                  the same format as for any access type.  But for some
                  reason GNAT encodes the System.Address type as an int,
                  so we have to work-around this deficiency by handling
-                 System.Address values as a special case.  */
+                 System.Address values as a special case.
+
+                We do this only for System.Address types defined in an
+                objfile.  For the built-in version of System.Address we
+                have installed the proper type to begin with.  */
+
+             struct gdbarch *gdbarch = get_objfile_arch (TYPE_OBJFILE (type));
+             struct type *ptr_type = builtin_type (gdbarch)->builtin_data_ptr;
+
               fprintf_filtered (stream, "(");
               type_print (type, "", stream, -1);
               fprintf_filtered (stream, ") ");
              fputs_filtered (paddress (extract_typed_address
-                                       (valaddr, builtin_type_void_data_ptr)),
+                                       (valaddr, ptr_type)),
                              stream);
             }
          else
@@ -802,57 +888,8 @@ ada_val_print_1 (struct type *type, const gdb_byte *valaddr0,
        }
 
     case TYPE_CODE_ARRAY:
-      elttype = TYPE_TARGET_TYPE (type);
-      if (elttype == NULL)
-       eltlen = 0;
-      else
-       eltlen = TYPE_LENGTH (elttype);
-      /* FIXME: This doesn't deal with non-empty arrays of
-        0-length items (not a typical case!) */
-      if (eltlen == 0)
-       len = 0;
-      else
-       len = TYPE_LENGTH (type) / eltlen;
-
-         /* For an array of chars, print with string syntax.  */
-      if (ada_is_string_type (type) && (format == 0 || format == 's'))
-       {
-         if (prettyprint_arrays)
-           {
-             print_spaces_filtered (2 + 2 * recurse, stream);
-           }
-         /* If requested, look for the first null char and only print
-            elements up to it.  */
-         if (stop_print_at_null)
-           {
-             int temp_len;
-
-             /* Look for a NULL char.  */
-             for (temp_len = 0;
-                  temp_len < len && temp_len < print_max
-                    && char_at (valaddr, temp_len, eltlen) != 0;
-                  temp_len += 1);
-             len = temp_len;
-           }
-
-         printstr (stream, valaddr, len, 0, eltlen);
-       }
-      else
-       {
-         len = 0;
-         fprintf_filtered (stream, "(");
-         print_optional_low_bound (stream, type);
-         if (TYPE_FIELD_BITSIZE (type, 0) > 0)
-           val_print_packed_array_elements (type, valaddr, 0, stream,
-                                            format, recurse, pretty);
-         else
-           val_print_array_elements (type, valaddr, address, stream,
-                                     format, deref_ref, recurse,
-                                     pretty, 0);
-         fprintf_filtered (stream, ")");
-       }
-      gdb_flush (stream);
-      return len;
+      return ada_val_print_array (type, valaddr, address, stream, format,
+                                  deref_ref, recurse, pretty);
 
     case TYPE_CODE_REF:
       /* For references, the debugger is expected to print the value as
@@ -864,9 +901,7 @@ ada_val_print_1 (struct type *type, const gdb_byte *valaddr0,
       
       if (TYPE_CODE (elttype) != TYPE_CODE_UNDEF)
         {
-          LONGEST deref_val_int = (LONGEST)
-            unpack_pointer (lookup_pointer_type (builtin_type_void),
-                            valaddr);
+          LONGEST deref_val_int = (LONGEST) unpack_pointer (type, valaddr);
           if (deref_val_int != 0)
             {
               struct value *deref_val =
@@ -876,7 +911,7 @@ ada_val_print_1 (struct type *type, const gdb_byte *valaddr0,
               val_print (value_type (deref_val),
                          value_contents (deref_val), 0,
                          VALUE_ADDRESS (deref_val), stream, format,
-                         deref_ref, recurse + 1, pretty);
+                         deref_ref, recurse + 1, pretty, current_language);
             }
           else
             fputs_filtered ("(null)", stream);
@@ -949,24 +984,8 @@ ada_value_print (struct value *val0, struct ui_file *stream, int format,
       return 0;
     }
 
-  if (TYPE_CODE (type) == TYPE_CODE_ARRAY
-      && TYPE_LENGTH (TYPE_TARGET_TYPE (type)) == 0
-      && TYPE_CODE (TYPE_INDEX_TYPE (type)) == TYPE_CODE_RANGE)
-    {
-      /* This is an array of zero-length elements, that is an array
-         of null records.  This array needs to be printed by hand,
-         as the standard routine to print arrays relies on the size of
-         the array elements to be nonzero.  This is because it computes
-         the number of elements in the array by dividing the array size
-         by the array element size.  */
-      fprintf_filtered (stream, "(%d .. %d => ())",
-                        TYPE_LOW_BOUND (TYPE_INDEX_TYPE (type)),
-                        TYPE_HIGH_BOUND (TYPE_INDEX_TYPE (type)));
-      return 0;
-    }
-  
   return (val_print (type, value_contents (val), 0, address,
-                    stream, format, 1, 0, pretty));
+                    stream, format, 1, 0, pretty, current_language));
 }
 
 static void
@@ -1096,7 +1115,8 @@ print_field_values (struct type *type, const gdb_byte *valaddr,
                                                  bit_size,
                                                  TYPE_FIELD_TYPE (type, i));
              val_print (TYPE_FIELD_TYPE (type, i), value_contents (v), 0, 0,
-                        stream, format, 0, recurse + 1, pretty);
+                        stream, format, 0, recurse + 1, pretty,
+                        current_language);
            }
        }
       else
This page took 0.027574 seconds and 4 git commands to generate.