diff options
| author | Bruno Larsen <blarsen@redhat.com> | 2022-05-13 13:23:57 -0300 |
|---|---|---|
| committer | Bruno Larsen <blarsen@redhat.com> | 2022-05-16 10:07:43 -0300 |
| commit | cdd4206647d0ef8d54d0fa66b3dfa57a4ec07db9 (patch) | |
| tree | be9e3ffcbe58cd24f751c1c828a91b8b79099784 /gdb/testsuite/gdb.opt | |
| parent | b7ff32f191ed7e708412e9faa31cf691f08ca695 (diff) | |
| download | binutils-gdb-cdd4206647d0ef8d54d0fa66b3dfa57a4ec07db9.tar.gz | |
gdb/testsuite: fix "continue outside of loop" TCL errors
Many test cases had a few lines in the beginning that look like:
if { condition } {
continue
}
Where conditions varied, but were mostly in the form of ![runto_main] or
[skip_*_tests], making it quite clear that this code block was supposed
to finish the test if it entered the code block. This generates TCL
errors, as most of these tests are not inside loops. All cases on which
this was an obvious mistake are changed in this patch.
Diffstat (limited to 'gdb/testsuite/gdb.opt')
| -rw-r--r-- | gdb/testsuite/gdb.opt/fortran-string.exp | 2 | ||||
| -rw-r--r-- | gdb/testsuite/gdb.opt/inline-cmds.exp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/gdb/testsuite/gdb.opt/fortran-string.exp b/gdb/testsuite/gdb.opt/fortran-string.exp index 4c633356160..8d4018e9b8b 100644 --- a/gdb/testsuite/gdb.opt/fortran-string.exp +++ b/gdb/testsuite/gdb.opt/fortran-string.exp @@ -29,7 +29,7 @@ if { [prepare_for_testing "prepare for testing" ${testfile} ${srcfile} \ if ![runto f] then { perror "couldn't run to f" - continue + return } gdb_test_no_output "set print frame-arguments all" diff --git a/gdb/testsuite/gdb.opt/inline-cmds.exp b/gdb/testsuite/gdb.opt/inline-cmds.exp index 7ee1a5a85d9..17ad83672d1 100644 --- a/gdb/testsuite/gdb.opt/inline-cmds.exp +++ b/gdb/testsuite/gdb.opt/inline-cmds.exp @@ -337,7 +337,7 @@ proc mi_cli_step {cli_output_re message} { # CLI output sent to MI's console. with_test_prefix "mi" { if [mi_gdb_start] { - continue + return } mi_gdb_load ${binfile} mi_runto_main |