Fix latent bug with custom word point completers
[deliverable/binutils-gdb.git] / gdb / ada-valprint.c
index 91dc301af5f2d0f9f31e36c796dd605ebc90db6b..0654049d77b9b60a454194ed5d2a55b026ccead4 100644 (file)
@@ -1,6 +1,6 @@
 /* Support for printing Ada values for GDB, the GNU debugger.
 
-   Copyright (C) 1986-2017 Free Software Foundation, Inc.
+   Copyright (C) 1986-2019 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -88,9 +88,11 @@ print_optional_low_bound (struct ui_file *stream, struct type *type,
       index_type = TYPE_TARGET_TYPE (index_type);
     }
 
+  /* Don't print the lower bound if it's the default one.  */
   switch (TYPE_CODE (index_type))
     {
     case TYPE_CODE_BOOL:
+    case TYPE_CODE_CHAR:
       if (low_bound == 0)
        return 0;
       break;
@@ -139,11 +141,45 @@ val_print_packed_array_elements (struct type *type, const gdb_byte *valaddr,
 
   {
     LONGEST high;
+    struct type *base_index_type;
 
     if (get_discrete_bounds (index_type, &low, &high) < 0)
       len = 1;
     else
       len = high - low + 1;
+
+    if (TYPE_CODE (index_type) == TYPE_CODE_RANGE)
+        base_index_type = TYPE_TARGET_TYPE (index_type);
+      else
+        base_index_type = index_type;
+
+    if (TYPE_CODE (base_index_type) == TYPE_CODE_ENUM)
+      {
+        LONGEST low_pos, high_pos;
+
+        /* Non-contiguous enumerations types can by used as index types
+           so the array length is computed from the positions of the
+           first and last literal in the enumeration type, and not from
+           the values of these literals.  */
+
+        if (!discrete_position (base_index_type, low, &low_pos)
+          || !discrete_position (base_index_type, high, &high_pos))
+          {
+            warning (_("unable to get positions in array, use bounds instead"));
+            low_pos = low;
+            high_pos = high;
+          }
+
+        /* The array length should normally be HIGH_POS - LOW_POS + 1.
+           But in Ada we allow LOW_POS to be greater than HIGH_POS for
+           empty arrays.  In that situation, the array length is just zero,
+           not negative!  */
+
+        if (low_pos > high_pos)
+          len = 0;
+        else
+          len = high_pos - low_pos + 1;
+      }
   }
 
   i = 0;
@@ -429,7 +465,6 @@ ada_print_scalar (struct type *type, LONGEST val, struct ui_file *stream)
     default:
       error (_("Invalid type code in symbol table."));
     }
-  gdb_flush (stream);
 }
 
 /* Print the character string STRING, printing at most LENGTH characters.
@@ -730,8 +765,7 @@ ada_val_print_gnat_array (struct type *type, const gdb_byte *valaddr,
                          int offset, CORE_ADDR address,
                          struct ui_file *stream, int recurse,
                          struct value *original_value,
-                         const struct value_print_options *options,
-                         const struct language_defn *language)
+                         const struct value_print_options *options)
 {
   struct value *mark = value_mark ();
   struct value *val;
@@ -753,7 +787,8 @@ ada_val_print_gnat_array (struct type *type, const gdb_byte *valaddr,
   else
     val_print (value_type (val),
               value_embedded_offset (val), value_address (val),
-              stream, recurse, val, options, language);
+              stream, recurse, val, options,
+              language_def (language_ada));
   value_free_to_mark (mark);
 }
 
@@ -765,8 +800,7 @@ ada_val_print_ptr (struct type *type, const gdb_byte *valaddr,
                   int offset, int offset_aligned, CORE_ADDR address,
                   struct ui_file *stream, int recurse,
                   struct value *original_value,
-                  const struct value_print_options *options,
-                  const struct language_defn *language)
+                  const struct value_print_options *options)
 {
   val_print (type, offset, address, stream, recurse,
             original_value, options, language_def (language_c));
@@ -792,8 +826,7 @@ ada_val_print_num (struct type *type, const gdb_byte *valaddr,
                   int offset, int offset_aligned, CORE_ADDR address,
                   struct ui_file *stream, int recurse,
                   struct value *original_value,
-                  const struct value_print_options *options,
-                  const struct language_defn *language)
+                  const struct value_print_options *options)
 {
   if (ada_is_fixed_point_type (type))
     {
@@ -824,12 +857,13 @@ ada_val_print_num (struct type *type, const gdb_byte *valaddr,
 
          val_print (target_type,
                     value_embedded_offset (v), 0, stream,
-                    recurse + 1, v, options, language);
+                    recurse + 1, v, options,
+                    language_def (language_ada));
        }
       else
        val_print (TYPE_TARGET_TYPE (type), offset,
                   address, stream, recurse, original_value,
-                  options, language);
+                  options, language_def (language_ada));
       return;
     }
   else
@@ -888,8 +922,7 @@ ada_val_print_enum (struct type *type, const gdb_byte *valaddr,
                    int offset, int offset_aligned, CORE_ADDR address,
                    struct ui_file *stream, int recurse,
                    struct value *original_value,
-                   const struct value_print_options *options,
-                   const struct language_defn *language)
+                   const struct value_print_options *options)
 {
   int i;
   unsigned int len;
@@ -932,8 +965,7 @@ ada_val_print_flt (struct type *type, const gdb_byte *valaddr,
                   int offset, int offset_aligned, CORE_ADDR address,
                   struct ui_file *stream, int recurse,
                   struct value *original_value,
-                  const struct value_print_options *options,
-                  const struct language_defn *language)
+                  const struct value_print_options *options)
 {
   if (options->format)
     {
@@ -953,8 +985,7 @@ ada_val_print_struct_union
   (struct type *type, const gdb_byte *valaddr, int offset,
    int offset_aligned, CORE_ADDR address, struct ui_file *stream,
    int recurse, struct value *original_value,
-   const struct value_print_options *options,
-   const struct language_defn *language)
+   const struct value_print_options *options)
 {
   if (ada_is_bogus_array_descriptor (type))
     {
@@ -966,7 +997,8 @@ ada_val_print_struct_union
 
   if (print_field_values (type, valaddr, offset_aligned,
                          stream, recurse, original_value, options,
-                         0, type, offset_aligned, language) != 0
+                         0, type, offset_aligned,
+                         language_def (language_ada)) != 0
       && options->prettyformat)
     {
       fprintf_filtered (stream, "\n");
@@ -1017,8 +1049,7 @@ ada_val_print_ref (struct type *type, const gdb_byte *valaddr,
                   int offset, int offset_aligned, CORE_ADDR address,
                   struct ui_file *stream, int recurse,
                   struct value *original_value,
-                  const struct value_print_options *options,
-                  const struct language_defn *language)
+                  const struct value_print_options *options)
 {
   /* For references, the debugger is expected to print the value as
      an address if DEREF_REF is null.  But printing an address in place
@@ -1042,7 +1073,7 @@ ada_val_print_ref (struct type *type, const gdb_byte *valaddr,
        deref_val = ada_tag_value_at_base_address (deref_val);
 
       common_val_print (deref_val, stream, recurse + 1, options,
-                       language);
+                       language_def (language_ada));
       return;
     }
 
@@ -1071,7 +1102,7 @@ ada_val_print_ref (struct type *type, const gdb_byte *valaddr,
   val_print (value_type (deref_val),
             value_embedded_offset (deref_val),
             value_address (deref_val), stream, recurse + 1,
-            deref_val, options, language);
+            deref_val, options, language_def (language_ada));
 }
 
 /* See the comment on ada_val_print.  This function differs in that it
@@ -1082,8 +1113,7 @@ ada_val_print_1 (struct type *type,
                 int offset, CORE_ADDR address,
                 struct ui_file *stream, int recurse,
                 struct value *original_value,
-                const struct value_print_options *options,
-                const struct language_defn *language)
+                const struct value_print_options *options)
 {
   int offset_aligned;
   const gdb_byte *valaddr = value_contents_for_printing (original_value);
@@ -1096,7 +1126,7 @@ ada_val_print_1 (struct type *type,
     {
       ada_val_print_gnat_array (type, valaddr, offset, address,
                                stream, recurse, original_value,
-                               options, language);
+                               options);
       return;
     }
 
@@ -1115,33 +1145,33 @@ ada_val_print_1 (struct type *type,
     case TYPE_CODE_PTR:
       ada_val_print_ptr (type, valaddr, offset, offset_aligned,
                         address, stream, recurse, original_value,
-                        options, language);
+                        options);
       break;
 
     case TYPE_CODE_INT:
     case TYPE_CODE_RANGE:
       ada_val_print_num (type, valaddr, offset, offset_aligned,
                         address, stream, recurse, original_value,
-                        options, language);
+                        options);
       break;
 
     case TYPE_CODE_ENUM:
       ada_val_print_enum (type, valaddr, offset, offset_aligned,
                          address, stream, recurse, original_value,
-                         options, language);
+                         options);
       break;
 
     case TYPE_CODE_FLT:
       ada_val_print_flt (type, valaddr, offset, offset_aligned,
                         address, stream, recurse, original_value,
-                        options, language);
+                        options);
       break;
 
     case TYPE_CODE_UNION:
     case TYPE_CODE_STRUCT:
       ada_val_print_struct_union (type, valaddr, offset, offset_aligned,
                                  address, stream, recurse,
-                                 original_value, options, language);
+                                 original_value, options);
       break;
 
     case TYPE_CODE_ARRAY:
@@ -1153,7 +1183,7 @@ ada_val_print_1 (struct type *type,
     case TYPE_CODE_REF:
       ada_val_print_ref (type, valaddr, offset, offset_aligned,
                         address, stream, recurse, original_value,
-                        options, language);
+                        options);
       break;
     }
 }
@@ -1168,18 +1198,16 @@ ada_val_print (struct type *type,
               struct value *val,
               const struct value_print_options *options)
 {
-
-  /* XXX: this catches QUIT/ctrl-c as well.  Isn't that busted?  */
-  TRY
+  try
     {
       ada_val_print_1 (type, embedded_offset, address,
-                      stream, recurse, val, options,
-                      current_language);
+                      stream, recurse, val, options);
     }
-  CATCH (except, RETURN_MASK_ALL)
+  catch (const gdb_exception_error &except)
     {
+      fprintf_filtered (stream, _("<error reading variable: %s>"),
+                       except.what ());
     }
-  END_CATCH
 }
 
 void
@@ -1188,7 +1216,7 @@ ada_value_print (struct value *val0, struct ui_file *stream,
 {
   struct value *val = ada_to_fixed_value (val0);
   CORE_ADDR address = value_address (val);
-  struct type *type = ada_check_typedef (value_enclosing_type (val));
+  struct type *type = ada_check_typedef (value_type (val));
   struct value_print_options opts;
 
   /* If it is a pointer, indicate what it points to.  */
This page took 0.046328 seconds and 4 git commands to generate.