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/deltrace.exp | |
parent | b8ffa8b32737636f1579fc164c625fe41abd61d8 (diff) | |
download | gdb-89c6d320b24567b6385e9dd92b3d4671c9796143.zip gdb-89c6d320b24567b6385e9dd92b3d4671c9796143.tar.gz 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/deltrace.exp')
-rw-r--r-- | gdb/testsuite/gdb.trace/deltrace.exp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gdb/testsuite/gdb.trace/deltrace.exp b/gdb/testsuite/gdb.trace/deltrace.exp index 1e2f10f..605223a 100644 --- a/gdb/testsuite/gdb.trace/deltrace.exp +++ b/gdb/testsuite/gdb.trace/deltrace.exp @@ -43,7 +43,7 @@ gdb_file_cmd $binfile # define relative source line numbers: # 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 } @@ -81,7 +81,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 } @@ -159,7 +159,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 } |