[gdb/testsuite] Fix count usage in gdb.reverse/insn-reverse.exp
authorTom de Vries <tdevries@suse.de>
Fri, 4 Dec 2020 12:36:47 +0000 (13:36 +0100)
committerTom de Vries <tdevries@suse.de>
Fri, 4 Dec 2020 12:36:47 +0000 (13:36 +0100)
commitf17727b3497a630b01dffd219cdc372a058be121
treed972f1a0d938b3eb66905b4c489a5f6b7cca2a48
parenta1499830feea0dc134e8b08d7cc5b6a52e370294
[gdb/testsuite] Fix count usage in gdb.reverse/insn-reverse.exp

Consider the test-case gdb.reverse/insn-reverse.exp.

After the loop setting count, the valid entries in various arrays range from 0
to $count - 1 inclusive.

Then $count is decremented:
...
       incr count -1
...
after which the valid entries range from 0 to $count inclusive.

The first subsequent loop handles that properly:
...
       for {set i $count} {$i >= 0} {incr i -1} {
...
but the following loop does not, because it treats $count as exclusive bound:
...
for {set i 0} {$i < $count} {incr i} {
...

Fix this by removing the incr, and using $count - 1 as starting value in the
first loop.

gdb/testsuite/ChangeLog:

2020-12-04  Tom de Vries  <tdevries@suse.de>

* gdb.reverse/insn-reverse.exp: Fix count handling.
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.reverse/insn-reverse.exp