Improve gdb.ada/rename_subscript_param.exp by using more unique names.
authorPhilippe Waroquiers <philippe.waroquiers@skynet.be>
Wed, 26 Dec 2018 13:25:46 +0000 (14:25 +0100)
committerPhilippe Waroquiers <philippe.waroquiers@skynet.be>
Thu, 27 Dec 2018 09:47:00 +0000 (10:47 +0100)
commitafcfda091e15beb9580398709235c1866a4c79e1
treed066ee772017a7ad2906b9f7238d84b15786a596
parent2ab54467f89763c96e2e9bda876eee1f9c8561b1
Improve gdb.ada/rename_subscript_param.exp by using more unique names.

With old compilers, the test fails because no debug info is generated
for 'B' and GDB finds some 'b' in atnat.h:

(gdb) print b
Multiple matches for b
[0] cancel
[1] b at ../sysdeps/ieee754/dbl-64/atnat.h:106
[2] b at ../sysdeps/ieee754/dbl-64/atnat.h:106
[3] b at ../sysdeps/ieee754/dbl-64/atnat.h:106
> FAIL: gdb.ada/rename_subscript_param.exp: print b before changing its value (timeout)

Avoid the timeout by renaming 'b' to rename_subscript_param_b.

Also, change 'before' to 'after' in the gdb_test message that prints
the value after changing it.

The test still fails with old compilers that do not properly
generate debug info for this renaming:
(gdb) print rename_subscript_param_b
No definition of "rename_subscript_param_b" in current context.
(gdb) FAIL: gdb.ada/rename_subscript_param.exp: print rename_subscript_param_b before changing its value

Note: if the compiler would generate the correct debug info, the test should
succeed with the name B.  However, waiting for this fix, changing the name
ensures that the test fails directly, instead of causing a timeout.

2018-12-26  Philippe Waroquiers  <philippe.waroquiers@skynet.be>

PR ada/23381
* gdb.ada/rename_subscript_param/pkg.adb (B): Rename to
Rename_Subscript_Param_B.  All users updated.
gdb.ada/rename_subscript_param.exp: Test names made unique.
Note that PR ada/23381 is only fully fixed when using a recent
compiler.
gdb/testsuite/gdb.ada/rename_subscript_param.exp
gdb/testsuite/gdb.ada/rename_subscript_param/pkg.adb