diff options
author | Joel Brobecker <brobecker@gnat.com> | 2012-02-29 19:30:53 +0000 |
---|---|---|
committer | Joel Brobecker <brobecker@gnat.com> | 2012-02-29 19:30:53 +0000 |
commit | c48db5caef25bc89b8f444300c7740f01405938b (patch) | |
tree | 325e5b90f3107c0a37f1c1bc67ea72cd674cb664 /gdb/ada-lang.c | |
parent | 2e6fda7d06e4fa05ef3abdde96805a5cb3880f88 (diff) | |
download | gdb-c48db5caef25bc89b8f444300c7740f01405938b.zip gdb-c48db5caef25bc89b8f444300c7740f01405938b.tar.gz gdb-c48db5caef25bc89b8f444300c7740f01405938b.tar.bz2 |
Ada: ada_to_fixed_value should also unwrap_value
We should always unwrap a value before trying to "fix" it. It is
therefore logical that ada_to_fixed_value would call unwrap_value
before creating the fixed value.
This simplifies the code in ada-lang.c a bit.
gdb/ChangeLog:
* ada-lang.c (ada_to_fixed_value): Call unwrap_value before
creating fixed value.
(ada_value_ind, ada_coerce_ref, assign_component)
(ada_evaluate_subexp): Remove call to unwrap_value before
call to ada_to_fixed_value.
Diffstat (limited to 'gdb/ada-lang.c')
-rw-r--r-- | gdb/ada-lang.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c index 7c13910..7243ab8 100644 --- a/gdb/ada-lang.c +++ b/gdb/ada-lang.c @@ -6902,7 +6902,7 @@ ada_which_variant_applies (struct type *var_type, struct type *outer_type, struct value * ada_value_ind (struct value *val0) { - struct value *val = unwrap_value (value_ind (val0)); + struct value *val = value_ind (val0); return ada_to_fixed_value (val); } @@ -6918,7 +6918,6 @@ ada_coerce_ref (struct value *val0) struct value *val = val0; val = coerce_ref (val); - val = unwrap_value (val); return ada_to_fixed_value (val); } else @@ -8138,9 +8137,11 @@ ada_to_fixed_value_create (struct type *type0, CORE_ADDR address, struct value * ada_to_fixed_value (struct value *val) { - return ada_to_fixed_value_create (value_type (val), - value_address (val), - val); + val = unwrap_value (val); + val = ada_to_fixed_value_create (value_type (val), + value_address (val), + val); + return val; } @@ -8702,7 +8703,7 @@ assign_component (struct value *container, struct value *lhs, LONGEST index, else { elt = ada_index_struct_field (index, lhs, 0, value_type (lhs)); - elt = ada_to_fixed_value (unwrap_value (elt)); + elt = ada_to_fixed_value (elt); } if (exp->elts[*pos].opcode == OP_AGGREGATE) @@ -9539,7 +9540,6 @@ ada_evaluate_subexp (struct type *expect_type, struct expression *exp, else { arg1 = evaluate_subexp_standard (expect_type, exp, pos, noside); - arg1 = unwrap_value (arg1); return ada_to_fixed_value (arg1); } |