[binutils-gdb] GDB: testsuite: MI: Don't return value from top-level (manual)
Thiago Bauermann via Gdb-cvs <[email protected]>
| Newsgroups | gmane.comp.gdb.cvs |
|---|---|
| Message-ID | <[email protected]> |
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f89688bc092937f3857fdce39514197dbeea8e2e commit f89688bc092937f3857fdce39514197dbeea8e2e Author: Thiago Jung Bauermann <[email protected]> Date: Wed Apr 29 20:04:18 2026 -0300 GDB: testsuite: MI: Don't return value from top-level (manual) This patch manually changes "return -1" statements that weren't caught by the sed command. It also changes a couple of "return 0" statements. These return statements fall into one of these criteria: - misaligned line, which is then fixed by this patch; - return at top level but inside block such as with_test_prefix, foreach_with_prefix, gdb_expect. Approved-By: Tom Tromey <[email protected]> Diff: --- gdb/testsuite/gdb.mi/gdb2549.exp | 4 ++-- gdb/testsuite/gdb.mi/gdb701.exp | 4 ++-- gdb/testsuite/gdb.mi/gdb792.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-async-run.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-async.exp | 6 +++--- gdb/testsuite/gdb.mi/mi-basics.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-break-qualified.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-break.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-cli.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-cmd-param-changed.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-complete.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-console.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-corefile.exp | 2 +- gdb/testsuite/gdb.mi/mi-detach.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-disassemble.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-eval.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-exec-run.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-file.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-fill-memory.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-frame-regs.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-inheritance-syntax-error.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-pending.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-read-memory.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-reg-undefined.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-regs.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-return.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-reverse.exp | 6 +++--- gdb/testsuite/gdb.mi/mi-simplerun.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-solib.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-stack.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-start.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-stepi.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-syn-frame.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-until.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-var-cmd.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-cp.exp | 6 +++--- gdb/testsuite/gdb.mi/mi-var-rtti.exp | 2 +- gdb/testsuite/gdb.mi/mi-vla-c99.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-vla-fortran.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-watch.exp | 4 ++-- gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp | 14 +++++++------- 41 files changed, 87 insertions(+), 87 deletions(-) diff --git a/gdb/testsuite/gdb.mi/gdb2549.exp b/gdb/testsuite/gdb.mi/gdb2549.exp index c644e3bd1d3..570e1bdeb4a 100644 --- a/gdb/testsuite/gdb.mi/gdb2549.exp +++ b/gdb/testsuite/gdb.mi/gdb2549.exp @@ -28,8 +28,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } proc register_tests_no_exec { } { diff --git a/gdb/testsuite/gdb.mi/gdb701.exp b/gdb/testsuite/gdb.mi/gdb701.exp index d72bd1ae81b..dd6fd6181b2 100644 --- a/gdb/testsuite/gdb.mi/gdb701.exp +++ b/gdb/testsuite/gdb.mi/gdb701.exp @@ -23,8 +23,8 @@ set MIFLAGS "-i=mi" standard_testfile if {[gdb_compile $srcdir/$subdir/$srcfile $binfile executable debug] != ""} { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # When varobj reports the types of objects, it often isn't really reporting diff --git a/gdb/testsuite/gdb.mi/gdb792.exp b/gdb/testsuite/gdb.mi/gdb792.exp index e2df8216956..d0b7a97f02d 100644 --- a/gdb/testsuite/gdb.mi/gdb792.exp +++ b/gdb/testsuite/gdb.mi/gdb792.exp @@ -24,8 +24,8 @@ set MIFLAGS "-i=mi" standard_testfile .cc if {[gdb_compile $srcdir/$subdir/$srcfile $binfile executable {debug c++}] != ""} { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-async-run.exp b/gdb/testsuite/gdb.mi/mi-async-run.exp index 5a4077b6d19..a60a8539322 100644 --- a/gdb/testsuite/gdb.mi/mi-async-run.exp +++ b/gdb/testsuite/gdb.mi/mi-async-run.exp @@ -18,8 +18,8 @@ load_lib mi-support.exp standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # Test the resolution of PR 18077 diff --git a/gdb/testsuite/gdb.mi/mi-async.exp b/gdb/testsuite/gdb.mi/mi-async.exp index d1674cdfe49..7f7603cf706 100644 --- a/gdb/testsuite/gdb.mi/mi-async.exp +++ b/gdb/testsuite/gdb.mi/mi-async.exp @@ -23,7 +23,7 @@ # we're actually testing native. require isnative if {![istarget *-linux*]} { - return + return } # Check if start command is supported. @@ -34,8 +34,8 @@ load_lib mi-support.exp standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # The plan is for async mode to become the default but toggle for now. diff --git a/gdb/testsuite/gdb.mi/mi-basics.exp b/gdb/testsuite/gdb.mi/mi-basics.exp index 27ac334e4a3..8f18f41af04 100644 --- a/gdb/testsuite/gdb.mi/mi-basics.exp +++ b/gdb/testsuite/gdb.mi/mi-basics.exp @@ -37,8 +37,8 @@ set testsubdir [standard_output_file ""] set envirodir [string_to_regexp $testsubdir] if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # In this file we want to test if the operations needed by the following diff --git a/gdb/testsuite/gdb.mi/mi-break-qualified.exp b/gdb/testsuite/gdb.mi/mi-break-qualified.exp index 1bb8258b953..d196c37ef78 100644 --- a/gdb/testsuite/gdb.mi/mi-break-qualified.exp +++ b/gdb/testsuite/gdb.mi/mi-break-qualified.exp @@ -24,8 +24,8 @@ set MIFLAGS "-i=mi" standard_testfile .cc if {[build_executable $testfile.exp $testfile $srcfile {c++ debug}] == -1} { - untested "failed to compile" - return -1 + untested "failed to compile" + return } set loc_ns_func_line [gdb_get_line_number "location NS::func here"] diff --git a/gdb/testsuite/gdb.mi/mi-break.exp b/gdb/testsuite/gdb.mi/mi-break.exp index a7b64faa8a5..b0df2cedc4e 100644 --- a/gdb/testsuite/gdb.mi/mi-break.exp +++ b/gdb/testsuite/gdb.mi/mi-break.exp @@ -26,8 +26,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # Locate line numbers in basics.c. diff --git a/gdb/testsuite/gdb.mi/mi-cli.exp b/gdb/testsuite/gdb.mi/mi-cli.exp index e3a5d3b87a2..9d696e4b291 100644 --- a/gdb/testsuite/gdb.mi/mi-cli.exp +++ b/gdb/testsuite/gdb.mi/mi-cli.exp @@ -25,8 +25,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-cmd-param-changed.exp b/gdb/testsuite/gdb.mi/mi-cmd-param-changed.exp index 29c44c8475c..9d43fc6e21c 100644 --- a/gdb/testsuite/gdb.mi/mi-cmd-param-changed.exp +++ b/gdb/testsuite/gdb.mi/mi-cmd-param-changed.exp @@ -22,8 +22,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } proc test_command_param_changed { } { diff --git a/gdb/testsuite/gdb.mi/mi-complete.exp b/gdb/testsuite/gdb.mi/mi-complete.exp index 4038ae57f8a..54084cab86d 100644 --- a/gdb/testsuite/gdb.mi/mi-complete.exp +++ b/gdb/testsuite/gdb.mi/mi-complete.exp @@ -22,8 +22,8 @@ set MIFLAGS "-i=mi" standard_testfile .cc if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug c++}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-console.exp b/gdb/testsuite/gdb.mi/mi-console.exp index a7a083a5676..23ae1a28099 100644 --- a/gdb/testsuite/gdb.mi/mi-console.exp +++ b/gdb/testsuite/gdb.mi/mi-console.exp @@ -49,8 +49,8 @@ proc semihosted_string { string } { standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-corefile.exp b/gdb/testsuite/gdb.mi/mi-corefile.exp index 67c6183f386..864c8f74ddc 100644 --- a/gdb/testsuite/gdb.mi/mi-corefile.exp +++ b/gdb/testsuite/gdb.mi/mi-corefile.exp @@ -30,7 +30,7 @@ if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} { set corefile [core_find $binfile {}] if {$corefile == ""} { untested "unable to create or find corefile" - return 0 + return } if {[mi_gdb_start]} { diff --git a/gdb/testsuite/gdb.mi/mi-detach.exp b/gdb/testsuite/gdb.mi/mi-detach.exp index 62bd699ea8a..e82e216315e 100644 --- a/gdb/testsuite/gdb.mi/mi-detach.exp +++ b/gdb/testsuite/gdb.mi/mi-detach.exp @@ -21,8 +21,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if {[build_executable $testfile.exp $testfile $srcfile {debug}] == -1} { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-disassemble.exp b/gdb/testsuite/gdb.mi/mi-disassemble.exp index cd4c91bd06a..2ecbce193c7 100644 --- a/gdb/testsuite/gdb.mi/mi-disassemble.exp +++ b/gdb/testsuite/gdb.mi/mi-disassemble.exp @@ -26,8 +26,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } proc test_disassembly_only {} { diff --git a/gdb/testsuite/gdb.mi/mi-eval.exp b/gdb/testsuite/gdb.mi/mi-eval.exp index 6720f878538..f2761f46219 100644 --- a/gdb/testsuite/gdb.mi/mi-eval.exp +++ b/gdb/testsuite/gdb.mi/mi-eval.exp @@ -28,8 +28,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-exec-run.exp b/gdb/testsuite/gdb.mi/mi-exec-run.exp index 9707504aa21..d7a680daa6c 100644 --- a/gdb/testsuite/gdb.mi/mi-exec-run.exp +++ b/gdb/testsuite/gdb.mi/mi-exec-run.exp @@ -35,8 +35,8 @@ set have_startup_shell [have_startup_shell] standard_testfile mi-start.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "could not build mi-exec-run" - return -1 + untested "could not build mi-exec-run" + return } # The test proper. INFTTY_MODE determines whether "set inferior-tty" diff --git a/gdb/testsuite/gdb.mi/mi-file.exp b/gdb/testsuite/gdb.mi/mi-file.exp index 8ca0511f55d..80411b82ff7 100644 --- a/gdb/testsuite/gdb.mi/mi-file.exp +++ b/gdb/testsuite/gdb.mi/mi-file.exp @@ -21,8 +21,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart]} { diff --git a/gdb/testsuite/gdb.mi/mi-fill-memory.exp b/gdb/testsuite/gdb.mi/mi-fill-memory.exp index 524732bebbd..f9bd1320982 100644 --- a/gdb/testsuite/gdb.mi/mi-fill-memory.exp +++ b/gdb/testsuite/gdb.mi/mi-fill-memory.exp @@ -23,8 +23,8 @@ set MIFLAGS "-i=mi" standard_testfile "mi-read-memory" if { [gdb_compile "${srcdir}/${subdir}/${srcfile}.c" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-frame-regs.exp b/gdb/testsuite/gdb.mi/mi-frame-regs.exp index 65e3bd93d4a..dbcda5ef755 100644 --- a/gdb/testsuite/gdb.mi/mi-frame-regs.exp +++ b/gdb/testsuite/gdb.mi/mi-frame-regs.exp @@ -26,8 +26,8 @@ standard_testfile basics.c if {[gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \ executable {debug}] != "" } then { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # Return the address of the specified breakpoint. diff --git a/gdb/testsuite/gdb.mi/mi-inheritance-syntax-error.exp b/gdb/testsuite/gdb.mi/mi-inheritance-syntax-error.exp index d9baa763952..17ed3f1dcef 100644 --- a/gdb/testsuite/gdb.mi/mi-inheritance-syntax-error.exp +++ b/gdb/testsuite/gdb.mi/mi-inheritance-syntax-error.exp @@ -21,8 +21,8 @@ set MIFLAGS "-i=mi" standard_testfile .cc if {[gdb_compile $srcdir/$subdir/$srcfile $binfile executable {debug c++}] != ""} { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-pending.exp b/gdb/testsuite/gdb.mi/mi-pending.exp index 1ad9aecdc8b..51be357d4c0 100644 --- a/gdb/testsuite/gdb.mi/mi-pending.exp +++ b/gdb/testsuite/gdb.mi/mi-pending.exp @@ -141,10 +141,10 @@ set testname "run till MI pending breakpoint on pendfunc3 on thread 2" gdb_expect { -re "\\*stopped,reason=\"breakpoint-hit\",disp=\"keep\",bkptno=\"4\",frame=\{addr=\"$hex\",func=\"pendfunc3\".*thread-id=\"2\".*" { pass $testname - return 0 + return } timeout { fail "$testname (timeout)" - return -1 + return } } diff --git a/gdb/testsuite/gdb.mi/mi-read-memory.exp b/gdb/testsuite/gdb.mi/mi-read-memory.exp index 782c0ee13dc..035c31e2ae8 100644 --- a/gdb/testsuite/gdb.mi/mi-read-memory.exp +++ b/gdb/testsuite/gdb.mi/mi-read-memory.exp @@ -21,8 +21,8 @@ set MIFLAGS "-i=mi" standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-reg-undefined.exp b/gdb/testsuite/gdb.mi/mi-reg-undefined.exp index dd7b2961b03..725925d01a8 100644 --- a/gdb/testsuite/gdb.mi/mi-reg-undefined.exp +++ b/gdb/testsuite/gdb.mi/mi-reg-undefined.exp @@ -25,8 +25,8 @@ require is_x86_64_m64_target standard_testfile .S if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug nopie}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-regs.exp b/gdb/testsuite/gdb.mi/mi-regs.exp index 7aa4fb62c7a..756855d9ded 100644 --- a/gdb/testsuite/gdb.mi/mi-regs.exp +++ b/gdb/testsuite/gdb.mi/mi-regs.exp @@ -28,8 +28,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } proc sparc_register_tests_no_exec { } { diff --git a/gdb/testsuite/gdb.mi/mi-return.exp b/gdb/testsuite/gdb.mi/mi-return.exp index bd90ac79893..c9265c80317 100644 --- a/gdb/testsuite/gdb.mi/mi-return.exp +++ b/gdb/testsuite/gdb.mi/mi-return.exp @@ -27,8 +27,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-reverse.exp b/gdb/testsuite/gdb.mi/mi-reverse.exp index 100857c6e29..dacb86048f6 100644 --- a/gdb/testsuite/gdb.mi/mi-reverse.exp +++ b/gdb/testsuite/gdb.mi/mi-reverse.exp @@ -35,8 +35,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile @@ -48,7 +48,7 @@ if {[supports_process_record]} { "=record-started,thread-group=\"i1\",method=\"full\"\r\n\\^done" \ "Turn on process record"]} { warning "Fail to activate process record/replay, tests in this group will not be performed.\n" - return -1 + return } } diff --git a/gdb/testsuite/gdb.mi/mi-simplerun.exp b/gdb/testsuite/gdb.mi/mi-simplerun.exp index 8f18a63e46c..98bc3da8919 100644 --- a/gdb/testsuite/gdb.mi/mi-simplerun.exp +++ b/gdb/testsuite/gdb.mi/mi-simplerun.exp @@ -30,8 +30,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-solib.exp b/gdb/testsuite/gdb.mi/mi-solib.exp index 0a13df407a0..a3c4d20b000 100644 --- a/gdb/testsuite/gdb.mi/mi-solib.exp +++ b/gdb/testsuite/gdb.mi/mi-solib.exp @@ -30,8 +30,8 @@ set bin_flags [list debug shlib=${binfile_lib}] if { [gdb_compile_shlib ${srcfile_lib} ${binfile_lib} $lib_flags] != "" || [gdb_compile ${srcfile_main} ${binfile} executable $bin_flags] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-stack.exp b/gdb/testsuite/gdb.mi/mi-stack.exp index 2b7884076da..fbfc0619d00 100644 --- a/gdb/testsuite/gdb.mi/mi-stack.exp +++ b/gdb/testsuite/gdb.mi/mi-stack.exp @@ -28,8 +28,8 @@ set MIFLAGS "-i=mi" standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-start.exp b/gdb/testsuite/gdb.mi/mi-start.exp index 5a232f0e6cd..ff18866987f 100644 --- a/gdb/testsuite/gdb.mi/mi-start.exp +++ b/gdb/testsuite/gdb.mi/mi-start.exp @@ -24,8 +24,8 @@ require !use_gdb_stub standard_testfile mi-start.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "could not build mi-start" - return -1 + untested "could not build mi-start" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-stepi.exp b/gdb/testsuite/gdb.mi/mi-stepi.exp index fe8854a7905..8b9ca60ac43 100644 --- a/gdb/testsuite/gdb.mi/mi-stepi.exp +++ b/gdb/testsuite/gdb.mi/mi-stepi.exp @@ -26,8 +26,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } proc test_stepi_nexti {} { diff --git a/gdb/testsuite/gdb.mi/mi-syn-frame.exp b/gdb/testsuite/gdb.mi/mi-syn-frame.exp index 410849a910a..f3b1a71ccb3 100644 --- a/gdb/testsuite/gdb.mi/mi-syn-frame.exp +++ b/gdb/testsuite/gdb.mi/mi-syn-frame.exp @@ -25,8 +25,8 @@ set any "\[^\"\]+" standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-until.exp b/gdb/testsuite/gdb.mi/mi-until.exp index a84a1238ae4..f508ce292e1 100644 --- a/gdb/testsuite/gdb.mi/mi-until.exp +++ b/gdb/testsuite/gdb.mi/mi-until.exp @@ -27,8 +27,8 @@ set MIFLAGS "-i=mi" standard_testfile until.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-cmd.exp b/gdb/testsuite/gdb.mi/mi-var-cmd.exp index bf207dad255..a08528002cb 100644 --- a/gdb/testsuite/gdb.mi/mi-var-cmd.exp +++ b/gdb/testsuite/gdb.mi/mi-var-cmd.exp @@ -592,7 +592,7 @@ lappend inline_tests floating foreach_with_prefix inline_test $inline_tests { if { [mi_run_inline_test $inline_test] < 0 } { - return -1 + return } } diff --git a/gdb/testsuite/gdb.mi/mi-var-cp.exp b/gdb/testsuite/gdb.mi/mi-var-cp.exp index db9efd67fd3..fb850ad51db 100644 --- a/gdb/testsuite/gdb.mi/mi-var-cp.exp +++ b/gdb/testsuite/gdb.mi/mi-var-cp.exp @@ -21,8 +21,8 @@ set MIFLAGS "-i=mi" standard_testfile .cc if {[gdb_compile $srcdir/$subdir/$srcfile $binfile executable {debug c++}] != ""} { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { @@ -41,7 +41,7 @@ set inline_tests { foreach_with_prefix inline_test $inline_tests { if { [mi_run_inline_test $inline_test] < 0 } { - return -1 + return } } diff --git a/gdb/testsuite/gdb.mi/mi-var-rtti.exp b/gdb/testsuite/gdb.mi/mi-var-rtti.exp index 9a98e2adfa9..4a1bee241f9 100644 --- a/gdb/testsuite/gdb.mi/mi-var-rtti.exp +++ b/gdb/testsuite/gdb.mi/mi-var-rtti.exp @@ -118,7 +118,7 @@ set inline_tests { foreach_with_prefix inline_test $inline_tests { if { [mi_run_inline_test $inline_test] < 0 } { - return -1 + return } } diff --git a/gdb/testsuite/gdb.mi/mi-vla-c99.exp b/gdb/testsuite/gdb.mi/mi-vla-c99.exp index 91934de2491..5e3c8f54940 100644 --- a/gdb/testsuite/gdb.mi/mi-vla-c99.exp +++ b/gdb/testsuite/gdb.mi/mi-vla-c99.exp @@ -25,8 +25,8 @@ standard_testfile vla.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" \ "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-vla-fortran.exp b/gdb/testsuite/gdb.mi/mi-vla-fortran.exp index fd144cb51e7..0cc51a36f5e 100644 --- a/gdb/testsuite/gdb.mi/mi-vla-fortran.exp +++ b/gdb/testsuite/gdb.mi/mi-vla-fortran.exp @@ -26,8 +26,8 @@ standard_testfile vla.f90 if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable \ {debug f90}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } # Depending on the compiler being used, diff --git a/gdb/testsuite/gdb.mi/mi-watch.exp b/gdb/testsuite/gdb.mi/mi-watch.exp index 4fcba501224..5b01793aa3d 100644 --- a/gdb/testsuite/gdb.mi/mi-watch.exp +++ b/gdb/testsuite/gdb.mi/mi-watch.exp @@ -34,8 +34,8 @@ set MIFLAGS "-i=mi" standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { - untested "failed to compile" - return -1 + untested "failed to compile" + return } proc test_watchpoint_creation_and_listing {} { diff --git a/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp b/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp index fcea890d78a..adcd713453c 100644 --- a/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp +++ b/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp @@ -44,7 +44,7 @@ foreach name {different breakhere_different breakhere_validity breakhere_invalid with_test_prefix "entry-values=no" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values no" {\^done} "no: set print entry-values" mi_send_resuming_command "exec-continue" "no: entry_equal: continue" @@ -63,7 +63,7 @@ with_test_prefix "entry-values=no" { with_test_prefix "entry-values=only" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values only" {\^done} "only: set print entry-values" mi_send_resuming_command "exec-continue" "only: entry_equal: continue" @@ -82,7 +82,7 @@ with_test_prefix "entry-values=only" { with_test_prefix "entry-values=preferred" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values preferred" {\^done} "preferred: set print entry-values" mi_send_resuming_command "exec-continue" "preferred: entry_equal: continue" @@ -101,7 +101,7 @@ with_test_prefix "entry-values=preferred" { with_test_prefix "entry-values=if-needed" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values if-needed" {\^done} "if-needed: set print entry-values" mi_send_resuming_command "exec-continue" "if-needed: entry_equal: continue" @@ -120,7 +120,7 @@ with_test_prefix "entry-values=if-needed" { with_test_prefix "entry-values=both" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values both" {\^done} "both: set print entry-values" mi_send_resuming_command "exec-continue" "both: entry_equal: continue" @@ -139,7 +139,7 @@ with_test_prefix "entry-values=both" { with_test_prefix "entry-values=compact" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values compact" {\^done} "compact: set print entry-values" mi_send_resuming_command "exec-continue" "compact: entry_equal: continue" @@ -158,7 +158,7 @@ with_test_prefix "entry-values=compact" { with_test_prefix "entry-values=default" { if {[mi_runto_main] == -1} { - return -1 + return } mi_gdb_test "-gdb-set print entry-values default" {\^done} "default: set print entry-values" mi_send_resuming_command "exec-continue" "default: entry_equal: continue"