diff options
author | Tom de Vries <tdevries@suse.de> | 2020-05-06 14:13:02 +0200 |
---|---|---|
committer | Tom de Vries <tdevries@suse.de> | 2020-05-06 14:13:02 +0200 |
commit | b8983c46637eaafe808f9c5b3f95c10e1f824402 (patch) | |
tree | 720a47f107758a446cc682afc7881b9ca1f09ca1 /gdb/testsuite/gdb.base | |
parent | abf6d805a0dfa792fdf232dabd7de18d2fe20834 (diff) | |
download | binutils-gdb-b8983c46637eaafe808f9c5b3f95c10e1f824402.tar.gz |
[gdb/testsuite] Fix cur_addr update in gdb.base/watchpoint-reuse-slot.exp
I noticed this code in gdb.base/watchpoint-reuse-slot.exp, proc stepi:
...
gdb_test_multiple "stepi" $test {
-re "($hex).*[string_to_regexp $srcline]\r\n$gdb_prompt $" {
set addr $expect_out(1,string)
if {$addr != $cur_addr} {
pass $test
} else {
fail $test
}
set cur_addr addr
}
}
...
The variable cur_addr is documented as:
...
# The address the program is stopped at currently.
set cur_addr ""
...
but in the gdb_test_multiple clause we assign the string "addr" to cur_addr,
while $addr contains the current address.
Fix this by assigning $addr instead "addr".
Tested on x86_64-linux.
gdb/testsuite/ChangeLog:
2020-05-06 Tom de Vries <tdevries@suse.de>
* gdb.base/watchpoint-reuse-slot.exp: Fix incorrect assignment.
Diffstat (limited to 'gdb/testsuite/gdb.base')
-rw-r--r-- | gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp b/gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp index 73213787c61..83b970380ca 100644 --- a/gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp +++ b/gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp @@ -73,7 +73,7 @@ proc stepi {} { } else { fail $test } - set cur_addr addr + set cur_addr $addr } } } |