diff options
| author | Andrew Burgess <aburgess@redhat.com> | 2022-11-14 14:20:53 +0000 |
|---|---|---|
| committer | Andrew Burgess <aburgess@redhat.com> | 2022-11-28 21:04:09 +0000 |
| commit | bde43e8dbeaf4c9b05514f0c749623db453747bb (patch) | |
| tree | 503dc5f53f91f5367401f15522dfa10885abbc09 /gdb/testsuite/gdb.reverse/sigall-reverse.exp | |
| parent | 89c6d320b24567b6385e9dd92b3d4671c9796143 (diff) | |
| download | fsf-binutils-gdb-bde43e8dbeaf4c9b05514f0c749623db453747bb.zip fsf-binutils-gdb-bde43e8dbeaf4c9b05514f0c749623db453747bb.tar.gz fsf-binutils-gdb-bde43e8dbeaf4c9b05514f0c749623db453747bb.tar.bz2 | |
gdb/testsuite: remove use of then keyword from gdb.reverse/*.exp
The canonical form of 'if' in modern TCL is 'if {} {}'. But there's
still a bunch of places in the testsuite where we make use of the
'then' keyword, and sometimes these get copies into new tests, which
just spreads poor practice.
This commit removes all use of the 'then' keyword from the gdb.reverse/
test script directory.
There should be no changes in what is tested after this commit.
Diffstat (limited to 'gdb/testsuite/gdb.reverse/sigall-reverse.exp')
| -rw-r--r-- | gdb/testsuite/gdb.reverse/sigall-reverse.exp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/gdb/testsuite/gdb.reverse/sigall-reverse.exp b/gdb/testsuite/gdb.reverse/sigall-reverse.exp index 36fdb74..7078067 100644 --- a/gdb/testsuite/gdb.reverse/sigall-reverse.exp +++ b/gdb/testsuite/gdb.reverse/sigall-reverse.exp @@ -47,7 +47,7 @@ proc test_one_sig {nextsig} { set need_another_continue 1 set missed_handler 0 - if $this_sig_supported then { + if {$this_sig_supported} { if { $thissig == "IO" } { setup_xfail "i*86-pc-linuxoldld-gnu" "i*86-pc-linuxaout-gnu" } @@ -72,7 +72,7 @@ proc test_one_sig {nextsig} { } } - if $need_another_continue then { + if {$need_another_continue} { if { $thissig == "URG" } { setup_xfail "i*86-pc-linuxoldld-gnu" "i*86-pc-linuxaout-gnu" } @@ -92,7 +92,7 @@ proc test_one_sig {nextsig} { } } - if { $missed_handler == "0" } then { + if {$missed_handler == "0"} { set testmsg "advance to $nextsig" gdb_test_multiple "signal 0" $testmsg { -re "Breakpoint.*gen_$nextsig.*kill.*$gdb_prompt $" { @@ -138,14 +138,14 @@ proc test_one_sig_reverse {prevsig} { exp_continue } -re "Breakpoint.*kill.*$gdb_prompt " { - if { $saw_signal } then { + if {$saw_signal} { pass "$testmsg (handled)" } else { xfail "$testmsg (handled)" } } -re "No more reverse-execution history.*kill.*$gdb_prompt " { - if { $saw_signal } then { + if {$saw_signal} { pass "$testmsg (handled)" } else { xfail "$testmsg (handled)" |