diff options
author | Andrew Burgess <aburgess@redhat.com> | 2022-11-14 14:18:33 +0000 |
---|---|---|
committer | Andrew Burgess <aburgess@redhat.com> | 2022-11-28 21:04:09 +0000 |
commit | 89c6d320b24567b6385e9dd92b3d4671c9796143 (patch) | |
tree | 4b025b811a7f5400719cd37298dea74e8e28e990 /gdb/testsuite/gdb.trace/actions.exp | |
parent | b8ffa8b32737636f1579fc164c625fe41abd61d8 (diff) | |
download | fsf-binutils-gdb-89c6d320b24567b6385e9dd92b3d4671c9796143.zip fsf-binutils-gdb-89c6d320b24567b6385e9dd92b3d4671c9796143.tar.gz fsf-binutils-gdb-89c6d320b24567b6385e9dd92b3d4671c9796143.tar.bz2 |
gdb/testsuite: remove use of then keyword from gdb.trace/*.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.trace/
test script directory.
There should be no changes in what is tested after this commit.
Diffstat (limited to 'gdb/testsuite/gdb.trace/actions.exp')
-rw-r--r-- | gdb/testsuite/gdb.trace/actions.exp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/testsuite/gdb.trace/actions.exp b/gdb/testsuite/gdb.trace/actions.exp index c14fb27..397b30e 100644 --- a/gdb/testsuite/gdb.trace/actions.exp +++ b/gdb/testsuite/gdb.trace/actions.exp @@ -42,7 +42,7 @@ gdb_file_cmd $binfile # all subsequent line numbers are relative to this first one (baseline) set baseline [gdb_find_recursion_test_baseline $srcfile] -if { $baseline == -1 } then { +if {$baseline == -1} { fail "could not find gdb_recursion_test function" return } @@ -57,7 +57,7 @@ gdb_delete_tracepoints set trcpt1 [gdb_gettpnum gdb_c_test] set trcpt2 [gdb_gettpnum gdb_asm_test] set trcpt3 [gdb_gettpnum $testline1] -if { $trcpt1 <= 0 || $trcpt2 <= 0 || $trcpt3 <= 0 } then { +if {$trcpt1 <= 0 || $trcpt2 <= 0 || $trcpt3 <= 0} { fail "setting tracepoints" return } |