This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] [Ada] Fix handling of array renamings


> gdb/ChangeLog:
> 
> 	* ada-lang.c (ada_evaluate_subexp) <OP_FUNCALL>: When the input
> 	value is a reference, actually dereference it in order to get
> 	the underlying value.
> 
> gdb/testsuite/ChangeLog:
> 
> 	* gdb.ada/array_ptr_renaming.exp: New testcase.
> 	* gdb.ada/array_ptr_renaming/foo.adb: New file.
> 	* gdb.ada/array_ptr_renaming/pack.ads: New file.

OK with one tiny little suggestion.

> +gdb_test "print ntp"     " = \\(access pack\\.table_type\\) 0x.*"
                                                               ^^^

Suggest using $hex.* instead of 0x.*

Thanks! And sorry for the late review...
-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]