[binutils-gdb] GDB: testsuite: MI: Don't return -1 from top-level (sed)
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=89876b0f94850d4aa2b5dbf3cc1cffdca9a4fac4 commit 89876b0f94850d4aa2b5dbf3cc1cffdca9a4fac4 Author: Thiago Jung Bauermann <[email protected]> Date: Wed Apr 29 20:04:17 2026 -0300 GDB: testsuite: MI: Don't return -1 from top-level (sed) This patch is purely the result of running: $ sed -i 's/^ return -1/ return/' *.exp* inside gdb/testsuite/gdb.mi. Approved-By: Tom Tromey <[email protected]> Diff: --- gdb/testsuite/gdb.mi/dw2-ref-missing-frame.exp | 2 +- gdb/testsuite/gdb.mi/gdb669.exp | 2 +- gdb/testsuite/gdb.mi/interrupt-thread-group.exp | 2 +- gdb/testsuite/gdb.mi/list-thread-groups-available.exp | 2 +- gdb/testsuite/gdb.mi/mi-add-inferior.exp | 2 +- gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-breakpoint-multiple-locations.exp | 2 +- gdb/testsuite/gdb.mi/mi-breakpoint-script.exp | 2 +- gdb/testsuite/gdb.mi/mi-catch-cpp-exceptions.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-catch-load.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-cmd-user-context.exp | 2 +- gdb/testsuite/gdb.mi/mi-condbreak-fail.exp | 2 +- gdb/testsuite/gdb.mi/mi-condbreak-throw.exp | 2 +- gdb/testsuite/gdb.mi/mi-corefile.exp | 2 +- gdb/testsuite/gdb.mi/mi-dprintf-pending.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-dprintf.exp | 2 +- gdb/testsuite/gdb.mi/mi-exit-code.exp | 2 +- gdb/testsuite/gdb.mi/mi-file-transfer.exp | 2 +- gdb/testsuite/gdb.mi/mi-fortran-modules.exp | 2 +- gdb/testsuite/gdb.mi/mi-fullname-deleted.exp | 6 +++--- gdb/testsuite/gdb.mi/mi-info-os.exp | 6 +++--- gdb/testsuite/gdb.mi/mi-info-sources.exp | 2 +- gdb/testsuite/gdb.mi/mi-linespec-err-cp.exp | 2 +- gdb/testsuite/gdb.mi/mi-logging.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-memory-changed.exp | 2 +- gdb/testsuite/gdb.mi/mi-nonstop-exit.exp | 2 +- gdb/testsuite/gdb.mi/mi-nonstop.exp | 2 +- gdb/testsuite/gdb.mi/mi-ns-stale-regcache.exp | 2 +- gdb/testsuite/gdb.mi/mi-nsintrall.exp | 2 +- gdb/testsuite/gdb.mi/mi-nsmoribund.exp | 2 +- gdb/testsuite/gdb.mi/mi-nsthrexec.exp | 2 +- gdb/testsuite/gdb.mi/mi-pending.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-pthreads.exp | 2 +- gdb/testsuite/gdb.mi/mi-py-modify-bp.exp | 2 +- gdb/testsuite/gdb.mi/mi-record-changed.exp | 2 +- gdb/testsuite/gdb.mi/mi-reg-undefined.exp | 2 +- gdb/testsuite/gdb.mi/mi-stepn.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-sym-info.exp | 2 +- gdb/testsuite/gdb.mi/mi-thread-bp-deleted.exp | 2 +- gdb/testsuite/gdb.mi/mi-thread-specific-bp.exp | 2 +- gdb/testsuite/gdb.mi/mi-threads-interrupt.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-block.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-child-f.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-child.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-var-cmd.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-create-rtti.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-var-display.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-invalidate-shlib.exp | 4 ++-- gdb/testsuite/gdb.mi/mi-var-invalidate.exp | 6 +++--- gdb/testsuite/gdb.mi/mi-var-list-children-invalid-grandchild.exp | 2 +- gdb/testsuite/gdb.mi/mi-var-rtti.exp | 2 +- gdb/testsuite/gdb.mi/mi-watch-nonstop.exp | 2 +- gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp | 2 +- gdb/testsuite/gdb.mi/mi2-cli-display.exp | 2 +- gdb/testsuite/gdb.mi/mi2-var-child.exp | 2 +- gdb/testsuite/gdb.mi/new-ui-bp-deleted.exp | 2 +- gdb/testsuite/gdb.mi/new-ui-mi-sync.exp | 2 +- gdb/testsuite/gdb.mi/print-simple-values.exp | 2 +- gdb/testsuite/gdb.mi/user-selected-context-sync.exp | 2 +- 59 files changed, 75 insertions(+), 75 deletions(-) diff --git a/gdb/testsuite/gdb.mi/dw2-ref-missing-frame.exp b/gdb/testsuite/gdb.mi/dw2-ref-missing-frame.exp index ad0dd162c49..94f42fe77ab 100644 --- a/gdb/testsuite/gdb.mi/dw2-ref-missing-frame.exp +++ b/gdb/testsuite/gdb.mi/dw2-ref-missing-frame.exp @@ -39,7 +39,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" $objsfile object {}] != "" || [gdb_compile "${srcdir}/${subdir}/${srcfile2}" $objfuncfile object {}] != "" || [gdb_compile "${srcdir}/${subdir}/${srcfile3}" $objmainfile object {debug}] != "" || [gdb_compile "$objfuncfile $objsfile $objmainfile" $binfile executable {}] != "" } { - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/gdb669.exp b/gdb/testsuite/gdb.mi/gdb669.exp index a07117a3804..b03577d8574 100644 --- a/gdb/testsuite/gdb.mi/gdb669.exp +++ b/gdb/testsuite/gdb.mi/gdb669.exp @@ -27,7 +27,7 @@ standard_testfile pthreads.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/interrupt-thread-group.exp b/gdb/testsuite/gdb.mi/interrupt-thread-group.exp index 17a2a42c5e6..f2c45a9dc27 100644 --- a/gdb/testsuite/gdb.mi/interrupt-thread-group.exp +++ b/gdb/testsuite/gdb.mi/interrupt-thread-group.exp @@ -25,7 +25,7 @@ standard_testfile .c if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile \ executable {debug additional_flags=-std=gnu99}] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/list-thread-groups-available.exp b/gdb/testsuite/gdb.mi/list-thread-groups-available.exp index 426e39ff2b6..77f932cd167 100644 --- a/gdb/testsuite/gdb.mi/list-thread-groups-available.exp +++ b/gdb/testsuite/gdb.mi/list-thread-groups-available.exp @@ -26,7 +26,7 @@ require can_spawn_for_attach if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_gdb_start]} { diff --git a/gdb/testsuite/gdb.mi/mi-add-inferior.exp b/gdb/testsuite/gdb.mi/mi-add-inferior.exp index fc9002bd627..607eaacdf22 100644 --- a/gdb/testsuite/gdb.mi/mi-add-inferior.exp +++ b/gdb/testsuite/gdb.mi/mi-add-inferior.exp @@ -23,7 +23,7 @@ standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \ executable {debug}] != "" } { untested "failed to compile" - return -1 + return } mi_clean_restart ${::testfile} diff --git a/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp b/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp index 78e2ebc3ea8..688ad46f1b3 100644 --- a/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp +++ b/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp @@ -32,7 +32,7 @@ set lib_opts "debug" if { [gdb_compile_shlib $libsrc1 $lib_sl1 $lib_opts] != "" || [gdb_compile_shlib $libsrc2 $lib_sl2 $lib_opts] != ""} { untested "failed to compile shared library" - return -1 + return } set MIFLAGS "-i=mi" @@ -40,7 +40,7 @@ set MIFLAGS "-i=mi" set exec_opts [list debug shlib=$lib_sl1 shlib_load] if { [gdb_compile $srcdir/$subdir/$srcfile $binfile executable $exec_opts] != "" } { untested "failed to compile" - return -1 + return } proc test_insert_delete_modify { } { diff --git a/gdb/testsuite/gdb.mi/mi-breakpoint-multiple-locations.exp b/gdb/testsuite/gdb.mi/mi-breakpoint-multiple-locations.exp index eb20460f759..95097568624 100644 --- a/gdb/testsuite/gdb.mi/mi-breakpoint-multiple-locations.exp +++ b/gdb/testsuite/gdb.mi/mi-breakpoint-multiple-locations.exp @@ -19,7 +19,7 @@ load_lib mi-support.exp standard_testfile .cc if {[gdb_compile "$srcdir/$subdir/$srcfile" $binfile executable {debug c++}] != "" } { - return -1 + return } # Generate the regexp pattern used to match the breakpoint description emitted diff --git a/gdb/testsuite/gdb.mi/mi-breakpoint-script.exp b/gdb/testsuite/gdb.mi/mi-breakpoint-script.exp index 70a03d9e0a0..0ec3b4d1abc 100644 --- a/gdb/testsuite/gdb.mi/mi-breakpoint-script.exp +++ b/gdb/testsuite/gdb.mi/mi-breakpoint-script.exp @@ -20,7 +20,7 @@ load_lib mi-support.exp standard_testfile .c if {[gdb_compile "$srcdir/$subdir/$srcfile" $binfile executable {debug}] != "" } { - return -1 + return } # Generate the regexp pattern used to match the breakpoint description emitted diff --git a/gdb/testsuite/gdb.mi/mi-catch-cpp-exceptions.exp b/gdb/testsuite/gdb.mi/mi-catch-cpp-exceptions.exp index ef3164034e7..3581bbe8d79 100644 --- a/gdb/testsuite/gdb.mi/mi-catch-cpp-exceptions.exp +++ b/gdb/testsuite/gdb.mi/mi-catch-cpp-exceptions.exp @@ -24,12 +24,12 @@ standard_testfile .cc if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug c++}] != "" } { untested "failed to compile" - return -1 + return } mi_clean_restart $::testfile if {[mi_runto_main] < 0} { - return -1 + return } set libstdcxx_probe_tests_supported [expr {![mi_skip_libstdcxx_probe_tests]}] diff --git a/gdb/testsuite/gdb.mi/mi-catch-load.exp b/gdb/testsuite/gdb.mi/mi-catch-load.exp index 4afd9154eb3..e42f9c075ab 100644 --- a/gdb/testsuite/gdb.mi/mi-catch-load.exp +++ b/gdb/testsuite/gdb.mi/mi-catch-load.exp @@ -20,7 +20,7 @@ standard_testfile mi-catch-load.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug shlib_load}] != "" } { untested "failed to compile" - return -1 + return } set testfile2 mi-catch-load-so @@ -29,7 +29,7 @@ set binfile2 [standard_output_file ${testfile2}.so] set binfile2_dlopen [shlib_target_file ${testfile2}.so] if { [gdb_compile_shlib "${srcdir}/${subdir}/${srcfile2}" ${binfile2} {debug}] != "" } { untested "failed to compile shared library" - return -1 + return } gdb_download_shlib $binfile2 diff --git a/gdb/testsuite/gdb.mi/mi-cmd-user-context.exp b/gdb/testsuite/gdb.mi/mi-cmd-user-context.exp index 6e5be0d2776..7ee1949e440 100644 --- a/gdb/testsuite/gdb.mi/mi-cmd-user-context.exp +++ b/gdb/testsuite/gdb.mi/mi-cmd-user-context.exp @@ -22,7 +22,7 @@ standard_testfile if {[build_executable $testfile.exp $testfile ${srcfile} "debug pthreads"] == -1} { untested "failed to compile" - return -1 + return } set main_break_line [gdb_get_line_number "main break line"] diff --git a/gdb/testsuite/gdb.mi/mi-condbreak-fail.exp b/gdb/testsuite/gdb.mi/mi-condbreak-fail.exp index db3b61e320f..99629840e04 100644 --- a/gdb/testsuite/gdb.mi/mi-condbreak-fail.exp +++ b/gdb/testsuite/gdb.mi/mi-condbreak-fail.exp @@ -23,7 +23,7 @@ set MIFLAGS "-i=mi" standard_testfile if {[build_executable ${testfile}.exp ${binfile} ${srcfile}]} { - return -1 + return } # Create a breakpoint with a condition that invokes an inferior diff --git a/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp b/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp index 97e671cd2da..e367f7a2420 100644 --- a/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp +++ b/gdb/testsuite/gdb.mi/mi-condbreak-throw.exp @@ -26,7 +26,7 @@ set MIFLAGS "-i=mi" standard_testfile .cc if {[build_executable ${testfile}.exp ${binfile} ${srcfile} {debug c++}]} { - return -1 + return } # Create a breakpoint with a condition that invokes an inferior diff --git a/gdb/testsuite/gdb.mi/mi-corefile.exp b/gdb/testsuite/gdb.mi/mi-corefile.exp index d3fd841abf4..67c6183f386 100644 --- a/gdb/testsuite/gdb.mi/mi-corefile.exp +++ b/gdb/testsuite/gdb.mi/mi-corefile.exp @@ -24,7 +24,7 @@ standard_testfile if {[build_executable $testfile.exp $testfile $srcfile debug] == -1} { untested "failed to compile" - return -1 + return } set corefile [core_find $binfile {}] diff --git a/gdb/testsuite/gdb.mi/mi-dprintf-pending.exp b/gdb/testsuite/gdb.mi/mi-dprintf-pending.exp index f9767fb8c27..b2e1cc3438c 100644 --- a/gdb/testsuite/gdb.mi/mi-dprintf-pending.exp +++ b/gdb/testsuite/gdb.mi/mi-dprintf-pending.exp @@ -31,12 +31,12 @@ set exec_opts [list debug shlib=$lib_sl1 shlib_load] if {[gdb_compile_shlib $libsrc1 $lib_sl1 $lib_opts] != ""} { untested "failed to compile shared library" - return -1 + return } if {[gdb_compile $srcdir/$subdir/$srcfile $binfile executable $exec_opts] != ""} { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-dprintf.exp b/gdb/testsuite/gdb.mi/mi-dprintf.exp index 3f0e4dba1d9..9b2c192e504 100644 --- a/gdb/testsuite/gdb.mi/mi-dprintf.exp +++ b/gdb/testsuite/gdb.mi/mi-dprintf.exp @@ -25,7 +25,7 @@ lappend_include_file flags $srcdir/lib/unbuffer_output.c if {[build_executable $testfile.exp $testfile $srcfile $flags] == -1} { untested "failed to compile" - return -1 + return } set bp_location1 [gdb_get_line_number "set breakpoint 1 here"] diff --git a/gdb/testsuite/gdb.mi/mi-exit-code.exp b/gdb/testsuite/gdb.mi/mi-exit-code.exp index c12d1158c07..2e48c7ce028 100644 --- a/gdb/testsuite/gdb.mi/mi-exit-code.exp +++ b/gdb/testsuite/gdb.mi/mi-exit-code.exp @@ -20,7 +20,7 @@ standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } proc test_list_thread_groups { } { diff --git a/gdb/testsuite/gdb.mi/mi-file-transfer.exp b/gdb/testsuite/gdb.mi/mi-file-transfer.exp index 7a42766964e..5fc137ba5a0 100644 --- a/gdb/testsuite/gdb.mi/mi-file-transfer.exp +++ b/gdb/testsuite/gdb.mi/mi-file-transfer.exp @@ -26,7 +26,7 @@ standard_testfile basics.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart]} { diff --git a/gdb/testsuite/gdb.mi/mi-fortran-modules.exp b/gdb/testsuite/gdb.mi/mi-fortran-modules.exp index 2d874612e05..c08d5c2038a 100644 --- a/gdb/testsuite/gdb.mi/mi-fortran-modules.exp +++ b/gdb/testsuite/gdb.mi/mi-fortran-modules.exp @@ -25,7 +25,7 @@ standard_testfile "mi-fortran-modules.f90" "mi-fortran-modules-2.f90" if {[build_executable "failed to prepare" ${testfile} \ [list $srcfile2 $srcfile] {debug f90}]} { - return -1 + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-fullname-deleted.exp b/gdb/testsuite/gdb.mi/mi-fullname-deleted.exp index 803c07b3ea0..9f56e03d747 100644 --- a/gdb/testsuite/gdb.mi/mi-fullname-deleted.exp +++ b/gdb/testsuite/gdb.mi/mi-fullname-deleted.exp @@ -32,13 +32,13 @@ set srcfileabs [standard_output_file $srcfile] # not match" test when using Clang (it can never fail). if { [regsub {/[^/]+$} $srcfileabs {/\0} srcfileabs] != 1 } { xfail "cannot double the last slash separator" - return -1 + return } if { [regsub {^(/[^/]+)/} $srcfileabs {\1subst/} srcfileabssubst] != 1 || [regsub {^(/[^/]+)/.*$} $srcfileabs {\1} initdir] != 1 } { xfail "missing root subdirectory" - return -1 + return } # Generate a regular expression which to match $srcfileabs with @@ -53,7 +53,7 @@ close $f if { [gdb_compile "$srcfileabs" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } file delete -- $srcfileabs diff --git a/gdb/testsuite/gdb.mi/mi-info-os.exp b/gdb/testsuite/gdb.mi/mi-info-os.exp index 9adc8d31c4e..f802d8cfd92 100644 --- a/gdb/testsuite/gdb.mi/mi-info-os.exp +++ b/gdb/testsuite/gdb.mi/mi-info-os.exp @@ -19,7 +19,7 @@ set MIFLAGS "-i=mi" # This test is Linux-only. if {![istarget *-*-linux*]} { unsupported "mi-info-os.exp" - return -1 + return } # Support for XML-output is needed to run this test. @@ -29,7 +29,7 @@ standard_testfile basics.c if {[build_executable "Failed to build $testfile" $testfile $srcfile \ debug]} { - return -1; + return; } if {[mi_clean_restart $::testfile]} { @@ -44,7 +44,7 @@ if {[mi_clean_restart $::testfile]} { # running any program, so we want to avoid "run". Using # mi_gdb_target_load directly instead achieves this. if {[mi_gdb_target_load] < 0} { - return -1 + return } # Try the argument-less form that lists all the types in a table. diff --git a/gdb/testsuite/gdb.mi/mi-info-sources.exp b/gdb/testsuite/gdb.mi/mi-info-sources.exp index ab92fff197c..0b3016c0b55 100644 --- a/gdb/testsuite/gdb.mi/mi-info-sources.exp +++ b/gdb/testsuite/gdb.mi/mi-info-sources.exp @@ -23,7 +23,7 @@ standard_testfile .c -base.c if {[build_executable $testfile.exp $testfile \ [list $srcfile $srcfile2] debug]} { untested $testfile.exp - return -1 + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-linespec-err-cp.exp b/gdb/testsuite/gdb.mi/mi-linespec-err-cp.exp index b06f4686e08..91261597fa9 100644 --- a/gdb/testsuite/gdb.mi/mi-linespec-err-cp.exp +++ b/gdb/testsuite/gdb.mi/mi-linespec-err-cp.exp @@ -26,7 +26,7 @@ standard_testfile .cc set exefile $testfile if {[build_executable "failed to prepare" $exefile $srcfile {debug c++}]} { - return -1 + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-logging.exp b/gdb/testsuite/gdb.mi/mi-logging.exp index 6cff709cfc5..1ed0d229bae 100644 --- a/gdb/testsuite/gdb.mi/mi-logging.exp +++ b/gdb/testsuite/gdb.mi/mi-logging.exp @@ -21,13 +21,13 @@ set opts {debug} if {[build_executable $testfile.exp $testfile $srcfile $opts]} { untested "failed to compile" - return -1 + return } mi_clean_restart $::testfile if {[mi_runto_main] < 0} { - return -1 + return } set milogfile [standard_output_file "milog.txt"] diff --git a/gdb/testsuite/gdb.mi/mi-memory-changed.exp b/gdb/testsuite/gdb.mi/mi-memory-changed.exp index 6f41ecd8ad3..916d240b09b 100644 --- a/gdb/testsuite/gdb.mi/mi-memory-changed.exp +++ b/gdb/testsuite/gdb.mi/mi-memory-changed.exp @@ -17,7 +17,7 @@ standard_testfile basics.c if { [gdb_compile "$srcdir/$subdir/$srcfile" $binfile \ executable {debug nowarnings}] != "" } { untested "failed to compile" - return -1 + return } load_lib mi-support.exp diff --git a/gdb/testsuite/gdb.mi/mi-nonstop-exit.exp b/gdb/testsuite/gdb.mi/mi-nonstop-exit.exp index 8958755c57a..ba62f74bf01 100644 --- a/gdb/testsuite/gdb.mi/mi-nonstop-exit.exp +++ b/gdb/testsuite/gdb.mi/mi-nonstop-exit.exp @@ -25,7 +25,7 @@ standard_testfile non-stop-exit.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/mi-nonstop.exp b/gdb/testsuite/gdb.mi/mi-nonstop.exp index e8504919158..970b11a2f8e 100644 --- a/gdb/testsuite/gdb.mi/mi-nonstop.exp +++ b/gdb/testsuite/gdb.mi/mi-nonstop.exp @@ -35,7 +35,7 @@ standard_testfile non-stop.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } diff --git a/gdb/testsuite/gdb.mi/mi-ns-stale-regcache.exp b/gdb/testsuite/gdb.mi/mi-ns-stale-regcache.exp index 34257ee5c77..d5f6a1ec16b 100644 --- a/gdb/testsuite/gdb.mi/mi-ns-stale-regcache.exp +++ b/gdb/testsuite/gdb.mi/mi-ns-stale-regcache.exp @@ -38,7 +38,7 @@ standard_testfile ns-stale-regcache.c set options [list debug] if {[gdb_compile "$srcdir/$subdir/$srcfile" \ $binfile executable $options] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/mi-nsintrall.exp b/gdb/testsuite/gdb.mi/mi-nsintrall.exp index 84319b142a5..7850b5ad98f 100644 --- a/gdb/testsuite/gdb.mi/mi-nsintrall.exp +++ b/gdb/testsuite/gdb.mi/mi-nsintrall.exp @@ -25,7 +25,7 @@ standard_testfile nsintrall.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/mi-nsmoribund.exp b/gdb/testsuite/gdb.mi/mi-nsmoribund.exp index 04c6162d87b..b82996b72b9 100644 --- a/gdb/testsuite/gdb.mi/mi-nsmoribund.exp +++ b/gdb/testsuite/gdb.mi/mi-nsmoribund.exp @@ -25,7 +25,7 @@ standard_testfile nsmoribund.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/mi-nsthrexec.exp b/gdb/testsuite/gdb.mi/mi-nsthrexec.exp index 9446d6f3c6c..64b27eacf91 100644 --- a/gdb/testsuite/gdb.mi/mi-nsthrexec.exp +++ b/gdb/testsuite/gdb.mi/mi-nsthrexec.exp @@ -30,7 +30,7 @@ standard_testfile nsthrexec.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/mi-pending.exp b/gdb/testsuite/gdb.mi/mi-pending.exp index 99719be76a1..1ad9aecdc8b 100644 --- a/gdb/testsuite/gdb.mi/mi-pending.exp +++ b/gdb/testsuite/gdb.mi/mi-pending.exp @@ -36,12 +36,12 @@ set exec_opts [list debug shlib=$lib_sl1 shlib_load] if { [gdb_compile_shlib $libsrc1 $lib_sl1 $lib_opts] != "" || [gdb_compile_shlib $libsrc2 $lib_sl2 $lib_opts] != ""} { untested "failed to compile shared library" - return -1 + return } if { [gdb_compile_pthreads $srcdir/$subdir/$srcfile $binfile executable $exec_opts] != ""} { untested "failed to compile" - return -1 + return } # Start with a fresh gdb. diff --git a/gdb/testsuite/gdb.mi/mi-pthreads.exp b/gdb/testsuite/gdb.mi/mi-pthreads.exp index 7e1d0123500..1db1aba5afa 100644 --- a/gdb/testsuite/gdb.mi/mi-pthreads.exp +++ b/gdb/testsuite/gdb.mi/mi-pthreads.exp @@ -60,7 +60,7 @@ standard_testfile pthreads.c set options [list debug] if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable $options] != "" } { - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-py-modify-bp.exp b/gdb/testsuite/gdb.mi/mi-py-modify-bp.exp index 022f6761362..5b4ba362ef0 100644 --- a/gdb/testsuite/gdb.mi/mi-py-modify-bp.exp +++ b/gdb/testsuite/gdb.mi/mi-py-modify-bp.exp @@ -25,7 +25,7 @@ standard_testfile require allow_python_tests if {[build_executable $testfile.exp $testfile $srcfile] == -1} { - return -1 + return } set remote_python_file [gdb_remote_download host \ diff --git a/gdb/testsuite/gdb.mi/mi-record-changed.exp b/gdb/testsuite/gdb.mi/mi-record-changed.exp index e4e54f1adf1..1f4dbf7b686 100644 --- a/gdb/testsuite/gdb.mi/mi-record-changed.exp +++ b/gdb/testsuite/gdb.mi/mi-record-changed.exp @@ -19,7 +19,7 @@ standard_testfile basics.c if { [gdb_compile "$srcdir/$subdir/$srcfile" $binfile \ executable {debug nowarnings}] != "" } { untested "failed to compile" - return -1 + return } load_lib mi-support.exp diff --git a/gdb/testsuite/gdb.mi/mi-reg-undefined.exp b/gdb/testsuite/gdb.mi/mi-reg-undefined.exp index 5b6e998d7d7..dd7b2961b03 100644 --- a/gdb/testsuite/gdb.mi/mi-reg-undefined.exp +++ b/gdb/testsuite/gdb.mi/mi-reg-undefined.exp @@ -35,7 +35,7 @@ if {[mi_clean_restart $::testfile]} { if {![mi_runto stop_frame]} { perror "Failed to stop in stop_frame" - return -1 + return } mi_gdb_test "111-stack-list-frames" \ diff --git a/gdb/testsuite/gdb.mi/mi-stepn.exp b/gdb/testsuite/gdb.mi/mi-stepn.exp index 42255de20b3..77a32060203 100644 --- a/gdb/testsuite/gdb.mi/mi-stepn.exp +++ b/gdb/testsuite/gdb.mi/mi-stepn.exp @@ -23,13 +23,13 @@ standard_testfile set opts {debug} if {[build_executable ${testfile}.exp ${testfile} ${srcfile} $opts]} { - return -1 + return } mi_clean_restart $::testfile if {[mi_runto_main] < 0} { - return -1 + return } mi_create_breakpoint do_nothing \ diff --git a/gdb/testsuite/gdb.mi/mi-sym-info.exp b/gdb/testsuite/gdb.mi/mi-sym-info.exp index 96d261ded40..e129d5746a3 100644 --- a/gdb/testsuite/gdb.mi/mi-sym-info.exp +++ b/gdb/testsuite/gdb.mi/mi-sym-info.exp @@ -30,7 +30,7 @@ standard_testfile mi-sym-info-1.c mi-sym-info-2.c if {[build_executable "failed to prepare" ${testfile} \ [list $srcfile $srcfile2] {debug}]} { - return -1 + return } mi_clean_restart $::testfile diff --git a/gdb/testsuite/gdb.mi/mi-thread-bp-deleted.exp b/gdb/testsuite/gdb.mi/mi-thread-bp-deleted.exp index bd7b606b87c..e80475911fc 100644 --- a/gdb/testsuite/gdb.mi/mi-thread-bp-deleted.exp +++ b/gdb/testsuite/gdb.mi/mi-thread-bp-deleted.exp @@ -32,7 +32,7 @@ standard_testfile if { [build_executable "failed to prepare" $testfile $srcfile \ {debug pthreads}] } { - return -1 + return } foreach_mi_ui_mode mode { diff --git a/gdb/testsuite/gdb.mi/mi-thread-specific-bp.exp b/gdb/testsuite/gdb.mi/mi-thread-specific-bp.exp index 9f250912b2c..ac76a5579c0 100644 --- a/gdb/testsuite/gdb.mi/mi-thread-specific-bp.exp +++ b/gdb/testsuite/gdb.mi/mi-thread-specific-bp.exp @@ -26,7 +26,7 @@ if {[mi_clean_restart]} { standard_testfile if {[build_executable ${testfile}.exp ${binfile} ${srcfile}]} { - return -1 + return } proc make_loc {num} { diff --git a/gdb/testsuite/gdb.mi/mi-threads-interrupt.exp b/gdb/testsuite/gdb.mi/mi-threads-interrupt.exp index 0d8b091d0ac..5de4fc06c80 100644 --- a/gdb/testsuite/gdb.mi/mi-threads-interrupt.exp +++ b/gdb/testsuite/gdb.mi/mi-threads-interrupt.exp @@ -21,7 +21,7 @@ set MIFLAGS "-i=mi" standard_testfile if {[gdb_compile_pthreads "$srcdir/$subdir/$srcfile" $binfile executable {debug}] != "" } { - return -1 + return } # This tests the resolution of PR 20039. diff --git a/gdb/testsuite/gdb.mi/mi-var-block.exp b/gdb/testsuite/gdb.mi/mi-var-block.exp index e2450527de6..27836717ef8 100644 --- a/gdb/testsuite/gdb.mi/mi-var-block.exp +++ b/gdb/testsuite/gdb.mi/mi-var-block.exp @@ -26,7 +26,7 @@ standard_testfile var-cmd.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-child-f.exp b/gdb/testsuite/gdb.mi/mi-var-child-f.exp index cb249364b68..ad81441db05 100644 --- a/gdb/testsuite/gdb.mi/mi-var-child-f.exp +++ b/gdb/testsuite/gdb.mi/mi-var-child-f.exp @@ -25,7 +25,7 @@ standard_testfile array.f90 if {[gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \ executable {debug f90}] != ""} { - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-child.exp b/gdb/testsuite/gdb.mi/mi-var-child.exp index cbc672a14c2..8c0b8472272 100644 --- a/gdb/testsuite/gdb.mi/mi-var-child.exp +++ b/gdb/testsuite/gdb.mi/mi-var-child.exp @@ -26,7 +26,7 @@ standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { @@ -1201,7 +1201,7 @@ mi_varobj_update * {psnp->ptrs.0.next.next.long_ptr} \ mi_prepare_inline_tests $srcfile if { [mi_run_inline_test child_deletion] < 0 } { - return -1 + return } diff --git a/gdb/testsuite/gdb.mi/mi-var-cmd.exp b/gdb/testsuite/gdb.mi/mi-var-cmd.exp index d95c9847e96..bf207dad255 100644 --- a/gdb/testsuite/gdb.mi/mi-var-cmd.exp +++ b/gdb/testsuite/gdb.mi/mi-var-cmd.exp @@ -26,7 +26,7 @@ standard_testfile var-cmd.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-create-rtti.exp b/gdb/testsuite/gdb.mi/mi-var-create-rtti.exp index 918d0497806..08b6076dea0 100644 --- a/gdb/testsuite/gdb.mi/mi-var-create-rtti.exp +++ b/gdb/testsuite/gdb.mi/mi-var-create-rtti.exp @@ -20,13 +20,13 @@ standard_testfile .c set opts {debug} if {[build_executable $testfile.exp $testfile $srcfile $opts]} { - return -1 + return } mi_clean_restart $::testfile if {[mi_runto_main] < 0} { - return -1 + return } # Test creating a register-based variable. We pick diff --git a/gdb/testsuite/gdb.mi/mi-var-display.exp b/gdb/testsuite/gdb.mi/mi-var-display.exp index 2efbd2903a1..325291efe50 100644 --- a/gdb/testsuite/gdb.mi/mi-var-display.exp +++ b/gdb/testsuite/gdb.mi/mi-var-display.exp @@ -26,7 +26,7 @@ standard_testfile var-cmd.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-invalidate-shlib.exp b/gdb/testsuite/gdb.mi/mi-var-invalidate-shlib.exp index 45e64bcc2c5..80a97fbcbf2 100644 --- a/gdb/testsuite/gdb.mi/mi-var-invalidate-shlib.exp +++ b/gdb/testsuite/gdb.mi/mi-var-invalidate-shlib.exp @@ -27,7 +27,7 @@ set shlib_path [standard_output_file ${testfile}-lib.so] if { [gdb_compile_shlib $srcdir/$subdir/$srcfile2 $shlib_path {debug}] != "" } { untested "failed to compile" - return -1 + return } set shlib_path_target [gdb_download_shlib $shlib_path] @@ -39,7 +39,7 @@ lappend opts additional_flags=-DSHLIB_PATH=[quote_for_host $shlib_path_target] if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable $opts] != "" } { untested "failed to compile" - return -1 + return } proc do_test { separate_debuginfo } { diff --git a/gdb/testsuite/gdb.mi/mi-var-invalidate.exp b/gdb/testsuite/gdb.mi/mi-var-invalidate.exp index 693db997791..0edb5b6b188 100644 --- a/gdb/testsuite/gdb.mi/mi-var-invalidate.exp +++ b/gdb/testsuite/gdb.mi/mi-var-invalidate.exp @@ -27,13 +27,13 @@ standard_testfile var-cmd.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } # Just change the output binary. set binfile_bis [standard_output_file mi-var-invalidate_bis] if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile_bis}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } set testfile2 "basics" @@ -41,7 +41,7 @@ set srcfile2 ${testfile2}.c set binfile2 [standard_output_file ${testfile2}] if { [gdb_compile "${srcdir}/${subdir}/${srcfile2}" "${binfile2}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-list-children-invalid-grandchild.exp b/gdb/testsuite/gdb.mi/mi-var-list-children-invalid-grandchild.exp index 42495401702..366e15aa676 100644 --- a/gdb/testsuite/gdb.mi/mi-var-list-children-invalid-grandchild.exp +++ b/gdb/testsuite/gdb.mi/mi-var-list-children-invalid-grandchild.exp @@ -23,7 +23,7 @@ standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-var-rtti.exp b/gdb/testsuite/gdb.mi/mi-var-rtti.exp index b254de20c1e..9a98e2adfa9 100644 --- a/gdb/testsuite/gdb.mi/mi-var-rtti.exp +++ b/gdb/testsuite/gdb.mi/mi-var-rtti.exp @@ -22,7 +22,7 @@ standard_testfile .cc set opts {debug c++} if {[build_executable $testfile.exp $testfile $srcfile $opts]} { - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi-watch-nonstop.exp b/gdb/testsuite/gdb.mi/mi-watch-nonstop.exp index 856a1a37008..7f972ea6ea6 100644 --- a/gdb/testsuite/gdb.mi/mi-watch-nonstop.exp +++ b/gdb/testsuite/gdb.mi/mi-watch-nonstop.exp @@ -33,7 +33,7 @@ proc mi_nonstop_resume { command test } { standard_testfile watch-nonstop.c if {[gdb_compile "$srcdir/$subdir/$srcfile" $binfile executable {debug}] != "" } { - return -1 + return } save_vars { GDBFLAGS } { diff --git a/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp b/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp index 16a45d00265..fcea890d78a 100644 --- a/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp +++ b/gdb/testsuite/gdb.mi/mi2-amd64-entry-value.exp @@ -28,7 +28,7 @@ if {[info exists COMPILE]} { } if {[build_executable ${testfile}.exp ${binfile} ${srcfile} $opts]} { - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi2-cli-display.exp b/gdb/testsuite/gdb.mi/mi2-cli-display.exp index 263568c3966..f339f3fbebf 100644 --- a/gdb/testsuite/gdb.mi/mi2-cli-display.exp +++ b/gdb/testsuite/gdb.mi/mi2-cli-display.exp @@ -22,7 +22,7 @@ standard_testfile if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/mi2-var-child.exp b/gdb/testsuite/gdb.mi/mi2-var-child.exp index e5b4adada38..def9faf36d5 100644 --- a/gdb/testsuite/gdb.mi/mi2-var-child.exp +++ b/gdb/testsuite/gdb.mi/mi2-var-child.exp @@ -26,7 +26,7 @@ standard_testfile var-cmd.c if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { untested "failed to compile" - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/new-ui-bp-deleted.exp b/gdb/testsuite/gdb.mi/new-ui-bp-deleted.exp index ea81a9ab2f6..910dd263684 100644 --- a/gdb/testsuite/gdb.mi/new-ui-bp-deleted.exp +++ b/gdb/testsuite/gdb.mi/new-ui-bp-deleted.exp @@ -31,7 +31,7 @@ set MIFLAGS "-i=mi" standard_testfile if { [build_executable "failed to prepare" $testfile $srcfile] } { - return -1 + return } # Helper proc to create a breakpoint location regexp. NUM is the diff --git a/gdb/testsuite/gdb.mi/new-ui-mi-sync.exp b/gdb/testsuite/gdb.mi/new-ui-mi-sync.exp index 915f40571e1..fe679b09f97 100644 --- a/gdb/testsuite/gdb.mi/new-ui-mi-sync.exp +++ b/gdb/testsuite/gdb.mi/new-ui-mi-sync.exp @@ -29,7 +29,7 @@ standard_testfile if {[build_executable $testfile.exp $testfile ${srcfile} "debug"] == -1} { untested "failed to compile" - return -1 + return } # The test driver. SYNC_COMMAND specifies which command is used to diff --git a/gdb/testsuite/gdb.mi/print-simple-values.exp b/gdb/testsuite/gdb.mi/print-simple-values.exp index 4de81c4cbd4..875da82ff7c 100644 --- a/gdb/testsuite/gdb.mi/print-simple-values.exp +++ b/gdb/testsuite/gdb.mi/print-simple-values.exp @@ -31,7 +31,7 @@ lappend opts c++ lappend opts additional_flags=-std=c++11 if {[build_executable "failed to prepare" $testfile $srcfile $opts]} { - return -1 + return } if {[mi_clean_restart $::testfile]} { diff --git a/gdb/testsuite/gdb.mi/user-selected-context-sync.exp b/gdb/testsuite/gdb.mi/user-selected-context-sync.exp index d3abbcb4145..7555469b579 100644 --- a/gdb/testsuite/gdb.mi/user-selected-context-sync.exp +++ b/gdb/testsuite/gdb.mi/user-selected-context-sync.exp @@ -45,7 +45,7 @@ require allow_multi_inferior_tests set compile_options "debug pthreads" if {[build_executable $testfile.exp $testfile ${srcfile} ${compile_options}] == -1} { untested "failed to compile" - return -1 + return } set main_break_line [gdb_get_line_number "main break line"]