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/passc-dyn.exp | |
parent | b8ffa8b32737636f1579fc164c625fe41abd61d8 (diff) | |
download | binutils-gdb-89c6d320b24567b6385e9dd92b3d4671c9796143.tar.gz |
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/passc-dyn.exp')
-rw-r--r-- | gdb/testsuite/gdb.trace/passc-dyn.exp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gdb/testsuite/gdb.trace/passc-dyn.exp b/gdb/testsuite/gdb.trace/passc-dyn.exp index 747e4207dd0..63f4788d07b 100644 --- a/gdb/testsuite/gdb.trace/passc-dyn.exp +++ b/gdb/testsuite/gdb.trace/passc-dyn.exp @@ -34,7 +34,7 @@ gdb_load $binfile runto_main gdb_reinitialize_dir $srcdir/$subdir -if { ![gdb_target_supports_trace] } then { +if {![gdb_target_supports_trace]} { unsupported "current target does not support trace" return 1 @@ -50,7 +50,7 @@ if { ![gdb_target_supports_trace] } then { set baseline [gdb_find_recursion_test_baseline $srcfile] -if { $baseline == -1 } then { +if {$baseline == -1} { fail "could not find gdb_recursion_test function" return } @@ -74,7 +74,7 @@ gdb_delete_tracepoints set tdp2 [gdb_gettpnum "$testline2"] set tdp3 [gdb_gettpnum "$testline3"] set tdp4 [gdb_gettpnum "$testline4"] -if { $tdp2 <= 0 || $tdp3 <= 0 || $tdp4 <= 0 } then { +if {$tdp2 <= 0 || $tdp3 <= 0 || $tdp4 <= 0} { fail "setting tracepoints" return } |