aboutsummaryrefslogtreecommitdiff
path: root/gdb/testsuite/gdb.trace/actions.exp
diff options
context:
space:
mode:
authorAndrew Burgess <aburgess@redhat.com>2022-11-14 14:18:33 +0000
committerAndrew Burgess <aburgess@redhat.com>2022-11-28 21:04:09 +0000
commit89c6d320b24567b6385e9dd92b3d4671c9796143 (patch)
tree4b025b811a7f5400719cd37298dea74e8e28e990 /gdb/testsuite/gdb.trace/actions.exp
parentb8ffa8b32737636f1579fc164c625fe41abd61d8 (diff)
downloadfsf-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.exp4
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
}