aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Arnez <arnez@linux.vnet.ibm.com>2014-10-24 11:57:22 +0000
committerAndreas Krebbel <krebbel@linux.vnet.ibm.com>2014-11-13 10:20:38 +0100
commit78f98cca318cbf5f38d85a09f11e17cc3684ac51 (patch)
tree8b0465b3278b235e98cbd94eaec4403211aa3543
parent58fa2af0b3d87dbc2aaf640ef31af301e5f3c9b6 (diff)
downloadgdb-78f98cca318cbf5f38d85a09f11e17cc3684ac51.zip
gdb-78f98cca318cbf5f38d85a09f11e17cc3684ac51.tar.gz
gdb-78f98cca318cbf5f38d85a09f11e17cc3684ac51.tar.bz2
Eliminate literal line numbers in jump.exp
Remove literal line numbers from the regexps in jump.exp. Add appropriate eye-catchers to jump.c and refer to those instead. gdb/testsuite/ChangeLog: * gdb.base/jump.c: Add eye-catchers. * gdb.base/jump.exp: Refer to eye-catchers instead of literal line numbers.
-rw-r--r--gdb/testsuite/ChangeLog6
-rw-r--r--gdb/testsuite/gdb.base/jump.c6
-rw-r--r--gdb/testsuite/gdb.base/jump.exp29
3 files changed, 25 insertions, 16 deletions
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index c4021c0..f1ef4bb 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,11 @@
2014-11-13 Andreas Arnez <arnez@linux.vnet.ibm.com>
+ * gdb.base/jump.c: Add eye-catchers.
+ * gdb.base/jump.exp: Refer to eye-catchers instead of literal line
+ numbers.
+
+2014-11-13 Andreas Arnez <arnez@linux.vnet.ibm.com>
+
* gdb.base/execd-prog.c: Add eye-catchers.
* gdb.base/foll-exec.c: Likewise.
* gdb.base/foll-exec.exp: Refer to eye-catchers instead of literal
diff --git a/gdb/testsuite/gdb.base/jump.c b/gdb/testsuite/gdb.base/jump.c
index aae94f7..3c05244 100644
--- a/gdb/testsuite/gdb.base/jump.c
+++ b/gdb/testsuite/gdb.base/jump.c
@@ -9,7 +9,7 @@ static int square (x)
int x;
#endif
{
- return x*x;
+ return x*x; /* out-of-func */
}
@@ -18,7 +18,7 @@ int main ()
int i = 99;
i++;
- i = square (i);
- i--;
+ i = square (i); /* bp-on-call */
+ i--; /* bp-on-non-call */
return 0;
}
diff --git a/gdb/testsuite/gdb.base/jump.exp b/gdb/testsuite/gdb.base/jump.exp
index f26c60c..4c0dbd6 100644
--- a/gdb/testsuite/gdb.base/jump.exp
+++ b/gdb/testsuite/gdb.base/jump.exp
@@ -38,8 +38,9 @@ if ![runto_main] then {
# The statement doesn't contain a function call.
#
set bp_on_non_call 0
-gdb_test_multiple "break 22" "break before jump to non-call" {
- -re "\[Bb\]reakpoint (\[0-9\]*) at 0x\[0-9a-fA-F\]*: file .*${srcfile}, line 22.*$gdb_prompt $" {
+set non_call_line [gdb_get_line_number "bp-on-non-call"]
+gdb_test_multiple "break $non_call_line" "break before jump to non-call" {
+ -re "\[Bb\]reakpoint (${decimal}) at ${hex}: file .*${srcfile}, line $non_call_line.*$gdb_prompt $" {
set bp_on_non_call $expect_out(1,string)
pass "break before jump to non-call"
}
@@ -47,15 +48,16 @@ gdb_test_multiple "break 22" "break before jump to non-call" {
# Can we jump to the statement? Do we stop there?
#
-gdb_test "jump 22" "Breakpoint \[0-9\]*, .*${srcfile}:22.*" \
+gdb_test "jump $non_call_line" "Breakpoint ${decimal}, .*${srcfile}:$non_call_line.*" \
"jump to non-call"
# Set a breakpoint on the statement that we're about to jump to.
# The statement does contain a function call.
#
set bp_on_call 0
-gdb_test_multiple "break 21" "break before jump to call" {
- -re "\[Bb\]reakpoint (\[0-9\]*) at 0x\[0-9a-fA-F\]*: file .*${srcfile}, line 21.*$gdb_prompt $" {
+set call_line [gdb_get_line_number "bp-on-call"]
+gdb_test_multiple "break $call_line" "break before jump to call" {
+ -re "\[Bb\]reakpoint (${decimal}) at ${hex}: file .*${srcfile}, line $call_line.*$gdb_prompt $" {
set bp_on_call $expect_out(1,string)
pass "break before jump to call"
}
@@ -63,8 +65,8 @@ gdb_test_multiple "break 21" "break before jump to call" {
# Can we jump to the statement? Do we stop there?
#
-gdb_test "jump 21" \
- "Breakpoint \[0-9\]*, .*${srcfile}:21.*" \
+gdb_test "jump $call_line" \
+ "Breakpoint ${decimal}, .*${srcfile}:$call_line.*" \
"jump to call"
# If we disable the breakpoint at the function call, and then
@@ -73,7 +75,7 @@ gdb_test "jump 21" \
#
gdb_test_no_output "disable $bp_on_call" "disable breakpoint on call"
-gdb_test "jump 21" "Breakpoint \[0-9\]*, .*${srcfile}:22.*" \
+gdb_test "jump $call_line" "Breakpoint ${decimal}, .*${srcfile}:$non_call_line.*" \
"jump to call with disabled breakpoint"
# Verify that GDB responds gracefully to the "jump" command without
@@ -86,7 +88,7 @@ gdb_test "jump" "Argument required .starting address.*" \
# Verify that GDB responds gracefully to the "jump" command with
# trailing junk.
#
-gdb_test "jump 21 100" \
+gdb_test "jump $call_line 100" \
"malformed linespec error: unexpected number, \"100\"" \
"jump with trailing argument junk"
@@ -99,16 +101,17 @@ gdb_test "jump 21 100" \
# Try it both ways: confirming and not confirming the jump.
#
-gdb_test "jump 12" \
+set out_line [gdb_get_line_number "out-of-func"]
+gdb_test "jump $out_line" \
"Not confirmed.*" \
"aborted jump out of current function" \
- "Line 12 is not in `main'. Jump anyway.*y or n. $" \
+ "Line $out_line is not in `main'. Jump anyway.*y or n. $" \
"n"
-gdb_test "jump 12" \
+gdb_test "jump $out_line" \
"Continuing at.*" \
"jump out of current function" \
- "Line 12 is not in `main'. Jump anyway.*y or n. $" \
+ "Line $out_line is not in `main'. Jump anyway.*y or n. $" \
"y"
gdb_exit