]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit - gdb/testsuite/ChangeLog
[gdb/testsuite] Fix cur_addr update in gdb.base/watchpoint-reuse-slot.exp
authorTom de Vries <tdevries@suse.de>
Wed, 6 May 2020 12:13:02 +0000 (14:13 +0200)
committerTom de Vries <tdevries@suse.de>
Wed, 6 May 2020 12:13:02 +0000 (14:13 +0200)
commitb8983c46637eaafe808f9c5b3f95c10e1f824402
tree720a47f107758a446cc682afc7881b9ca1f09ca1
parentabf6d805a0dfa792fdf232dabd7de18d2fe20834
[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.
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp