gdb/testsuite: remove use of then keyword from gdb.base/*.exp
authorAndrew Burgess <aburgess@redhat.com>
Mon, 14 Nov 2022 11:31:04 +0000 (11:31 +0000)
committerAndrew Burgess <aburgess@redhat.com>
Mon, 28 Nov 2022 21:04:09 +0000 (21:04 +0000)
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.base/
test script directory.

There should be no changes in what is tested after this commit.

216 files changed:
gdb/testsuite/gdb.base/a2-run.exp
gdb/testsuite/gdb.base/advance.exp
gdb/testsuite/gdb.base/alias.exp
gdb/testsuite/gdb.base/all-bin.exp
gdb/testsuite/gdb.base/annota-input-while-running.exp
gdb/testsuite/gdb.base/annota1.exp
gdb/testsuite/gdb.base/anon.exp
gdb/testsuite/gdb.base/arithmet.exp
gdb/testsuite/gdb.base/arrayidx.exp
gdb/testsuite/gdb.base/assign.exp
gdb/testsuite/gdb.base/async.exp
gdb/testsuite/gdb.base/attach-wait-input.exp
gdb/testsuite/gdb.base/attach.exp
gdb/testsuite/gdb.base/auto-connect-native-target.exp
gdb/testsuite/gdb.base/auxv.exp
gdb/testsuite/gdb.base/backtrace.exp
gdb/testsuite/gdb.base/bfp-test.exp
gdb/testsuite/gdb.base/bigcore.exp
gdb/testsuite/gdb.base/bitfields2.exp
gdb/testsuite/gdb.base/bp-permanent.exp
gdb/testsuite/gdb.base/break-idempotent.exp
gdb/testsuite/gdb.base/break-unload-file.exp
gdb/testsuite/gdb.base/break.exp
gdb/testsuite/gdb.base/bt-on-error-and-warning.exp
gdb/testsuite/gdb.base/bt-on-fatal-signal.exp
gdb/testsuite/gdb.base/bt-selected-frame.exp
gdb/testsuite/gdb.base/call-ar-st.exp
gdb/testsuite/gdb.base/call-rt-st.exp
gdb/testsuite/gdb.base/call-sc.exp
gdb/testsuite/gdb.base/call-strs.exp
gdb/testsuite/gdb.base/callfuncs.exp
gdb/testsuite/gdb.base/catch-fork-static.exp
gdb/testsuite/gdb.base/catch-signal-siginfo-cond.exp
gdb/testsuite/gdb.base/catch-syscall.exp
gdb/testsuite/gdb.base/checkpoint.exp
gdb/testsuite/gdb.base/clear_non_user_bp.exp
gdb/testsuite/gdb.base/compare-sections.exp
gdb/testsuite/gdb.base/completion.exp
gdb/testsuite/gdb.base/complex-parts.exp
gdb/testsuite/gdb.base/complex.exp
gdb/testsuite/gdb.base/cond-eval-mode.exp
gdb/testsuite/gdb.base/cond-expr.exp
gdb/testsuite/gdb.base/condbreak-call-false.exp
gdb/testsuite/gdb.base/consecutive-step-over.exp
gdb/testsuite/gdb.base/consecutive.exp
gdb/testsuite/gdb.base/constvars.exp
gdb/testsuite/gdb.base/coredump-filter.exp
gdb/testsuite/gdb.base/corefile.exp
gdb/testsuite/gdb.base/corefile2.exp
gdb/testsuite/gdb.base/ctf-constvars.exp
gdb/testsuite/gdb.base/ctf-ptype.exp
gdb/testsuite/gdb.base/decl-before-def.exp
gdb/testsuite/gdb.base/default.exp
gdb/testsuite/gdb.base/define.exp
gdb/testsuite/gdb.base/dfp-test.exp
gdb/testsuite/gdb.base/disabled-location.exp
gdb/testsuite/gdb.base/display.exp
gdb/testsuite/gdb.base/dprintf-detach.exp
gdb/testsuite/gdb.base/dump.exp
gdb/testsuite/gdb.base/duplicate-bp.exp
gdb/testsuite/gdb.base/eh_return.exp
gdb/testsuite/gdb.base/ena-dis-br.exp
gdb/testsuite/gdb.base/endianity.exp
gdb/testsuite/gdb.base/enum_cond.exp
gdb/testsuite/gdb.base/eval-skip.exp
gdb/testsuite/gdb.base/exe-lock.exp
gdb/testsuite/gdb.base/exec-invalid-sysroot.exp
gdb/testsuite/gdb.base/execl-update-breakpoints.exp
gdb/testsuite/gdb.base/exprs.exp
gdb/testsuite/gdb.base/fileio.exp
gdb/testsuite/gdb.base/finish.exp
gdb/testsuite/gdb.base/fixsection.exp
gdb/testsuite/gdb.base/float.exp
gdb/testsuite/gdb.base/float128.exp
gdb/testsuite/gdb.base/floatn.exp
gdb/testsuite/gdb.base/foll-exec-mode.exp
gdb/testsuite/gdb.base/foll-exec.exp
gdb/testsuite/gdb.base/foll-vfork.exp
gdb/testsuite/gdb.base/fork-running-state.exp
gdb/testsuite/gdb.base/frame-args.exp
gdb/testsuite/gdb.base/frame-info-consistent.exp
gdb/testsuite/gdb.base/frameapply.exp
gdb/testsuite/gdb.base/funcargs.exp
gdb/testsuite/gdb.base/gcore-buffer-overflow.exp
gdb/testsuite/gdb.base/gcore.exp
gdb/testsuite/gdb.base/gdb1090.exp
gdb/testsuite/gdb.base/gdb11530.exp
gdb/testsuite/gdb.base/gdb11531.exp
gdb/testsuite/gdb.base/gdb1250.exp
gdb/testsuite/gdb.base/gdb1555.exp
gdb/testsuite/gdb.base/gdb1821.exp
gdb/testsuite/gdb.base/gnu-ifunc.exp
gdb/testsuite/gdb.base/gnu_vector.exp
gdb/testsuite/gdb.base/gold-gdb-index.exp
gdb/testsuite/gdb.base/hbreak-in-shr-unsupported.exp
gdb/testsuite/gdb.base/hbreak2.exp
gdb/testsuite/gdb.base/hook-stop.exp
gdb/testsuite/gdb.base/huge.exp
gdb/testsuite/gdb.base/infcall-input.exp
gdb/testsuite/gdb.base/inferior-args.exp
gdb/testsuite/gdb.base/inferior-died.exp
gdb/testsuite/gdb.base/info-fun.exp
gdb/testsuite/gdb.base/info-locals-unused-static-var.exp
gdb/testsuite/gdb.base/info-os.exp
gdb/testsuite/gdb.base/info-proc.exp
gdb/testsuite/gdb.base/info-program.exp
gdb/testsuite/gdb.base/info_qt.exp
gdb/testsuite/gdb.base/inline-frame-cycle-unwind.exp
gdb/testsuite/gdb.base/interp.exp
gdb/testsuite/gdb.base/interrupt.exp
gdb/testsuite/gdb.base/jit-reader-exec.exp
gdb/testsuite/gdb.base/jump.exp
gdb/testsuite/gdb.base/label-without-address.exp
gdb/testsuite/gdb.base/langs.exp
gdb/testsuite/gdb.base/large-frame.exp
gdb/testsuite/gdb.base/list-missing-source.exp
gdb/testsuite/gdb.base/list.exp
gdb/testsuite/gdb.base/logical.exp
gdb/testsuite/gdb.base/long-inferior-output.exp
gdb/testsuite/gdb.base/long_long.exp
gdb/testsuite/gdb.base/longjmp.exp
gdb/testsuite/gdb.base/maint-info-sections.exp
gdb/testsuite/gdb.base/maint-print-frame-id.exp
gdb/testsuite/gdb.base/maint.exp
gdb/testsuite/gdb.base/max-value-size.exp
gdb/testsuite/gdb.base/memattr.exp
gdb/testsuite/gdb.base/mips_pro.exp
gdb/testsuite/gdb.base/miscexprs.exp
gdb/testsuite/gdb.base/multi-forks.exp
gdb/testsuite/gdb.base/nodebug.exp
gdb/testsuite/gdb.base/non-lazy-array-index.exp
gdb/testsuite/gdb.base/noreturn-finish.exp
gdb/testsuite/gdb.base/noreturn-return.exp
gdb/testsuite/gdb.base/opaque.exp
gdb/testsuite/gdb.base/overlays.exp
gdb/testsuite/gdb.base/paginate-after-ctrl-c-running.exp
gdb/testsuite/gdb.base/paginate-bg-execution.exp
gdb/testsuite/gdb.base/paginate-inferior-exit.exp
gdb/testsuite/gdb.base/pc-fp.exp
gdb/testsuite/gdb.base/permissions.exp
gdb/testsuite/gdb.base/pie-execl.exp
gdb/testsuite/gdb.base/pointers.exp
gdb/testsuite/gdb.base/pr11022.exp
gdb/testsuite/gdb.base/premature-dummy-frame-removal.exp
gdb/testsuite/gdb.base/printcmds.exp
gdb/testsuite/gdb.base/prologue.exp
gdb/testsuite/gdb.base/ptype.exp
gdb/testsuite/gdb.base/quit-live.exp
gdb/testsuite/gdb.base/radix.exp
gdb/testsuite/gdb.base/readnever.exp
gdb/testsuite/gdb.base/recurse.exp
gdb/testsuite/gdb.base/reggroups.exp
gdb/testsuite/gdb.base/relational.exp
gdb/testsuite/gdb.base/remote.exp
gdb/testsuite/gdb.base/return.exp
gdb/testsuite/gdb.base/return2.exp
gdb/testsuite/gdb.base/savedregs.exp
gdb/testsuite/gdb.base/scope.exp
gdb/testsuite/gdb.base/sect-cmd.exp
gdb/testsuite/gdb.base/sepdebug.exp
gdb/testsuite/gdb.base/set-noassign.exp
gdb/testsuite/gdb.base/setshow.exp
gdb/testsuite/gdb.base/setvar.exp
gdb/testsuite/gdb.base/shlib-call.exp
gdb/testsuite/gdb.base/shreloc.exp
gdb/testsuite/gdb.base/sigall.exp
gdb/testsuite/gdb.base/sigaltstack.exp
gdb/testsuite/gdb.base/sigbpt.exp
gdb/testsuite/gdb.base/siginfo-addr.exp
gdb/testsuite/gdb.base/siginfo-obj.exp
gdb/testsuite/gdb.base/siginfo-thread.exp
gdb/testsuite/gdb.base/siginfo.exp
gdb/testsuite/gdb.base/signals-state-child.exp
gdb/testsuite/gdb.base/signals.exp
gdb/testsuite/gdb.base/signest.exp
gdb/testsuite/gdb.base/signull.exp
gdb/testsuite/gdb.base/sigrepeat.exp
gdb/testsuite/gdb.base/sizeof.exp
gdb/testsuite/gdb.base/so-impl-ld.exp
gdb/testsuite/gdb.base/solib-disc.exp
gdb/testsuite/gdb.base/solib-display.exp
gdb/testsuite/gdb.base/solib-symbol.exp
gdb/testsuite/gdb.base/source-dir.exp
gdb/testsuite/gdb.base/source-execution.exp
gdb/testsuite/gdb.base/sss-bp-on-user-bp.exp
gdb/testsuite/gdb.base/stack-checking.exp
gdb/testsuite/gdb.base/stack-protector.exp
gdb/testsuite/gdb.base/step-break.exp
gdb/testsuite/gdb.base/step-line.exp
gdb/testsuite/gdb.base/step-over-syscall.exp
gdb/testsuite/gdb.base/step-test.exp
gdb/testsuite/gdb.base/store.exp
gdb/testsuite/gdb.base/structs.exp
gdb/testsuite/gdb.base/structs2.exp
gdb/testsuite/gdb.base/sym-file.exp
gdb/testsuite/gdb.base/symbol-alias.exp
gdb/testsuite/gdb.base/term.exp
gdb/testsuite/gdb.base/twice.exp
gdb/testsuite/gdb.base/type-opaque.exp
gdb/testsuite/gdb.base/until.exp
gdb/testsuite/gdb.base/unwind-on-each-insn.exp
gdb/testsuite/gdb.base/varargs.exp
gdb/testsuite/gdb.base/volatile.exp
gdb/testsuite/gdb.base/watch-cond-infcall.exp
gdb/testsuite/gdb.base/watch-cond.exp
gdb/testsuite/gdb.base/watch-non-mem.exp
gdb/testsuite/gdb.base/watch-read.exp
gdb/testsuite/gdb.base/watch-vfork.exp
gdb/testsuite/gdb.base/watch_thread_num.exp
gdb/testsuite/gdb.base/watchpoint-reuse-slot.exp
gdb/testsuite/gdb.base/watchpoint-stops-at-right-insn.exp
gdb/testsuite/gdb.base/watchpoint.exp
gdb/testsuite/gdb.base/wchar.exp
gdb/testsuite/gdb.base/whatis-exp.exp
gdb/testsuite/gdb.base/whatis-ptype-typedefs.exp
gdb/testsuite/gdb.base/whatis.exp

index 0b814335053ca4063979be933aab13295a3b3310..5c94becc5925896e9fc407a380eab940148b9977 100644 (file)
@@ -124,7 +124,7 @@ if {$res == 0} {
 
 # The remaining tests don't work for targets can't take arguments...
 
-if [target_info exists noargs] then {
+if {[target_info exists noargs]} {
     verbose "Skipping rest of a2-run.exp because of noargs."
     return
 }
@@ -166,7 +166,7 @@ gdb_test_stdio "" "720" "" "run \"$testfile\" again after setting args"
 
 # GOAL: Test that shell is being used with "run".  For remote debugging
 # targets, there is no guarantee that a "shell" (whatever that is) is used.
-if ![is_remote target] then {
+if {![is_remote target]} {
     gdb_test_stdio "run `echo 8`" \
        "40320" "" "run \"$testfile\" with shell"
 }
index cbc7515749d20c5b92d4945582577079b96a77fb..10f81174ad087f70037a3c7de8e7a739b2648637 100644 (file)
@@ -21,7 +21,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index d73a2183c98be4b7a388c42dbd85fdc6628bd7fa..98f660ae77115630f0360e87450ca346feb5e173 100644 (file)
@@ -32,7 +32,7 @@ proc test_abbrev_not_present { alias_name } {
            exp_continue
        }
        -re ".*$gdb_prompt $" {
-           if { !$alias_present } then {
+           if {!$alias_present} {
                pass $test_name
            } else {
                fail $test_name
index cf53cd7997e49887c9df3594564a7c60783c4811..2bb9cfe900e543142975dcd2028b3492d20cfbb0 100644 (file)
@@ -37,7 +37,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 2b4c5e17321f13e23139e4db54e56cba0517ddd9..2eff5be733f38e609f1870b7e862ba4e74c7aba1 100644 (file)
@@ -24,7 +24,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug] == -1} {
 
 # Because runto_main doesn't know how to handle the prompt with annotations,
 # run to main before we set the annotation level.
-if ![runto_main] then {
+if {![runto_main]} {
    return 1
 }
 
index 8060d3c8934dba7f445dc3af721e973dc893bb37..675f3afe2d47db488017aa80faff2358689fb4e2 100644 (file)
@@ -501,7 +501,7 @@ proc thread_test {} {
        gdb_start
        gdb_reinitialize_dir $srcdir/$subdir
        gdb_load ${binfile}
-       if { ![runto_main] } then {
+       if {![runto_main]} {
            return
        }
 
index 83db599bf721624addb5fc87054f88c7603fa14d..8d67b8e1b07b90a4c069f237ca61bccff5963628 100644 (file)
@@ -20,7 +20,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile {debug}] } {
     return -1
 }
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return
 }
 
index 4905c2e27061ba970ae131e47038d28529846248..4fa2ab2f8884f5aee6d1fb5679ac63e2a597f1ba 100644 (file)
@@ -37,7 +37,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile {debug nowarning
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 7df03346045c37547b421a260bee7f52556ab9db..677e4f0fcc381f37a2c835d4ab956c1c769a431b 100644 (file)
@@ -26,7 +26,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debu
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index a0dd46d7bb24d8d371d116df0297948b587a4d70..526e0638fcc4dd7c17332ecf1462728be0f6c7cd 100644 (file)
@@ -40,7 +40,7 @@ clean_restart ${binfile}
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 83e56e71580c8a65e34624b13ac24b01857a54ba..7630581edca7d9f2e1d6e085094146622104be1e 100644 (file)
@@ -27,7 +27,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 #
 # set it up at a breakpoint so we can play with it
 #
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 2f02c14f1d21a47062dff57e17c0c0b964efa07d..b4b19ac8592e5bca45f048837d336ed5218e425c 100644 (file)
@@ -47,7 +47,7 @@ proc start_program {binfile} {
 
     clean_restart $binfile
 
-    if ![runto setup_done] then {
+    if {![runto setup_done]} {
        return 0
     }
 
index cfe806bb9dd05acd614295cf49b524485e9c9a05..453fdbc78576ce98720e4e6d42bc898322eba692 100644 (file)
@@ -453,7 +453,7 @@ proc_with_prefix do_command_attach_tests {} {
     global gdb_prompt
     global binfile
 
-    if ![isnative] then {
+    if {![isnative]} {
        unsupported "command attach test"
        return 0
     }
index 94220464d7a592ace1baa087bee2b2254f7c909f..b60cf8fad01e46a3a28f273c200167cddf5eaac8 100644 (file)
@@ -68,7 +68,7 @@ if { $topmost != "exec" } {
 
 # Check which target this board connects to.  If testing with a native
 # target board, this should cause the native target to auto connect.
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 390f559a4a30b845ce2058911ca4b6848772234e..a1bb38d4b2cab8af5869c268e3c0f4ee37ea8e5e 100644 (file)
@@ -56,7 +56,7 @@ if {$core_works} {
     }
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 set print_core_line [gdb_get_line_number "ABORT;"]
index dec4ed6abb4af9cbb9fa9de0eb13c8f2e44b6900..6e584287ad92eb4ff5bd294dd121b66a0d1ea599 100644 (file)
@@ -21,7 +21,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 07282595b446c90db12b586ba1785a2eb64cf454..174152140f9901a6516095810d89cca34d41e65b 100644 (file)
@@ -25,7 +25,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debu
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 13b70f7dd5d3b93845666d6f50c729317928b57d..8acb716c9b554d9a5955b3ee2ec18aadab19bca4 100644 (file)
@@ -20,7 +20,7 @@
 # Are we on a target board?  As of 2004-02-12, GDB didn't have a
 # mechanism that would let it efficiently access a remote corefile.
 
-if ![isnative] then {
+if {![isnative]} {
     untested "remote system"
     return
 }
@@ -55,7 +55,7 @@ gdb_test_no_output "set width 0"
 # Get the core into the output directory.
 set_inferior_cwd_to_output_dir
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 set print_core_line [gdb_get_line_number "Dump core"]
index f71ca423f5df3be30e6051255ff238af73b7b611..5b89455572d6286d9f4769cdaefbe4c3874b7638 100644 (file)
@@ -160,7 +160,7 @@ proc bitfield_signedness {} {
     }
 
     set test "most negative signed bitfield values"
-    if $has_signed_bitfields then {
+    if {$has_signed_bitfields} {
         gdb_test "print flags" "u1 = 0, u2 = 0, u3 = 0, s1 = -16384, s2 = -4294967296, s3 = -32768.*" $test
     } else {
        unsupported $test
@@ -169,7 +169,7 @@ proc bitfield_signedness {} {
     continue_test break4 "#2"
 
     set test "signed bitfields containing -1"
-    if $has_signed_bitfields then {
+    if {$has_signed_bitfields} {
        gdb_test "print flags" "u1 = 0, u2 = 0, u3 = 0, s1 = -1, s2 = -1, s3 = -1.*" $test
     } else {
        unsupported $test
@@ -212,7 +212,7 @@ proc bitfield_set {} {
     gdb_test "print flags" "u1 = 0, u2 = 4294967296, u3 = 0, s1 = 0, s2 = 2147483648, s3 = 0.*" "long long bitfield values after set"
 
     set test "set long long signed bitfield negative"
-    if $has_signed_bitfields then {
+    if {$has_signed_bitfields} {
        gdb_test_multiple "print flags.s2 = -1" $test {
            -re "warning: Value does not fit.*$gdb_prompt $" {
                fail "$test"
@@ -226,7 +226,7 @@ proc bitfield_set {} {
     }
 
     set test  "long long bitfield values after set negative"
-    if $has_signed_bitfields then {
+    if {$has_signed_bitfields} {
        gdb_test "print flags" "u1 = 0, u2 = 4294967296, u3 = 0, s1 = 0, s2 = -1, s3 = 0.*" $test
     } else {
        unsupported $test
index 1ad9a698db8e6ae9dc723f4cd145ec3fdd84602f..5997b5685d0016bfbd3fe31d551d7ab936e60537 100644 (file)
@@ -44,7 +44,7 @@ proc test {always_inserted sw_watchpoint} {
 
     clean_restart $binfile
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return -1
     }
 
index 837ac000b5770cc2e31c59b1fabf2df486bd8030..b3500101a1aa464c7684b6e53dac7b17f74fec0c 100644 (file)
@@ -115,7 +115,7 @@ proc test_break { always_inserted break_command } {
     with_test_prefix "$cmd" {
        delete_breakpoints
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return
        }
 
index ed14772e71ca67446a56c23ac973bb5bd86aedb8..83edaedb45c10612b889371d883dfb8851eff13b 100644 (file)
@@ -59,7 +59,7 @@ proc test_break { initial_load always_inserted break_command } {
        gdb_reload
        set GDBFLAGS $saved_gdbflags
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return
        }
 
index 258c468ed001d98946f7000c5f6f61359cbcf619..7ef588edf08892e1b75f4298d6659d7613ba1d43 100644 (file)
@@ -161,7 +161,7 @@ proc_with_prefix test_break {} {
            exp_continue
        }
        -re ".*$::gdb_prompt $" {
-           if { !$see1 && $see2 && !$see3 && $see4 && !$see5 && $see6 } then {
+           if {!$see1 && $see2 && !$see3 && $see4 && !$see5 && $see6} {
                pass "info break 2 4 6"
            } else {
                fail "info break 2 4 6"
@@ -202,7 +202,7 @@ proc_with_prefix test_break {} {
            exp_continue
        }
        -re ".*$::gdb_prompt $" {
-           if { !$see1 && !$see2 && $see3 && $see4 && $see5 && !$see6 } then {
+           if {!$see1 && !$see2 && $see3 && $see4 && $see5 && !$see6} {
                pass "info break 3-5"
            } else {
                fail "info break 3-5"
@@ -260,7 +260,7 @@ proc_with_prefix test_break {} {
                exp_continue
            }
            -re ".*$::gdb_prompt $" {
-               if { $see1 && $see2 && $see3 && $see4 && $see5 && $see6 } then {
+               if {$see1 && $see2 && $see3 && $see4 && $see5 && $see6} {
                    pass "check disable with history values"
                } else {
                    fail "check disable with history values"
@@ -308,7 +308,7 @@ proc_with_prefix test_break {} {
                exp_continue
            }
            -re ".*$::gdb_prompt $" {
-               if { $see1 && $see2 && $see3 && $see4 && $see5 && $see6 } then {
+               if {$see1 && $see2 && $see3 && $see4 && $see5 && $see6} {
                    pass "check disable with convenience values"
                } else {
                    fail "check disable with convenience values"
@@ -458,7 +458,7 @@ test_tbreak
 proc_with_prefix test_no_break_on_catchpoint {} {
     clean_restart break
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -482,7 +482,7 @@ test_no_break_on_catchpoint
 proc_with_prefix test_break_nonexistent_line {} {
     clean_restart break
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -497,7 +497,7 @@ test_break_nonexistent_line
 proc_with_prefix test_break_default {} {
     clean_restart break
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -541,7 +541,7 @@ test_break_default
 proc_with_prefix test_break_silent_and_more {} {
     clean_restart break
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -686,7 +686,7 @@ test_break_user_call
 proc_with_prefix test_finish_arguments {} {
     clean_restart break
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
index 0dfd13c24281615533dde257dd1f83088bb86080..a9026e1d69ce8e5a5f6ea2241d99f40a3a2dfe83 100644 (file)
@@ -25,7 +25,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile]} {
 
 # Check we can run to main.  If this works this time then we just
 # assume that it will work later on (when we repeatedly restart GDB).
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 1af88d50c4cdb7ca2e5efaa05a7920a8794a2136..bd5e46bcb2f959e55ce4c34bf2c577adf00d9c46 100644 (file)
@@ -34,7 +34,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile]} {
 
 # Check we can run to main.  If this works this time then we just
 # assume that it will work later on (when we repeatedly restart GDB).
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index e1018e4992af1965e9574008ed9a20faefcc5815..912b87512ad714ed08e8f9cf004d282a9aa0ee5b 100644 (file)
@@ -28,7 +28,7 @@ proc check_selected_frame_after_bt { bt_cmd stack_pattern } {
 
     with_test_prefix $bt_cmd {
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return 0
        }
 
index 2da438ccd962a987fa299418c44e4f1d10d75f27..a5599d741a6d609718468227702c770982e7d188 100644 (file)
@@ -44,7 +44,7 @@ gdb_test_no_output "set print address off"
 gdb_test_no_output "set width 0"
 
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
@@ -432,7 +432,7 @@ if {!$skip_float_test && \
     # On 32-bit SPARC, some of the args are passed by ref, others by
     # value, and GDB gets confused and says "Invalid cast" because it
     # thinks it has to cast the structure into a pointer to structure.
-    if { [test_debug_format "stabs"] } then {
+    if {[test_debug_format "stabs"]} {
        setup_kfail "gdb/1539" "sparc-*-*"
     }
 
index 00af32c13a18cd46d18f16a4ac3176b46e911a4b..63792ee73e5acf24fdfe68443c8d6ac9082ba27f 100644 (file)
@@ -45,7 +45,7 @@ gdb_test_no_output "set print address off"
 gdb_test_no_output "set width 0"
 
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index ddf2ceedf96e98d26639cce74ffef6550f685553..46da6506b251a79f87653ec7e7e35855ec5fbe12 100644 (file)
@@ -61,7 +61,7 @@ proc start_scalars_test { type } {
     }
 
     # Advance to main
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return
     }
 
index 58cb3cda38dcc1ccac273f162db3cc149ba4c7cf..f9335815da3ef001d05e68f081da7ef3eb67a485 100644 (file)
@@ -42,7 +42,7 @@ gdb_test_no_output "set print address off"
 gdb_test_no_output "set print symbol off"
 gdb_test_no_output "set width 0"
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 44289eebc8b5d49cd11136e2da1ce8ec8cda9adb..9b98f681fc15fe1c7ac6bb2aee907547360b27c2 100644 (file)
@@ -368,7 +368,7 @@ proc perform_all_tests {prototypes} {
     # Check if all registers still have the same value.
     set new_reg_content [fetch_all_registers \
                             "register contents after gdb function calls"]
-    if {$old_reg_content == $new_reg_content} then {
+    if {$old_reg_content == $new_reg_content} {
        pass "gdb function calls preserve register contents"
     } else {
        set old_reg_content $new_reg_content
@@ -394,7 +394,7 @@ proc perform_all_tests {prototypes} {
             "bt after continuing from call dummy breakpoint"] then {
                 set new_reg_content [fetch_all_registers \
                                          "register contents after stop in call dummy"]
-                if {$old_reg_content == $new_reg_content} then {
+                if {$old_reg_content == $new_reg_content} {
                     pass "continue after stop in call dummy preserves register contents"
                 } else {
                     fail "continue after stop in call dummy preserves register contents"
@@ -420,7 +420,7 @@ proc perform_all_tests {prototypes} {
             "bt after finishing from call dummy breakpoint"] then {
                 set new_reg_content [fetch_all_registers \
                                          "register contents after finish in call dummy"]
-                if {$old_reg_content == $new_reg_content} then {
+                if {$old_reg_content == $new_reg_content} {
                     pass "finish after stop in call dummy preserves register contents"
                 } else {
                     fail "finish after stop in call dummy preserves register contents"
@@ -445,7 +445,7 @@ proc perform_all_tests {prototypes} {
             "y"] then {
                 set new_reg_content [fetch_all_registers \
                                          "register contents after return in call dummy"]
-                if {$old_reg_content == $new_reg_content} then {
+                if {$old_reg_content == $new_reg_content} {
                     pass "return after stop in call dummy preserves register contents"
                 } else {
                     fail "return after stop in call dummy preserves register contents"
@@ -508,7 +508,7 @@ proc perform_all_tests {prototypes} {
 
     set new_reg_content [fetch_all_registers \
                             "register contents after nested call dummies"]
-    if {$old_reg_content == $new_reg_content} then {
+    if {$old_reg_content == $new_reg_content} {
        pass "nested call dummies preserve register contents"
     } else {
        fail "nested call dummies preserve register contents"
index 51031a84fbd44f5ea84076348d583f6254995eaa..07f05951e2dd1e289497c24ecfe8fad8591f1982 100644 (file)
@@ -23,7 +23,7 @@
 
 # Until "catch fork" is implemented on other targets...
 #
-if { ![istarget "*-*-linux*"] && ![istarget "*-*-openbsd*"] } then {
+if {![istarget "*-*-linux*"] && ![istarget "*-*-openbsd*"]} {
     return
 }
 
index 85a55b29ea5384fb8b4645fc07d121f2d2cda48a..c18c5fb0cb8ddb3d575a2f2e87f5c4c016fecabb 100644 (file)
@@ -41,7 +41,7 @@ if { [prepare_for_testing "failed to prepare" "${testfile}" $srcfile {debug}] }
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index d8b6898b3e0517fe49b195066f6ab5b2f793a8e7..2364b47bf7e493f7cae7f1f28fa2c9b1dae445b1 100644 (file)
@@ -26,7 +26,7 @@ if  { [prepare_for_testing "failed to prepare" $testfile ${testfile}.c] } {
 }
 
 # Check target supports catch syscall or not.
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -464,40 +464,40 @@ proc do_syscall_tests {} {
 
     # Testing the 'catch syscall' command without arguments.
     # This test should catch any syscalls.
-    if [runto_main] then { test_catch_syscall_without_args }
+    if {[runto_main]} { test_catch_syscall_without_args }
 
     # Testing the 'catch syscall' command with arguments.
     # This test should only catch the specified syscall.
-    if [runto_main] then { test_catch_syscall_with_args }
+    if {[runto_main]} { test_catch_syscall_with_args }
 
     # Testing the 'catch syscall' command with many arguments.
     # This test should catch $all_syscalls.
-    if [runto_main] then { test_catch_syscall_with_many_args }
+    if {[runto_main]} { test_catch_syscall_with_many_args }
 
     # Testing the 'catch syscall' command with WRONG arguments.
     # This test should not trigger any catchpoints.
-    if [runto_main] then { test_catch_syscall_with_wrong_args }
+    if {[runto_main]} { test_catch_syscall_with_wrong_args }
 
     # Testing the 'catch syscall' command during a restart of
     # the inferior.
-    if [runto_main] then { test_catch_syscall_restarting_inferior }
+    if {[runto_main]} { test_catch_syscall_restarting_inferior }
 
     # Testing the 'catch syscall' command toggling off past a
     # syscall return, then resuming entry/return as normal.
-    if [runto_main] then { test_catch_syscall_skipping_return }
+    if {[runto_main]} { test_catch_syscall_skipping_return }
 
     # Testing the 'catch syscall' command starting mid-vfork.
-    if [runto_main] then { test_catch_syscall_mid_vfork }
+    if {[runto_main]} { test_catch_syscall_mid_vfork }
 
     # Testing that 'catch syscall' entry/return tracks across execve.
-    if [runto_main] then { test_catch_syscall_execve }
+    if {[runto_main]} { test_catch_syscall_execve }
 
     # Testing if the 'catch syscall' command works when switching to
     # different architectures on-the-fly (PR gdb/10737).
-    if [runto_main] then { test_catch_syscall_multi_arch }
+    if {[runto_main]} { test_catch_syscall_multi_arch }
 
     # Testing the 'catch' syscall command for a group of syscalls.
-    if [runto_main] then { test_catch_syscall_group }
+    if {[runto_main]} { test_catch_syscall_group }
 }
 
 proc test_catch_syscall_without_args_noxml {} {
@@ -662,16 +662,16 @@ proc do_syscall_tests_without_xml {} {
 
     # Let's test if we can catch syscalls without XML support.
     # We should succeed, but GDB is not supposed to print syscall names.
-    if [runto_main] then { test_catch_syscall_without_args_noxml }
+    if {[runto_main]} { test_catch_syscall_without_args_noxml }
 
     # The only valid argument "catch syscall" should accept is the
     # syscall number, and not the name (since it can't translate a
     # name to a number).
-    if [runto_main] then { test_catch_syscall_with_args_noxml }
+    if {[runto_main]} { test_catch_syscall_with_args_noxml }
 
     # Now, we'll try to provide a syscall name (valid or not) to the command,
     # and expect it to fail.
-    if [runto_main] then { test_catch_syscall_with_wrong_args_noxml }
+    if {[runto_main]} { test_catch_syscall_with_wrong_args_noxml }
 }
 
 # This procedure fills the vector "all_syscalls_numbers" with the proper
index e81e4f44e485a691c935580446f2fbeaa9e6a393..0fef9c551bd4beb71c0f69abcc144e8982a4c455 100644 (file)
@@ -16,7 +16,7 @@
 # Until "set follow-fork-mode" and "catch fork" are implemented on
 # other targets...
 #
-if {![istarget "*-*-linux*"]} then {
+if {![istarget "*-*-linux*"]} {
     return
 }
 
index da694351c877aabfe779fd98de47bf6a3a57b96f..2995f0d567f2d304ec4798441e214a3e14635c9d 100644 (file)
@@ -49,7 +49,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/main.c" "${binfile}" executable {debug}]
 # Start with a fresh gdb.
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index d9d10d42680f55287c75b72e5a219027d1865dc5..9885b0c0602ed4a87713aff4b1f2cf6f9c027f45 100644 (file)
@@ -79,7 +79,7 @@ gdb_test_multiple $command $test {
 }
 
 # Now get past startup code.
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index e3803c9d2d584de030b9e4263b0b2fa3a539b432..2a46d4fd21b48062f5f0b9eaa40a088689adabcc 100644 (file)
@@ -57,7 +57,7 @@ if {[prepare_for_testing "failed to prepare" $testfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
         perror "tests suppressed"
 }
 
index 7fa94c72cd4ff357cd07f236a54ed8fc1f891e1f..0075b8333e3fa9a9bb1676b177e64e061e936f5a 100644 (file)
@@ -19,7 +19,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return 0
 }
 
index e2c60029593997aa431e6c888bbe91a7cc74fa91..5d97c87e42a55702da5e3b27e2c4304b0580d815 100644 (file)
@@ -20,16 +20,16 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if [runto f2] then {
+if {[runto f2]} {
     get_debug_format
-    if { [test_compiler_info gcc-2-*] && [test_debug_format "DWARF \[0-9\]"] } then {
+    if {[test_compiler_info gcc-2-*] && [test_debug_format "DWARF \[0-9\]"]} {
        setup_xfail "*-*-*"
     }
     gdb_test "p *y" "\\\$\[0-9\]* = \{c = 42 '\\*', f = 1 \\+ 0i\}" \
            "print complex packed value in C"
 }
 
-if [runto f4] then {
+if {[runto f4]} {
     gdb_test "p *y" "\\\$\[0-9\]* = \{c = 42 '\\*', f = 1 \\+ 0i\}" \
            "print complex value in C"
 }
index f6e8b3e71b2d12a659c60a5a6719f6f5400764bb..61c28b5925d62b71302fa3fc9b64899443697024 100644 (file)
@@ -29,7 +29,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index f0c4c32859414b05364725e8e57c071e23ee641d..36754f9215862e925b841171ca2c4eb5edf4b282 100644 (file)
@@ -42,7 +42,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 clean_restart ${binfile}
 
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 2d0325df18d0e433174efa6477883a91ca3494de..4b1ed1b57bc751d2618dc25b0b137bae082cba71 100644 (file)
@@ -22,7 +22,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 5cb124145559f0b8312878358dfa36611d85f147..a8cecb50d6213df591bd52bfbce1ea0bf02a745b 100644 (file)
@@ -24,7 +24,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 4b38de0c3bc77cc21b867a2a16244bcc07ea1669..b517b42e62320e5ddac656b4eb8451fde9ef71e7 100644 (file)
@@ -26,7 +26,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile {debug nowarning
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
@@ -55,7 +55,7 @@ gdb_test "break \*$bp_addr" "Breakpoint $decimal at $bp_addr: file .*" \
 gdb_test_multiple "step" "stopped at bp, 2nd instr" {
     -re -wrap "Breakpoint $decimal, ($hex) in foo.*" {
        set stop_addr $expect_out(1,string)
-       if [eval expr "$bp_addr == $stop_addr"] then {
+       if {[eval expr "$bp_addr == $stop_addr"]} {
            pass "stopped at bp, 2nd instr"
        } else {
            fail "stopped at bp, 2nd instr (wrong address)"
@@ -64,9 +64,9 @@ gdb_test_multiple "step" "stopped at bp, 2nd instr" {
     -re -wrap "Breakpoint $decimal, foo.*" {
        set stop_addr [get_valueof "/x" "\$pc" "" "value of pc"]
        set stop_addr_is_stmt [hex_in_list $stop_addr $is_stmt]
-       if { ! $stop_addr_is_stmt } {
+       if {!$stop_addr_is_stmt} {
            fail "stopped at bp, 2nd instr (missing hex prefix)"
-       } elseif [eval expr "$bp_addr == $stop_addr"] then {
+       } elseif {[eval expr "$bp_addr == $stop_addr"]} {
            pass "stopped at bp, 2nd instr"
        } else {
            fail "stopped at bp, 2nd instr (wrong address)"
index 671baf62f016f39c430bde5c28274c071ce9ee4c..8d1af0b4fdc7562a21c36bb56557a2a128540a12 100644 (file)
@@ -40,7 +40,7 @@ clean_restart ${binfile}
 #
 # set it up at a breakpoint so we can play with the variable values
 #
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
@@ -51,7 +51,7 @@ get_debug_format
 # TODO: check out the hp side of this.
 
 proc local_compiler_xfail_check { } {
-    if { [test_compiler_info gcc-2-*] } then {
+    if {[test_compiler_info gcc-2-*]} {
        if { ![test_debug_format "HP"] \
                && ![test_debug_format "DWARF \[0-9\]"] } then {
            setup_xfail "*-*-*" 
index 14c3db44f51a0e708e54d181c6115f991f7aaedf..499bfae0fffdd0d0dcaff10b4c8777eafd409e99 100644 (file)
@@ -16,7 +16,7 @@
 standard_testfile
 
 # This test is Linux-only.
-if ![istarget *-*-linux*] then {
+if {![istarget *-*-linux*]} {
     untested "coredump-filter.exp"
     return -1
 }
index 0a44c2da49bf0de13b01fafd73c47273a5045f4e..f15e1e9e81dd36262323abbaefc687ea04fb5544 100644 (file)
@@ -17,7 +17,7 @@
 
 
 # are we on a target board
-if ![isnative] then {
+if {![isnative]} {
     return
 }
 
@@ -49,7 +49,7 @@ if {$corefile == ""} {
 # it a pass, but note that the program name is bad.
 
 gdb_exit
-if $verbose>1 then {
+if {$verbose>1} {
     send_user "Spawning $GDB $INTERNAL_GDBFLAGS $GDBFLAGS -core=$corefile\n"
 }
 
@@ -82,7 +82,7 @@ expect {
 
 close
 
-if $verbose>1 then {
+if {$verbose>1} {
     send_user "Spawning $GDB $INTERNAL_GDBFLAGS $GDBFLAGS $binfile -core=$corefile\n"
 }
 
index 5ae681b06fbdb4aaafa7207b9eadc7eb5517cbee..b08bd714ca937910219e5cf320f69cf7b09f2662 100644 (file)
@@ -18,7 +18,7 @@
 # coremaker2.c for details.
 
 # are we on a target board
-if ![isnative] then {
+if {![isnative]} {
     return
 }
 
@@ -174,7 +174,7 @@ with_test_prefix "renamed binfile" {
 
 clean_restart $binfile
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index a5e7fe0b015104dbf7807c9e375ddea2f162f3bb..4a8307e05bff515e2c803f618b4319a6172f344b 100644 (file)
@@ -46,7 +46,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} \
 #
 # set it up at a breakpoint so we can play with the variable values
 #
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 28deabb6ac5877f300b6ea5bf61209621cf8ae06..2e3b972ebe9f6250a623fc51aaed0d0b2efa0c0b 100644 (file)
@@ -64,16 +64,16 @@ gdb_test "ptype struct t_struct" "type = struct t_struct \{.*\[\r\n\]    (unsign
 
 # Test the equivalence between '.' and '->' for struct member references.
 
-if [gdb_test "ptype v_struct1.v_float_member"  "type = float"]<0 then {
+if {[gdb_test "ptype v_struct1.v_float_member" "type = float"] < 0} {
     return -1
 }
-if [gdb_test "ptype v_struct1->v_float_member" "type = float"]<0 then {
+if {[gdb_test "ptype v_struct1->v_float_member"        "type = float"] < 0} {
     return -1
 }
-if [gdb_test "ptype v_t_struct_p.v_float_member"       "type = float"]<0 then {
+if {[gdb_test "ptype v_t_struct_p.v_float_member"      "type = float"] < 0} {
     return -1
 }
-if [gdb_test "ptype v_t_struct_p->v_float_member"      "type = float"]<0 then {
+if {[gdb_test "ptype v_t_struct_p->v_float_member"     "type = float"] < 0} {
     return -1
 }
 
@@ -278,7 +278,7 @@ ptype_maybe_prototyped "fffptr" "int (*(*(*)(char))(short int))(long int)" \
 # requires a running process.  These call malloc, and can take a long
 # time to execute over a slow serial link, so increase the timeout.
 
-if [runto_main] then {
+if {[runto_main]} {
 
   if [target_info exists gdb,cannot_call_functions] {
     unsupported "this target can not call functions"
index c4120a2b81302a0cc020931fb3c38ac4431d4200..206d5e31b1113bb31342d54763cb97607a36c28d 100644 (file)
@@ -25,7 +25,7 @@ gdb_test "maint expand-symtabs"
 
 gdb_test "p a" { = \{1, 2\}} "no running process: p a"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index f3e4b2867ef2596a43f87dd70ec46a30238c8995..20cbab2e586f1138cf854b5400007c7eb7a62e08 100644 (file)
@@ -184,7 +184,7 @@ gdb_expect {
     }
     -re "exec-file.*A program is being debugged already.  Kill it. .y or n.*$" {
        send_gdb "n\n"
-       if $verbose>1 then {
+       if {$verbose > 1} {
            send_user "\tDidn't kill program being debugged\n"
        }
        gdb_expect -re "$gdb_prompt $" { }
@@ -208,7 +208,7 @@ gdb_expect {
            { pass "file" }
     -re ".*A program is being debugged already.  Kill it. .y or n.*$" {
        send_gdb "n\n"
-       if $verbose>1 then {
+       if {$verbose > 1} {
            send_user "\t\tDidn't kill program being debugged\n"
        }
        gdb_expect -re "$gdb_prompt $" { }
@@ -748,7 +748,7 @@ gdb_expect {
     -re "No core file specified..*$gdb_prompt $" { pass "target core" }
     -re ".*A program is being debugged already.  Kill it. .y or n.*$" {
        send_gdb "n\n"
-       if $verbose>1 then {
+       if {$verbose > 1} {
            send_user "\t\tDidn't kill program being debugged\n"
        }
        gdb_expect -re "$gdb_prompt $" { }
@@ -766,7 +766,7 @@ gdb_expect {
                        { pass "target exec" }
        -re ".*A program is being debugged already.  Kill it. .y or n.*$" {
            send_gdb "n\n"
-           if $verbose>1 then {
+           if {$verbose > 1} {
                send_user "\t\tDidn't kill program being debugged\n"
            }
            gdb_expect -re "$gdb_prompt $" { }
@@ -783,7 +783,7 @@ gdb_expect {
        { pass "target remote" }
     -re ".*A program is being debugged already.  Kill it. .y or n.*$" {
        send_gdb "n\n"
-       if $verbose>1 then {
+       if {$verbose > 1} {
            send_user "\t\tDidn't kill program being debugged\n"
        }
        gdb_expect -re "$gdb_prompt $" { }
index 52a4f0d5e5c9b9abb4a145268e2ee0ba94b624a1..e5616341e6dab76c88e2f45378090190768ebe71 100644 (file)
@@ -30,7 +30,7 @@ if {[prepare_for_testing "failed to prepare" ${testfile} \
 set bp_location1 [gdb_get_line_number "set breakpoint 1 here"]
 set bp_location11 [gdb_get_line_number "set breakpoint 11 here"]
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index 307952122918ceef5148112aa8c9c0008ebdbe6a..0d7314f247b8af261486752438b338eb540254e8 100644 (file)
@@ -30,7 +30,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {quie
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index d90bc5b91cf45747bc619f558e527e5ee176dcc4..1fc2e155ec298180f625d2c9e03a16ce62cdf3e3 100644 (file)
@@ -19,7 +19,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile}] } {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index b26c08433b8cebd63c371324a704b9aa42a4331e..d8ca284da3e537e6971b699de611d492986cf56e 100644 (file)
@@ -60,7 +60,7 @@ if !$use_gdb_stub {
 
 # Ok, on to real life
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index b6da01a2845aa7b864e5c00843dfa14a09e9c8be..6c71c61cba846d6db63b340d6609baf0f696acbb 100644 (file)
@@ -21,7 +21,7 @@
 load_lib gdbserver-support.exp
 
 # The test relies on "detach/attach".
-if { [use_gdb_stub] } then {
+if {[use_gdb_stub]} {
     return 0
 }
 
index c7dd1b139fc08347186a1867f84767df335be81e..1efc3c881b1622ac6a018e1a00134ab2ff6cdcca 100644 (file)
@@ -26,7 +26,7 @@ set endian "auto"
 
 set formats {binary ihex srec tekhex  verilog}
 
-if [istarget "alpha*-*-*"] then {
+if {[istarget "alpha*-*-*"]} {
     # SREC etc cannot handle 64-bit addresses.  Force the test
     # program into the low 31 bits of the address space.
     lappend options "ldflags=-Wl,-taso"
@@ -68,7 +68,7 @@ gdb_load ${binfile}
 # IHEX and TEKHEX.  We skip those tests then.
 set max_32bit_address "0xffffffff"
 set data_address [get_hexadecimal_valueof "&intarray" 0x100000000]
-if {${data_address} > ${max_32bit_address}} then {
+if {${data_address} > ${max_32bit_address}} {
     set is64bitonly "yes"
 }
 
@@ -105,7 +105,7 @@ remote_exec host "rm -f $filenames"
 
 # Run target program until data structs are initialized.
 
-if { ! [ runto checkpoint1 ] } then {
+if {![runto checkpoint1]} {
     untested "couldn't run to checkpoint"
     return -1
 }
@@ -283,7 +283,7 @@ proc test_restore_saved_value { restore_args msg oldval newval } {
     }
 }
 
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
 
 
   test_restore_saved_value "[set intarr1.srec]" "array as value, srec" \
@@ -365,7 +365,7 @@ set struct2_offset \
 print_zero_all
 
 
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
   test_restore_saved_value "[set intarr1.srec] $array2_offset" \
        "array copy, srec" \
        $array_val "intarray2"
@@ -421,7 +421,7 @@ set element3_offset \
 set element4_offset \
        [capture_value "/x (char *) &intarray\[4\] - (char *) &intarray\[0\]"]
 
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
   print_zero_all
 
   test_restore_saved_value "[set intarr1.srec] 0 $element3_start $element4_start" \
@@ -456,7 +456,7 @@ test_restore_saved_value \
 gdb_test "print intarray\[2\] == 0" " = 1" "element 2 not changed - 4"
 gdb_test "print intarray\[4\] == 0" " = 1" "element 4 not changed - 4"
 
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
   print_zero_all
 
   # restore with expressions 
@@ -520,7 +520,7 @@ proc test_reload_saved_value { filename msg oldval newval } {
 }
 
 # srec format can not be loaded for 64-bit-only platforms
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
   test_reload_saved_value "[set intarr1.srec]" "reload array as value, srec" \
        $array_val "\*$array_ptr_type"
   test_reload_saved_value "[set intstr1.srec]" "reload struct as value, srec" \
@@ -532,7 +532,7 @@ if ![string compare $is64bitonly "no"] then {
 }
 
 # ihex format can not be loaded for 64-bit-only platforms
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
 
   test_reload_saved_value "[set intarr1.ihex]" \
       "reload array as value, intel hex" \
@@ -549,7 +549,7 @@ if ![string compare $is64bitonly "no"] then {
 }
 
 # tekhex format can not be loaded for 64-bit-only platforms
-if ![string compare $is64bitonly "no"] then {
+if {![string compare $is64bitonly "no"]} {
   test_reload_saved_value "[set intarr1.tekhex]" \
       "reload array as value, tekhex" \
        $array_val "\*$array_ptr_type"
index 9a512928bae0cf2e2784fc6d7902844dd0b224ee..39ba3a5cbf71f4af8bf67ddd5416075baca3177b 100644 (file)
@@ -26,7 +26,7 @@ proc test_setup { count } {
 
     clean_restart ${binfile}
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
index a6dda78e01376f1aa6d11b01eda2b7f63f38b9cc..e4781e302a089224e6af0b878f64d2c5899dbd50 100644 (file)
@@ -19,7 +19,7 @@
 standard_testfile
 
 # Set compiler flags.
-if {[istarget "powerpc*"]} then {
+if {[istarget "powerpc*"]} {
     # PowerPC generates a Traceback Table, as defined in the PPC64 ABI,
     # following each function by default.  The Traceback Table information is
     # typically interpreted by the disassembler as data represented with
index 93ac124dbc43d6a1741f0b58f2484bc5b97c705f..8900f5347239c56f496dbb65e78c28faf42811af 100644 (file)
@@ -34,7 +34,7 @@ set bp_location14 [gdb_get_line_number "set breakpoint 14 here" $srcfile2]
 set bp_location15 [gdb_get_line_number "set breakpoint 15 here" $srcfile2]
 set bp_location17 [gdb_get_line_number "set breakpoint 17 here" $srcfile2]
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -123,7 +123,7 @@ gdb_continue_to_end "no stop at auto-disabled break marker2"
 
 # Verify that we can set a breakpoint to be self-deleting after the
 # first time it triggers.
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -154,7 +154,7 @@ gdb_test "info break $bp" \
     "\[0-9\]*\[ \t\]+breakpoint\[ \t\]+keep\[ \t\]+n.*" \
     "info break marker4"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -193,7 +193,7 @@ gdb_test "continue" \
 # should cause the next N triggers of the bp to be ignored.  (This is
 # a flavor of enablement/disablement, after all.)
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -254,7 +254,7 @@ gdb_test_multiple "continue" "$test" {
 
 # Verify that we can specify both an ignore count and an auto-delete.
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -286,7 +286,7 @@ gdb_test "continue" \
 # Verify that a disabled breakpoint's ignore count isn't updated when
 # the bp is encountered.
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -315,7 +315,7 @@ gdb_test "info break $bp" \
 # (Also verify that GDB gracefully handles the case where the inferior
 # isn't stopped at a breakpoint.)
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -343,7 +343,7 @@ gdb_test_multiple "continue 2" "$test" {
 # Verify that GDB correctly handles the "enable/disable" command
 # with arguments, that include multiple locations.
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index a6a87e66a44125296c2ef065ee8c4df165cda317..243af1c3da7979014ebb7e36fc4ae268915ac859 100644 (file)
@@ -25,7 +25,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} \
 }
 
 set bp_location [gdb_get_line_number "START"]
-if ![runto "endianity.c:$bp_location" ] then {
+if {![runto "endianity.c:$bp_location"]} {
   return -1
 }
 
index 0f03f0296d9a82fff98e2b71a2eae38b6c6b8287..bb5d6c2954804129ca917cbf3c00759be6d7da5d 100644 (file)
@@ -27,7 +27,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable $opts
 
 clean_restart $testfile
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "could not run to main"
     return
 }
index 659a8b8a61ef6fa5c073a1aff3e163f24fa5a455..fe6f7f0f52a674c6b90a860b0b7450e6ddc01d85 100644 (file)
@@ -42,7 +42,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 clean_restart ${binfile}
 
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 19b4e1a873243fcd269b87eac745fb8e5824679c..1276ec0984fa4e42a24291523586232ae39d08cf 100644 (file)
@@ -37,7 +37,7 @@ if { ! [file exists $binfile] } {
     return -1
     }
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 7f7599cbf86d07452fa2c000efcdd48866dfbe77..dec631b701a8b6a3b4b97f2767c1a66d2250325f 100644 (file)
@@ -50,7 +50,7 @@ proc do_exec_sysroot_test {} {
 
     # Start the program running, and stop at main.
     #
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
index 29647958e65c61fa5ba6b718a916d055c5990727..fdea0b442ad5e7b0a52b6fc33e58e47cf818d6fe 100644 (file)
@@ -42,7 +42,7 @@ if { [gdb_compile $objfile $exec1 executable {debug text_segment=0x1000000}] !=
 # exec2.  If it is, then skip the test as unsupported.
 
 clean_restart ${exec1}
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
@@ -56,7 +56,7 @@ gdb_test_multiple "p/x &main" $test {
 }
 
 clean_restart ${exec2}
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
@@ -88,7 +88,7 @@ proc test { always_inserted } {
 
     gdb_test_no_output "set breakpoint always-inserted $always_inserted"
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return -1
     }
 
index 0605dd62480c7728f1ef36dfc2e14209ce2d1e06..7b081ec91f9412d3bc2d2681793a9ff488300f61 100644 (file)
@@ -32,7 +32,7 @@ clean_restart ${binfile}
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index e00a1cefa0a577f16322ce59b47de5e589fb6d51..18e429c7553c651cb691eb776fd331c40b2dbc44 100644 (file)
@@ -55,7 +55,7 @@ gdb_test_no_output "set print sevenbit-strings"
 gdb_test_no_output "set print address off"
 gdb_test_no_output "set width 0"
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index e60c33b4d963b844f3d1981b1954d16ec45876ec..a0f5ced718bcdca3357ea4988664209e4de03a4a 100644 (file)
@@ -76,7 +76,7 @@ proc finish_void { } {
 
 proc finish_abbreviation { abbrev } {
 
-    if { ! [ runto "int_func" ] } then {
+    if {![runto "int_func"]} {
         return -1
     }
 
@@ -102,7 +102,7 @@ proc finish_no_print {} {
 proc finish_tests { } {
     global gdb_prompt skip_float_test
 
-    if { ! [ runto_main ] } then {
+    if {![runto_main]} {
        return -1
     }
 
index 5734fcaae7448199045a087662e3dad63a433697..0872017e0152ba8896bfb16149b4c1dc9d398890 100644 (file)
@@ -42,7 +42,7 @@ gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
 gdb_load_shlib ${lib_sl}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 1
 }
 
index 7fff2b90727b2cb9538c32d68bb587107ed19da4..b9c5057c6ee20816d4d83a515b1154aa83bc5825 100644 (file)
@@ -29,18 +29,18 @@ if { [prepare_for_testing "failed to prepare" float float.c] } {
 
 # Set it up at a breakpoint so we have its registers.
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
 
 # Test "info float".
 
-if { [is_aarch64_target] } then {
+if {[is_aarch64_target]} {
     gdb_test "info float" "d0.*d1.*d31.*s0.*s1.*s31.*"
-} elseif { [istarget "alpha*-*-*"] } then {
+} elseif {[istarget "alpha*-*-*"]} {
     gdb_test "info float" "f0.*"
-} elseif { [is_aarch32_target] } then {
+} elseif {[is_aarch32_target]} {
     gdb_test_multiple "info float" "info float" {
        -re "Software FPU type.*mask:.*flags:.*$gdb_prompt $" {
            pass "info float (FPA)"
@@ -54,11 +54,11 @@ if { [is_aarch64_target] } then {
             pass "info float (without FPU)"
        }
     }
-} elseif { [istarget "i?86-*-*"] || [istarget "x86_64-*-*"] } then {
+} elseif {[istarget "i?86-*-*"] || [istarget "x86_64-*-*"]} {
     gdb_test "info float" "R7:.*Status Word:.*Opcode:.*"
-} elseif [istarget "ia64-*-*"] then {
+} elseif {[istarget "ia64-*-*"]} {
     gdb_test "info float" "f0.*f1.*f127.*"
-} elseif [istarget "m68k-*-*"] then {
+} elseif {[istarget "m68k-*-*"]} {
     gdb_test_multiple "info float" "info_float" {
         -re "fp0.*fp1.*fp7.*$gdb_prompt $" {
             pass "info float (with FPU)"
@@ -67,7 +67,7 @@ if { [is_aarch64_target] } then {
             pass "info float (without FPU)"
        }
     }
-} elseif [istarget "mips*-*-*"] then {
+} elseif {[istarget "mips*-*-*"]} {
     gdb_test_multiple "info float" "info float" {
        -re "fpu type: none / unused\r\n$gdb_prompt $" {
              pass "info float (without FPU)"
@@ -76,7 +76,7 @@ if { [is_aarch64_target] } then {
              pass "info float (with FPU)"
          }
     }
-} elseif [istarget "nds32*-*-*"] then {
+} elseif {[istarget "nds32*-*-*"]} {
     gdb_test_multiple "info float" "info_float" {
         -re "fd0.*fd3.*$gdb_prompt $" {
             pass "info float (with FPU)"
@@ -85,7 +85,7 @@ if { [is_aarch64_target] } then {
             pass "info float (without FPU)"
        }
     }
-} elseif [istarget "powerpc*-*-*"] then {
+} elseif {[istarget "powerpc*-*-*"]} {
     gdb_test_multiple "info float" "info_float" {
         -re "f0.*f1.*f31.*fpscr.*$gdb_prompt $" {
             pass "info float (with FPU)"
@@ -94,9 +94,9 @@ if { [is_aarch64_target] } then {
             pass "info float (without FPU)"
        }
     }
-} elseif [istarget "s390*-*-*"] then {
+} elseif {[istarget "s390*-*-*"]} {
     gdb_test "info float" "fpc.*f0.*f1.*f15.*" "info float"
-} elseif [istarget "sh*-*"] then {
+} elseif {[istarget "sh*-*"]} {
     # SH may or may not have an FPU
     gdb_test_multiple "info float" "info float" {
        -re "fpul.*fr0.*fr1.*fr15.*$gdb_prompt $" {
@@ -106,11 +106,11 @@ if { [is_aarch64_target] } then {
              pass "info float (without FPU)"
        }
     }
-} elseif [istarget "hppa*-*"] then {
+} elseif {[istarget "hppa*-*"]} {
     gdb_test "info float" "fr4.*fr4R.*fr31R.*" "info float"
-} elseif [istarget "sparc*-*-*"] then {
+} elseif {[istarget "sparc*-*-*"]} {
     gdb_test "info float" "f0.*f1.*f31.*d0.*d30.*" "info float"
-} elseif [istarget "riscv*-*-*"] then {
+} elseif {[istarget "riscv*-*-*"]} {
     # RISC-V may or may not have an FPU.  Additionally, the order of
     # fcsr relative to fflags and frm can change depending on whether
     # the fflags and frm registers are implemented as real registers
index 5e471b39d22b67c248e263d9f663b060d6f7fa1c..a4d94d4c16215b3dd4f322d59731745d9700f95c 100644 (file)
@@ -32,7 +32,7 @@ if { [do_compile] != "" && [do_compile {-mfloat128}] != "" } {
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 1e4f94a70e0b54f4d05d553a2872c15cf2061feb..58be8eb4c37caff554708a47d4c77a7841d8f8a0 100644 (file)
@@ -32,7 +32,7 @@ if { [do_compile] != "" && [do_compile {-mfloat128}] != "" } {
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index cb6d975a767369a785b9971947e98d2068273ba2..23700156db4a63789d27b4d1133dbafd22f246b8 100644 (file)
@@ -35,7 +35,7 @@ if { [target_info exists gdb_protocol]
 
 # Until "catch exec" is implemented on other targets...
 #
-if {![istarget "*-linux*"]} then {
+if {![istarget "*-linux*"]} {
      return
 }
 
@@ -68,7 +68,7 @@ proc do_catch_exec_test { } {
 
     # Start the program running, and stop at main.
     #
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -107,7 +107,7 @@ proc do_follow_exec_mode_tests { mode cmd infswitch } {
 
        # Start the program running, and stop at main.
        #
-       if ![runto_main] then {
+       if {![runto_main]} {
            return
        }
 
index a3d1a8781e9ea831c29af1c89b6f44f4d9beb9f6..c66fc787bea5fcca9d072317364dfc2ba1e12adf 100644 (file)
@@ -18,7 +18,7 @@
 
 # Until "catch exec" is implemented on other targets...
 #
-if { ![istarget "*-linux*"] } then {
+if {![istarget "*-linux*"]} {
     return
 }
 
@@ -51,7 +51,7 @@ proc do_exec_tests {} {
 
    # Start the program running, and stop at main.
    #
-   if ![runto_main] then {
+   if {![runto_main]} {
      return
    }
 
@@ -77,7 +77,7 @@ proc do_exec_tests {} {
 
    # Start the program running, and stop at main.
    #
-   if ![runto_main] then {
+   if {![runto_main]} {
      return
    }
 
@@ -166,7 +166,7 @@ proc do_exec_tests {} {
 
    # Start the program running, and stop at main.
    #
-   if ![runto_main] then {
+   if {![runto_main]} {
      return
    }
 
@@ -234,7 +234,7 @@ proc do_exec_tests {} {
 
    # Start the program running, and stop at main.
    #
-   if ![runto_main] then {
+   if {![runto_main]} {
      return
    }
 
@@ -301,7 +301,7 @@ proc do_exec_tests {} {
 
    # Start the program running, and stop at main.
    #
-   if ![runto_main] then {
+   if {![runto_main]} {
      return
    }
 
@@ -362,7 +362,7 @@ proc do_exec_tests {} {
 
    # Start the program running, and stop at main.
    #
-   if ![runto_main] then {
+   if {![runto_main]} {
      return
    }
 
index 9627a177896c39fb5f79e0d149886c8cfc9a2cd1..c7711a17e5628ea685cfb029db7f8e6d77b27fbf 100644 (file)
@@ -21,7 +21,7 @@
 # Until "set follow-fork-mode" and "catch vfork" are implemented on
 # other targets...
 #
-if {![istarget "*-linux*"]} then {
+if {![istarget "*-linux*"]} {
     continue
 }
 
index 961852ebab1b2ea59c9b12a89b79c49350c6d7e3..4dab3532009337673346b056caf9fa2c083dadab 100644 (file)
@@ -35,7 +35,7 @@ proc do_test { detach_on_fork follow_fork non_stop schedule_multiple } {
        }
     }
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
index a067406857357305b5cf6091a07bbf280934acde..adc2a9e8bb8da149d3cc083fb5bfb7dd53cd03b8 100644 (file)
@@ -21,7 +21,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debu
 
 clean_restart ${binfile}
 
-if ![runto break_me] then {
+if {![runto break_me]} {
   perror "Couldn't run ${testfile}"
   return
 }
index 1a8c567a75e1b5cc90c508ceccf96e252c1a6095..966c350bcb37807a335b2c75becc761d88dbc56d 100644 (file)
@@ -97,7 +97,7 @@ proc test {dwarf_unwinder} {
 
     gdb_test_no_output "maint set dwarf unwinder $dwarf_unwinder"
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
index 7ea1cb92598190a5e9a5cf3653880e8155e738ae..ed48e5cd6519624cd7a370348326fc76f63ad104 100644 (file)
@@ -24,7 +24,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile}] } {
     return -1
 }
 
-if ![runto setup_done] then {
+if {![runto setup_done]} {
     return 0
 }
 
index 483fbd9426367f4b67e298f2af4d9cf40fe8f96b..e260001beae4a62b5e00d2819e5930fa2b83e3ec 100644 (file)
@@ -49,7 +49,7 @@ proc_with_prefix integral_args {} {
     gdb_breakpoint call0e
 
     # Run; should stop at call0a and print actual arguments.
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_run_cmd
     gdb_test "" " call0a \\(c=97 'a', s=1, i=2, l=3\\) .*" "run to call0a"
 
@@ -91,7 +91,7 @@ proc unsigned_integral_args {} {
     gdb_breakpoint call1e
 
     # Run; should stop at call1a and print actual arguments.
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_run_cmd
     gdb_test "" " call1a \\(uc=98 'b', us=6, ui=7, ul=8\\) .*" "run to call1a"
 
@@ -137,7 +137,7 @@ proc float_and_integral_args {} {
 
     # Run; should stop at call2a and print actual arguments.
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_run_cmd
     set test "run to call2a"
     gdb_test_multiple "" $test {
@@ -637,7 +637,7 @@ proc_with_prefix shuffle_round_robin {} {
     gdb_run_cmd
     gdb_test "" "Breakpoint $decimal, call7a .*" "run to call7a"
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test_multiple "backtrace 100" "backtrace from call7a" {
        -re " call7a \\(c=97 'a', i=2, s=1, l=3, f=4, uc=98 'b', d=5, us=6, ul=8, ui=7\\) .* main \\(.*\\) .*$gdb_prompt $" {
            pass "backtrace from call7a"
@@ -652,7 +652,7 @@ proc_with_prefix shuffle_round_robin {} {
 
     gdb_continue call7b
 
-    if {$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {$gcc_compiled} { setup_xfail "rs6000-*-*" }
 
     gdb_test_sequence "backtrace 100" "backtrace from call7b" {
        "\[\r\n\]#0 .* call7b \\(i=2, s=1, l=3, f=4, uc=98 'b', d=5, us=6, ul=8, ui=7, c=97 'a'\\) "
@@ -853,9 +853,9 @@ proc_with_prefix localvars_after_alloca { } {
     # Print each arg as a double check to see if we can print
     # them here as well as with backtrace.
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "print c" " = 97 'a'" "print c after runto localvars_after_alloca"
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "print s" " = 1" "print s after runto localvars_after_alloca"
     gdb_test "print i" " = 2" "print i after runto localvars_after_alloca"
     gdb_test "print l" " = 3" "print l after runto localvars_after_alloca"
@@ -887,14 +887,14 @@ proc_with_prefix call_after_alloca { } {
     # Print each arg as a double check to see if we can print
     # them here as well as with backtrace.
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "print c" " = 97 'a'" "print c in call_after_alloca"
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "print s" " = 1" "print s in call_after_alloca"
     gdb_test "print i" " = 2" "print i in call_after_alloca"
     gdb_test "print l" " = 3" "print l in call_after_alloca"
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "backtrace 8" "#0.*call_after_alloca_subr \\(c=97 'a', s=1, i=2, l=3, uc=98 'b', us=11, ui=12, ul=13\\).*#1.*call_after_alloca \\(c=97 'a', s=1, i=2, l=3\\).*#2.*main.*" "backtrace from call_after_alloca_subr"
 }
 
@@ -925,14 +925,14 @@ proc_with_prefix localvars_in_indirect_call { } {
     # Print each arg as a double check to see if we can print
     # them here as well as with backtrace.
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "print c" " = 97 'a'" "print c in localvars_in_indirect_call"
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "print s" " = 1" "print s in localvars_in_indirect_call"
     gdb_test "print i" " = 2" "print i in localvars_in_indirect_call"
     gdb_test "print l" " = 3" "print l in localvars_in_indirect_call"
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "backtrace 8" \
        "#0.*call0a \\(c=97 'a', s=1, i=2, l=3\\).*#1.*main.*" \
        "backtrace in indirectly called function"
@@ -965,7 +965,7 @@ proc_with_prefix localvars_in_indirect_call { } {
        }
     }
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "step" "call0a \\(c=97 'a', s=1, i=2, l=3\\).*" \
        "stepping into indirectly called function"
 }
index 609d61328f4b6d1777dc9cf55c685cc97510cab1..ead21e62593fd721ca56ad7fde51c570a30ff845 100644 (file)
@@ -34,7 +34,7 @@ clean_restart ${binfile}
 gdb_test_no_output "set args ${pattern}"       \
     "set buffer exceeding arguments"
 
-if { ! [ runto_main ] } then {
+if {![runto_main]} {
     return -1
 }
 
index 93d943c32e42a08f276c119b62672aa2c5bdb33f..1c836c29d25db0de07ce465870de102bfa05eba1 100644 (file)
@@ -23,7 +23,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if { ! [ runto_main ] } then {
+if {![runto_main]} {
     return -1
 }
 
@@ -81,21 +81,21 @@ gdb_test_sequence "where" "where in corefile" {
 }
 
 set post_corefile_regs [capture_command_output "info registers" ""]
-if ![string compare $pre_corefile_regs $post_corefile_regs] then {
+if {![string compare $pre_corefile_regs $post_corefile_regs]} {
     pass "corefile restored general registers"
 } else {
     fail "corefile restored general registers"
 }
 
 set post_corefile_allregs [capture_command_output "info all-reg" ""]
-if ![string compare $pre_corefile_allregs $post_corefile_allregs] then {
+if {![string compare $pre_corefile_allregs $post_corefile_allregs]} {
     pass "corefile restored all registers"
 } else {
     fail "corefile restored all registers"
 }
 
 set post_corefile_sysregs [capture_command_output "info reg system" ""]
-if ![string compare $pre_corefile_sysregs $post_corefile_sysregs] then {
+if {![string compare $pre_corefile_sysregs $post_corefile_sysregs]} {
     pass "corefile restored system registers"
 } else {
     fail "corefile restored system registers"
index 16bc19b2a80c1733b250f459fd06b0bfda81ffd6..36f5fa0fb68f1c9eb5e7f76dcfe7a82baff9a122 100644 (file)
@@ -28,7 +28,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 1
 }
 
index 7f16965d11d6cc30c1ca66993043c335c61cc11e..77d61c7cbbd79fd8602be04f389f3f96ad598d54 100644 (file)
@@ -31,7 +31,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $testfile.c {debug}] } {
 }
 
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return
 }
 
index 4520a0dc12582f8992e7eb080f150cfed03409bb..d27dcfe8fc03f94c62a3cd2d1971016439d4a2a1 100644 (file)
@@ -38,7 +38,7 @@ if {$skip_hw_watchpoint_tests_p} {
    gdb_test_no_output "set can-use-hw-watchpoints 0" ""
 }
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return
 }
 
index 06390b1002bf65da18632b0dab13d26e2b850421..70e219ce5809150271f43fd474c07adc9db488f3 100644 (file)
@@ -31,7 +31,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 
 clean_restart ${binfile}
 
-if ![runto abort {allow-pending}] then {
+if {![runto abort {allow-pending}]} {
     return
 }
 
index c8c86edbc23fb98017af16afeffa946a0ed6eaf1..1d603ab94c0c0a19c54ef251734b9372d331a1f7 100644 (file)
@@ -39,7 +39,7 @@ if { [gdb_compile_shlib $libsrc $libobj {debug}] != ""
 clean_restart ${binfile}
 gdb_load_shlib $libobj
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index cfa95aa5c24ad3cbf83657b4e3aae34e163b14fb..40dcb0cc75ba94dcca9ba38050042316ece4d562 100644 (file)
@@ -28,7 +28,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 } 
index 3fbf237c71cc8ffc34c42d016249a9d33f155de7..715f5be04cb99f6bfb643fb569aa2be2716adfb9 100644 (file)
@@ -105,7 +105,7 @@ proc_with_prefix set-break {resolver_attr resolver_debug final_debug} {
     clean_restart $binfile-$suffix
     gdb_load_shlib ${lib_so}
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 1
     }
 
@@ -227,7 +227,7 @@ proc misc_tests {resolver_attr resolver_debug final_debug} {
     clean_restart $binfile-$suffix
     gdb_load_shlib ${lib_so}
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 1
     }
 
index 796d9e947e12347f54ca2c57ecf27512a66a1d3d..5b6eb7af5a0da32f9febbe50b1206391e58d3085 100644 (file)
@@ -105,7 +105,7 @@ gdb_test "ptype (char4) 0x01010101" \
 gdb_test "ptype (int2) lla" \
     "type = int __attribute__ \\(\\(vector_size\\(2\\)\\)\\)"
 
-if { ![string compare $endian big] } then {
+if {![string compare $endian big]} {
     gdb_test "print (char4) ia" "\\\$$decimal = \\{0, 0, 0, 2\\}"
 } else {
     gdb_test "print (char4) ia" "\\\$$decimal = \\{2, 0, 0, 0\\}"
index 6073a64d0a1b3cc1f5382d4c4b6cd249bd3080cc..f9a1d07b0d099bcb424b896e75538b78e0b11e7a 100644 (file)
@@ -34,7 +34,7 @@ if { [have_index $binfile] != "gdb_index" } {
 
 gdb_test_no_output "set auto-solib-add off"
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index a521637e17e81bdb484dbcee0890237940ae0839..4836e133107e770f93f42c41ad238a369f195166 100644 (file)
@@ -39,7 +39,7 @@ if { [gdb_compile_shlib ${srcdir}/${subdir}/${lib_src} ${lib_so} $lib_opts] != "
 clean_restart $binfile
 gdb_load_shlib $lib_so
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index aecf613643d68807a3aaf582e14f4dcd6ce2dc5f..a5e870a5708058e4da30dbcc4c680e932db1ee7a 100644 (file)
@@ -25,7 +25,7 @@ if { [prepare_for_testing "failed to prepare" "hbreak2" {break.c break1.c} {debu
 set srcfile break.c
 set srcfile1 break1.c
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -287,7 +287,7 @@ gdb_test "info break" \
 
 #***********
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -315,7 +315,7 @@ gdb_test "hbreak" "Hardware assisted breakpoint \[0-9\]*.*" \
 # Verify that a "silent" breakpoint can be set, and that GDB is indeed
 # "silent" about its triggering.
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -520,7 +520,7 @@ if { [prepare_for_testing "failed to prepare" "hbreak2o2" {break.c break1.c} {de
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index f861709e491483671d60bc348d8ce5d6797136bb..a4f5ec2b82a9791551da8878528b422e09c8a523 100644 (file)
@@ -39,7 +39,7 @@ proc setup {commands} {
 
     clean_restart $binfile
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return -code return
     }
 
index e8274626e8a69a61e94e31a688939f70e5b4a985..9be4f328f87b70837c54781614f7153503ee63f8 100644 (file)
@@ -42,7 +42,7 @@ clean_restart ${binfile}
 set prev_timeout $timeout
 set timeout 30
 
-if { ! [ runto_main ] } then {
+if {![runto_main]} {
     return -1
 }
 
index a09a28a69049043da9057402eeafd74a75e218a2..a1e98ee194561d1fbd798a16396bf4137614291c 100644 (file)
@@ -27,7 +27,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 691ce0ae8843e6d9854277d054be53249df526a4..30ce076161243764d7e4925906368b964168c227 100644 (file)
@@ -16,7 +16,7 @@
 # Test running an inferior with arguments.
 
 # This does not work on boards that don't support inferior arguments.
-if [target_info exists noargs] then {
+if {[target_info exists noargs]} {
     verbose "skipping gdb.base/inferior-args.exp because of noargs"
     return
 }
index 28ea8df1e74f4d83bb41d2a82f213ecaf77f25c5..fc0a722d4b44c4b95291eaeb50f7ef9cacbcad54 100644 (file)
@@ -16,7 +16,7 @@
 # Until "set follow-fork-mode" and "catch fork" are implemented on
 # other targets...
 #
-if { ![istarget "*-*-linux*"] } then {
+if {![istarget "*-*-linux*"]} {
     unsupported "inferior-died.exp"
     return
 }
index e683bc3aa7077d7b47b9984531db1bf7eb159393..f2ebd6f19b79c67318b9618d8e5d1f08bcbcb993 100644 (file)
@@ -53,7 +53,7 @@ foreach_with_prefix n_flag { 0 1 } {
 
        clean_restart $executable
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return 0
        }
 
index 8c4027bcc8aaa23f73b44922fed08e8639125e67..221e416e2afb40712481ae4daa90ad22d928bb3f 100644 (file)
@@ -32,7 +32,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index d12f8750a9cc5b319f520e2f22417a5469a6f648..f4eb42fcb8baec587b54b745bbe44aa8c57315f6 100644 (file)
 standard_testfile .c
 
 # This test is Linux-only.
-if ![istarget *-*-linux*] then {
+if {![istarget *-*-linux*]} {
     unsupported "info-os.exp"
     return -1
 }
 
 # Support for XML-output is needed to run this test.
-if [gdb_skip_xml_test] then {
+if {[gdb_skip_xml_test]} {
     unsupported "info-os.exp"
     return -1
 }
@@ -32,7 +32,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile {debug pthreads
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index f90c9c45c71a6e69db75fe174168e45d7e7dbc01..179efb4085543ebaef473197596aeac1a6d8a878 100644 (file)
@@ -48,7 +48,7 @@ if { ! [use_gdb_stub] } {
     gdb_test_no_output "set args $cmdline" "set args"
 }
 
-if { ! [ runto_main ] } then {
+if {![runto_main]} {
     return -1
 }
 
index 1dc470b6e79fc74fcad203261b905b67f6a07734..1496ea518dd7ea7e72d565a16d803263b08ba8c4 100644 (file)
@@ -33,7 +33,7 @@ gdb_test "next" ".*" "step before info program"
 gdb_test "info program" "Program stopped at $hex\.\r\nIt stopped after being stepped\.\r\nType \"info stack\" or \"info registers\" for more information\." \
     "info program after next"
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return -1
 }
 
index e51d0279d709c802a0138cb05cf6bf82e4f7eb9e..ee61c277be035f56916b8f4a45c2337f421f6ebe 100644 (file)
@@ -27,7 +27,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile}] } {
     return -1
 }
 
-if ![runto setup_done] then {
+if {![runto setup_done]} {
     return 0
 }
 
index c1273f148b1a2181fae14de72320bea851fed4c5..4001bc9b7ca0b89660166ac8f496153a8bb15248 100644 (file)
@@ -58,7 +58,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}]} {
 # Skip this test if Python scripting is not enabled.
 if { [skip_python_tests] } { continue }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 3db90f64103047d04fc5f3c8d2f5ecd6a77fa8c4..a42d4ae60ae5c069712d87a328aa00d45d900ab8 100644 (file)
@@ -90,7 +90,7 @@ gdb_test "$commands\nend" "" "finish defining hello command"
 
 gdb_test "hello" [string_to_regexp "^done,value=\"23\""]
 
-if ![runto_main] then {
+if {![runto_main]} {
   return -1
 }
 
index 26741d59112dd25d238edeacf52551cb490c3f2b..143116c4124319bc5a69f1530650ca05b80d0b82 100644 (file)
@@ -38,7 +38,7 @@ if {[build_executable $testfile.exp $testfile $srcfile $options] == -1} {
 gdb_start
 
 
-if ![file exists $binfile] then {
+if {![file exists $binfile]} {
     perror "$binfile does not exist."
     return 0
 } else {
@@ -47,7 +47,7 @@ if ![file exists $binfile] then {
     # Hope this is unix :-)
     gdb_test "shell stty intr '^C'" ".*" \
        "set interrupt character"
-    if [runto_main] then {
+    if {[runto_main]} {
        global inferior_spawn_id gdb_spawn_id
 
        set msg "process is alive"
index ad47c5a96103de4b657a31d7a1d2a10f032b2c90..61139728b6edfee64ec02ac6b742bce13585a0d4 100644 (file)
@@ -17,7 +17,7 @@
 # inferior that used the JIT API then exec'd a program that did not
 # use it.
 
-if { ![istarget "*-linux*"] } then {
+if {![istarget "*-linux*"]} {
     return
 }
 
index 91fde018b4de580cff78ca35ec78c81b81039763..527a1ae22d5bc81e5a3ba232098fd272ddbc3bcb 100644 (file)
@@ -29,7 +29,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
   perror "Couldn't run to main"
   return -1
 }
index 9c4a94911157a3c2b155014a90e90304298a8e76..e55cf7ffdac5cb81b37edc863da0259520e7efe8 100644 (file)
@@ -19,7 +19,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index fdc813336470a7d6d9acbf4ca7e01f92d971381d..1957927dbfec6c5af147aee2450f3c6254640f93 100644 (file)
@@ -49,7 +49,7 @@ set isfixed 0
 set foo_func foo__Fi
 set do_func langs0__2do
 
-if [runto csub] then {
+if {[runto csub]} {
 
     if { !$isfixed } { set lang c }
     gdb_test "show language" "currently $lang\".*" \
@@ -58,20 +58,20 @@ if [runto csub] then {
     gdb_test "bt" "#0.*csub.*#1.*(foo|$foo_func) \\(.*#2.*cppsub_ .*#3.*fsub.*#4.*$do_func \\(.*#5  \[0-9a-fx\]* in main.*" "backtrace"
 
     if { !$isfixed } { set lang c\\+\\+; set ext cxx }
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "up" ".* in (foo|$foo_func).* at .*langs2\\.$ext.*return csub \\(.*" \
        "up to foo"
     gdb_test "show language" "currently $lang.*" \
        "show language at foo"
 
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "up" ".* in cppsub_ .* at .*langs2\\.$ext.*return foo \\(.*" \
        "up to cppsub_"
     gdb_test "show language" "currently $lang.*" \
        "show language at cppsub_"
 
     if { !$isfixed } { set lang fortran }
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "up" ".* in fsub.* at .*langs1\\.f.*" \
        "up to fsub"
     gdb_test "show language" "currently $lang.*" \
@@ -100,7 +100,7 @@ gdb_load $binfile
 
 # Try exercising the "minimal" language a bit...
 
-if [runto csub] then {
+if {[runto csub]} {
     gdb_test "set lang minimal" \
              "Warning: the current language does not match this frame." \
              "set lang to minimal"
@@ -117,7 +117,7 @@ if [runto csub] then {
        "set lang back to auto for verbose test"
     gdb_test_no_output "set verbose on"
     if { !$isfixed } { set lang c\\+\\+; set ext cxx }
-    if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+    if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
     gdb_test "up" \
        ".* in (foo|$foo_func).* at .*langs2\\.$ext.*return csub \\(.*Current language:  auto.*" \
        "up to foo when verbose"
index b58a43f81a35eca934365a175f7c67cf36c4cb63..94bfd5527f16bcb99aeeee44396ef504e68c0319 100644 (file)
@@ -39,7 +39,7 @@ proc run_test { opt_level } {
        return
     }
 
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return
     }
 
index 6801ab4289c19b8a67141435bde5d00e7a9b3773..e3b8d142f10c375a4bf3d402eaf4d6eb3e3f3eba 100644 (file)
@@ -45,7 +45,7 @@ remote_file host delete $srcfile
 # Now start GDB, run to main and try to list the source.
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index fe894a0098e01851a6821d350ecc49026c63dd06..9cd1daacc33961fab3c6d0967a0415458878eb73 100644 (file)
@@ -98,7 +98,7 @@ proc test_listsize {} {
 
     # Try small listsize > 2 that is an even number.
 
-    if [ set_listsize 4 ] then {
+    if {[set_listsize 4]} {
        gdb_test "list 1" "1\[ \t\]+#include \"list0.h\".*4\[ \t\]+\{" "list line 1 with listsize 4"
        gdb_test "list 2" "1\[ \t\]+#include \"list0.h\".*4\[ \t\]+\{" "list line 2 with listsize 4"
        
@@ -108,7 +108,7 @@ proc test_listsize {} {
 
     # Try a size larger than the entire file.
 
-    if [ set_listsize 100 ] then {
+    if {[set_listsize 100]} {
        gdb_test "list 1" "1\[ \t\]+#include \"list0.h\".*\r\n${last_line_re}" "list line 1 with listsize 100"
        
        gdb_test "list 10" "1\[ \t\]+#include \"list0.h\".*\r\n${last_line_re}" "list line 10 with listsize 100"
@@ -411,7 +411,7 @@ gdb_test_no_output "set width 0"
 
 test_listsize
 get_debug_format
-if [ set_listsize 10 ] then {
+if {[set_listsize 10]} {
     test_list_include_file
     test_list_filename_and_number
     test_list_function
@@ -466,7 +466,7 @@ proc test_list {command listsize1 listsize2 linerange1 linerange2} {
        global binfile
 
        clean_restart $binfile
-       if ![runto_main] then {
+       if {![runto_main]} {
            return
        }
 
index b1d3c757768326732419fe3585570741f8891ee2..25eb0711b8249c46b146801d7e20c3e4ca9c3af0 100644 (file)
@@ -39,7 +39,7 @@ clean_restart ${binfile}
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 784eaf9b5a08d4d95037c62838e08a966cea9b40..e3d1f991f7df83eb69c621bb06312a1888d5050e 100644 (file)
@@ -39,7 +39,7 @@ if [prepare_for_testing "failed to prepare" $testfile {} {debug}] {
     return -1
 }
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return
 }
 
index 9c8dd16b7ebb091e001ab439b7cbbef8b7da104d..81d006f73798f5593d3265d0732f183aba10bebe 100644 (file)
@@ -34,7 +34,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [con
 
 clean_restart ${binfile}
 
-if { ![runto known_types] } then { 
+if {![runto known_types]} {
    return 
 }
 
index f301cac2897ca1f7227f7aa86c7f65e3befde887..bd2fc21a442a4c6bf72f9256c59cf773826f1d1a 100644 (file)
@@ -27,7 +27,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index 4c7987e84c7f9b7870353e1b9cc2548d280c6715..766a3af8f2b48246c8e227f8c4e7ecd1dbb0585a 100644 (file)
@@ -25,7 +25,7 @@ if {[prepare_for_testing "failed to prepare" $testfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 395927a22c42f7521bfbb0114c120afaae22136e..17190ccea5a0017df693e0b575c5f85911eaee72 100644 (file)
@@ -22,7 +22,7 @@ if {[prepare_for_testing "failed to prepare" $testfile \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index 16628b4a8b63ca9939572e72eb8e14f19a19a1e7..981c2f626ac4b7b4feb65e9f512c732cb700f54e 100644 (file)
@@ -113,7 +113,7 @@ gdb_test "mt set per off" ".*" "mt set per off for expand-symtabs"
 
 gdb_load ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
         perror "tests suppressed"
 }
 
index 94e2ff6154a8112554db16874c773f2c9adcaeff..a05cca325741e3aa3356373eaa3f86abb096a44f 100644 (file)
@@ -19,7 +19,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
@@ -46,7 +46,7 @@ proc get_max_value_size {} {
 proc do_value_printing { max_value_size test_prefix } {
     with_test_prefix ${test_prefix} {
        gdb_test "p/d one" " = 0"
-       if { $max_value_size != "unlimited" && $max_value_size < 100 } then {
+       if {$max_value_size != "unlimited" && $max_value_size < 100} {
            gdb_test "p/d one_hundred" \
                "value requires 100 bytes, which is more than max-value-size"
        } else {
@@ -59,7 +59,7 @@ proc do_value_printing { max_value_size test_prefix } {
 # Install SET_VALUE as the value for max-value-size, then print the
 # test values.
 proc set_and_check_max_value_size { set_value } {
-    if { $set_value == "unlimited" } then {
+    if {$set_value == "unlimited"} {
        set check_pattern "unlimited"
     } else {
        set check_pattern "${set_value} bytes"
index 3420e6c5cae321334d08a9e2c866edc1e6a3a0f9..28ca8293b5e46616be9a08bf9f9e63c192fc4b81 100644 (file)
@@ -142,7 +142,7 @@ gdb_test_multiple "info mem" "info mem(1)" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { $see1 && $see2 && $see3 && $see4 && $see5 } then {
+       if {$see1 && $see2 && $see3 && $see4 && $see5} {
            pass "info mem (1)"
        } else {
            fail "info mem (1)"
@@ -216,7 +216,7 @@ gdb_test_multiple "info mem" "mem 2 and 4 were disabled" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { $see1 && $see2 && $see3 && $see4 && $see5 } then {
+       if {$see1 && $see2 && $see3 && $see4 && $see5} {
            pass "mem 2 and 4 were disabled"
        } else {
            fail "mem 2 and 4 were disabled"
@@ -258,7 +258,7 @@ gdb_test_multiple "info mem" "mem 2-4 were enabled" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { $see1 && $see2 && $see3 && $see4 && $see5 } then {
+       if {$see1 && $see2 && $see3 && $see4 && $see5} {
            pass "mem 2-4 were enabled"
        } else {
            fail "mem 2-4 were enabled"
@@ -300,7 +300,7 @@ gdb_test_multiple "info mem" "mem 1 to 5 were disabled" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { $see1 && $see2 && $see3 && $see4 && $see5 } then {
+       if {$see1 && $see2 && $see3 && $see4 && $see5} {
            pass "mem 1 to 5 were disabled"
        } else {
            fail "mem 1 to 5 were disabled"
@@ -342,7 +342,7 @@ gdb_test_multiple "info mem" "mem 1 to 5 were enabled" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { $see1 && $see2 && $see3 && $see4 && $see5 } then {
+       if {$see1 && $see2 && $see3 && $see4 && $see5} {
            pass "mem 1 to 5 were enabled"
        } else {
            fail "mem 1 to 5 were enabled"
@@ -391,7 +391,7 @@ gdb_test_multiple "info mem" "mem 1 was deleted" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { !$see1 && $see2 && $see3 && $see4 && $see5 } then {
+       if {!$see1 && $see2 && $see3 && $see4 && $see5} {
            pass "mem 1 was deleted"
        } else {
            fail "mem 1 was deleted"
@@ -432,7 +432,7 @@ gdb_test_multiple "info mem" "mem 2 and 4 were deleted" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { !$see1 && !$see2 && $see3 && !$see4 && $see5 } then {
+       if {!$see1 && !$see2 && $see3 && !$see4 && $see5} {
            pass "mem 2 and 4 were deleted"
        } else {
            fail "mem 2 and 4 were deleted"
@@ -474,7 +474,7 @@ gdb_test_multiple "info mem" "mem 2-4 were deleted" {
        exp_continue
     }
     -re "$gdb_prompt $" {
-       if { !$see1 && !$see2 && !$see3 && !$see4 && $see5 } then {
+       if {!$see1 && !$see2 && !$see3 && !$see4 && $see5} {
            pass "mem 2-4 were deleted"
        } else {
            fail "mem 2-4 were deleted"
index fe5eb94f0011b62c4ff3a781e3e68b900dc545bd..45186161cb758cbccbc710f960d65b9c92a70214 100644 (file)
@@ -19,7 +19,7 @@ standard_testfile
 # This test must be compiled with -O2 if using gcc.
 
 set options debug
-if { [test_compiler_info gcc-*-*] } then {
+if {[test_compiler_info gcc-*-*]} {
     lappend options additional_flags=-O2 additional_flags=-fno-inline
 }
 
@@ -27,11 +27,11 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile $options]} {
     return -1
 }
 
-if [runto middle] then {
+if {[runto middle]} {
     # PR 3016
     #   warning: Hit heuristic-fence-post without finding
     #   warning: enclosing function for pc 0x1006ead0
-    if { [test_compiler_info gcc-*-*] } then {
+    if {[test_compiler_info gcc-*-*]} {
        setup_xfail "mips64*-*-elf"
     }
     # The call chain is main -> top -> middle.  But gcc can optimize a tail
index 7d9542fbebae19e6f73a459980f4a7d4b6981938..1cf121751b49d0c40331aaa72e37c6989c00e1bd 100644 (file)
@@ -50,7 +50,7 @@ clean_restart ${binfile}
 #
 # set it up at a breakpoint so we can play with the variable values
 #
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 0e35fbbc4228f5e8f488389feed349c56b704137..14ff95c53bbed8e9691f7d57598f0fcb03faa219 100644 (file)
@@ -16,7 +16,7 @@
 # Until "set follow-fork-mode" and "catch fork" are implemented on
 # other targets...
 #
-if { ![istarget "*-*-linux*"] } then {
+if {![istarget "*-*-linux*"]} {
     return
 }
 
index 906dc7e823962ca19b9422367c6cd40183cce144..8eca167da80c1a9b7ddd5346db98d5fad71aadde 100644 (file)
@@ -84,7 +84,7 @@ proc test_call_promotion {} {
     gdb_test "p /d ((uint8 (*) ()) add8_noproto)((uint8) 2, (uint8) 3)" " = 5"
 }
 
-if [nodebug_runto inner] then {
+if {[nodebug_runto inner]} {
     
     # Expect to find global/local symbols in each of text/data/bss.
     
@@ -292,7 +292,7 @@ if [nodebug_runto inner] then {
 
     # Now, try that we can give names of file-local symbols which happen
     # to be unique, and have it still work
-    if [nodebug_runto middle] then {
+    if {[nodebug_runto middle]} {
        gdb_test "backtrace 10" "#0.*middle.*#1.*top.*#2.*main.*" \
            "backtrace from middle"
     }
index 66686cfdd633ddbedbedb991f1cf761403ddb1e3..e35cfd9122fcd7cff116cb24d5751d6f07495216 100644 (file)
@@ -22,7 +22,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index b4e358e14c2f26e33ee295a5acf58de3e9d3c41c..140fb91968cb11673fcf2db0bc5e60e793829259 100644 (file)
@@ -22,7 +22,7 @@ if [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} {debug}] {
 proc noreturn_finish_test { } {
     global gdb_prompt
 
-    if { ! [ runto_main ] } then {
+    if {![runto_main]} {
        return -1
     }
 
index 2560908d58df1c260c15be45825d3bac8c1e8016..6db7023c18118b998847b947fa2ac311b16a2459 100644 (file)
@@ -22,7 +22,7 @@ if [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} {debug}] {
 proc noreturn_test { } {
     global gdb_prompt
 
-    if { ! [ runto_main ] } then {
+    if {![runto_main]} {
        return -1
     }
 
index 4d7a325788061453a2c96cf5c5675f1a1f1f43fe..4d3f11b32a9bed96920ec92e11981f2ce0de5dea 100644 (file)
@@ -51,7 +51,7 @@ proc setup_xfail_on_opaque_pointer {} {
        global gcc_compiled
 
        setup_xfail "vax-*-*" "i*86-sequent-bsd*"
-       if {!$gcc_compiled} then {
+       if {!$gcc_compiled} {
                setup_xfail "alpha-*-*"
        }
 }
@@ -59,7 +59,7 @@ proc setup_xfail_on_opaque_pointer {} {
 # This seems easier than trying to track different versions of xlc; I'm
 # not sure there is much rhyme or reason regarding which tests it fails
 # and which ones it passes.
-if {[istarget "rs6000-*-aix*"] && !$gcc_compiled} then {
+if {[istarget "rs6000-*-aix*"] && !$gcc_compiled} {
     warning "xfails in opaque.exp may not be set up correctly for xlc"
 }
 
@@ -72,7 +72,7 @@ gdb_test "whatis foop" \
 # Ensure that we know the form of the structure that foop points to.
 
 setup_xfail_on_opaque_pointer
-if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
 gdb_test "ptype foop" \
     "type = struct foo \{\[\r\n\]+    int a;\[\r\n\]+    int b;\[\r\n\]+\} \[*\]+" \
     "ptype on opaque struct pointer (statically)"
@@ -128,7 +128,7 @@ gdb_test "whatis foop" \
 # Ensure that we know the form of the thing foop points to.
 
 setup_xfail_on_opaque_pointer
-if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
 gdb_test "ptype foop" \
     "type = struct foo \{\[\r\n\]+    int a;\[\r\n\]+    int b;\[\r\n\]+\} \[*\]+" \
     "ptype on opaque struct pointer (dynamically) 1"
@@ -147,7 +147,7 @@ gdb_test "ptype afoo" \
 
 # Ensure that we know the form of an explicit struct foo.
 
-if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" }
+if {!$gcc_compiled} { setup_xfail "rs6000-*-*" }
 gdb_test "ptype struct foo" \
     "type = struct foo \{\[\r\n\]+    int a;\[\r\n\]+    int b;\[\r\n\]+\}" \
     "ptype on opaque struct tagname (dynamically) 1"
index e3c94ac03d9417d789ab6048efe512a14173094b..2c74107297497844e5d458c31f17df64215ffcdc 100644 (file)
 
 set data_overlays 1
 
-if [istarget "m32r-*-*"] then {
+if {[istarget "m32r-*-*"]} {
     set linker_script "${srcdir}/${subdir}/m32r.ld"
 } else {
     verbose "Skipping overlay test -- not implemented for this target."
     return 
 }
 
-if [istarget "*-*-linux*"] then {
+if {[istarget "*-*-linux*"]} {
     verbose "Skipping overlay test -- Linux doesn't support overlayed programs."
     return 
 }
@@ -54,7 +54,7 @@ gdb_load ${binfile}
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
@@ -95,7 +95,7 @@ set bar_lma  [get_func_address "bar"  "\\*bar\\*"  "bar  load address"]
 set baz_lma  [get_func_address "baz"  "\\*baz\\*"  "baz  load address"]
 set grbx_lma [get_func_address "grbx" "\\*grbx\\*" "grbx load address"]
 
-if $data_overlays then {
+if {$data_overlays} {
     gdb_test "print \$foox_lma = &foox" \
        ".* $iptrcast 0x.*"  "foox load addr"
     gdb_test "print \$barx_lma = &barx" \
@@ -125,7 +125,7 @@ gdb_test "overlay map .ovly3" ""
 gdb_test "overlay list" "Section .ovly3, loaded at.*, mapped at.*" "list ovly3"
 set grbx_vma [get_func_address "grbx" "grbx" "grbx runtime address"]
 
-if $data_overlays then {
+if {$data_overlays} {
     gdb_test "overlay map .data00" "" 
     gdb_test "overlay list" "Section .data00, loaded .*, mapped .*" "list data00"
     gdb_test "print \$foox_vma = &foox" \
@@ -152,7 +152,7 @@ gdb_test "print $foo_lma   != $foo_vma" ".* = 1"   "foo's LMA   != VMA"
 gdb_test "print $bar_lma   != $bar_vma" ".* = 1"   "bar's LMA   != VMA"
 gdb_test "print $baz_lma   != $baz_vma" ".* = 1"   "baz's LMA   != VMA"
 gdb_test "print $grbx_lma  != $grbx_vma" ".* = 1"  "grbx's LMA  != VMA"
-if $data_overlays then {
+if {$data_overlays} {
     gdb_test "print \$foox_lma  != \$foox_vma" ".* = 1"  "foox's LMA  != VMA"
     gdb_test "print \$barx_lma  != \$barx_vma" ".* = 1"  "barx's LMA  != VMA"
     gdb_test "print \$bazx_lma  != \$bazx_vma" ".* = 1"  "bazx's LMA  != VMA"
@@ -203,7 +203,7 @@ simultaneous_pair .ovly0 .ovly3
 simultaneous_pair .ovly1 .ovly2
 simultaneous_pair .ovly1 .ovly3
 
-if $data_overlays then {
+if {$data_overlays} {
     simultaneous_pair .data00 .data02
     simultaneous_pair .data00 .data03
     simultaneous_pair .data01 .data02
index df466f2193fd467e9e0e1c992ace20d096c9da1c..6ab88a18f87402b5cbe505d9d9eb38aabe856897 100644 (file)
@@ -36,7 +36,7 @@ proc test_ctrlc_while_target_running_does_not_paginate {} {
     with_test_prefix "ctrlc target running" {
        clean_restart $binfile
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return 0
        }
 
index 34d5ceb1703292ffea1f6ecfdeb95d11392af31f..3074eea74c8d15a45248f21d41bc0259dae9dafd 100644 (file)
@@ -32,7 +32,7 @@ proc test_bg_execution_pagination_return {} {
     with_test_prefix "paginate" {
        clean_restart $binfile
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return 0
        }
 
index 63cb98a82c347c97d4f0cb20516978d1c815836a..eb38daf8f8c31e106ccb8924f5e1b0f4e46aad9f 100644 (file)
@@ -31,7 +31,7 @@ proc test_paginate_inferior_exited {} {
     with_test_prefix "paginate" {
        clean_restart $binfile
 
-       if ![runto_main] then {
+       if {![runto_main]} {
            return 0
        }
 
index b0e7ec9b3359e01ec792e8951ee273696a113fe5..191c842af2494f92e1534f81fa23a8751e28aa98 100644 (file)
@@ -30,7 +30,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile {debug nowarning
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index f13e985e77ec8d39d291bd6af99113d0d69eafda..af6c7f47ac8c3af62cc4a5655f31b92e3a33a015 100644 (file)
@@ -81,7 +81,7 @@ gdb_test_no_output "set non-stop off"
 
 gdb_load ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index f5dce989e28148d0ec16f2f5e42e4fbc69d1692d..85affd3e2c23a286ed5ffca11eea98ba8c2caf23 100644 (file)
@@ -25,8 +25,8 @@ if ![istarget *-linux*] {
 
 # In remote mode we cannot use the 'set args' command, and this
 # test requires it.
-if { [target_info exists gdb_protocol] } then {
-    if { [target_info gdb_protocol] == "remote" } then {
+if {[target_info exists gdb_protocol]} {
+    if {[target_info gdb_protocol] == "remote"} {
        return
     }
 }
index 41981ee66232bd454f59e8c94bcd85c77242ebdc..5fb7cf67023252d27a1aed15e11b9f6665857d3d 100644 (file)
@@ -39,7 +39,7 @@ clean_restart ${binfile}
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 3e16255b4c9f1e2a60e1edf1bcb16af619a8f17c..56d26ccd13211eda2f101bba7245a11abf5266c7 100644 (file)
@@ -28,7 +28,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index c22dea400c0f368493ba5aa1fea6ded59ed31e5e..973e46e366df254521935ab4f9fce9f8b7d52caa 100644 (file)
@@ -40,7 +40,7 @@ if { [prepare_for_testing "failed to prepare" $testfile $srcfile] } {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 60b9de47e52bb8f24e19d133b7a6d887865ae7d0..13479ab3ca85be2355fe9261fbba93050771e8c6 100644 (file)
@@ -1105,7 +1105,7 @@ gdb_test "p ctable1\[120\]" "120 'x'" "p ctable1\[120\] #1"
 
 gdb_load ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 8a3545299c9d3aafe841dc8fd1b9fb2702b3e774..9d5d40a8a42412feeb71b9483f09b9342f06ffdc 100644 (file)
@@ -25,7 +25,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 0fee624d1dbdde869641317d3e4a1b6ca01abb52..be190e2ae712bd5d4e51dabf5d58cd5138e98380 100644 (file)
@@ -56,16 +56,16 @@ gdb_test "ptype struct t_struct" "type = struct t_struct \{.*\[\r\n\]    (unsign
 
 # Test the equivalence between '.' and '->' for struct member references.
 
-if [gdb_test "ptype v_struct1.v_float_member"  "type = float"]<0 then {
+if {[gdb_test "ptype v_struct1.v_float_member" "type = float"] < 0} {
     return -1
 }
-if [gdb_test "ptype v_struct1->v_float_member" "type = float"]<0 then {
+if {[gdb_test "ptype v_struct1->v_float_member" "type = float"] < 0} {
     return -1
 }
-if [gdb_test "ptype v_t_struct_p.v_float_member"       "type = float"]<0 then {
+if {[gdb_test "ptype v_t_struct_p.v_float_member" "type = float"] < 0} {
     return -1
 }
-if [gdb_test "ptype v_t_struct_p->v_float_member"      "type = float"]<0 then {
+if {[gdb_test "ptype v_t_struct_p->v_float_member" "type = float"] < 0} {
     return -1
 }
 
@@ -346,7 +346,7 @@ gdb_test "ptype v_int" "type = int.*" "ptype int"
 #}
 #
 
-if {!$gcc_compiled} then { setup_xfail "rs6000-*-*" "i*86-*-sysv4*" }
+if {!$gcc_compiled} { setup_xfail "rs6000-*-*" "i*86-*-sysv4*" }
 gdb_test "ptype t_char_array" "type = (|unsigned )char \\\[0?\\\]"
 
 gdb_test "ptype pv_char_array" "type = (|unsigned )char \\(\\*\\)\\\[0?\\\]"
@@ -584,7 +584,7 @@ gdb_test "ptype foo" "type = char" "ptype foo typedef after second list of charf
 # requires a running process.  These call malloc, and can take a long
 # time to execute over a slow serial link, so increase the timeout.
 
-if [runto_main] then {
+if {[runto_main]} {
 
   if [target_info exists gdb,cannot_call_functions] {
     unsupported "this target can not call functions"
index 00e4af14a9a2018029d43977ed3e16652f4c06d3..78fd7b5fbc2060de6847d3988296c71a1e0d88a8 100644 (file)
@@ -101,7 +101,7 @@ proc quit_with_live_inferior {appear_how extra_inferior quit_how} {
     }
 
     if {$appear_how == "run"} {
-       if ![runto_main] then {
+       if {![runto_main]} {
            return
        }
     } elseif {$appear_how == "attach" || $appear_how == "attach-nofile"} {
index 50191de9a550c99b85f9d1c0aef7d65103a87895..1c4cf3074ef1a49f006d7fa2a70cc391eee8fbdf 100644 (file)
@@ -37,7 +37,7 @@ proc test_input_radix { iradix iradixhex iradixoctal } {
        "initialize radix, input radix $iradix"
     gdb_test "set input-radix $iradix" \
        "Input radix now set to decimal $iradix, hex $iradixhex, octal $iradixoctal."
-    if { $iradix == 10 } then {
+    if {$iradix == 10} {
        gdb_test "show radix" \
            "Input and output radices set to decimal 10, hex a, octal 12." \
            "show radix, input radix $iradix"
@@ -110,7 +110,7 @@ proc test_output_radix { oradix oradixhex oradixoctal } {
        "initialize radix, output radix $oradix"
     gdb_test "set output-radix $oradix" \
        "Output radix now set to decimal $oradix, hex $oradixhex, octal $oradixoctal."
-    if { $oradix == 10 } then {
+    if {$oradix == 10} {
        gdb_test "show radix" \
            "Input and output radices set to decimal 10, hex a, octal 12." \
            "show radix, output radix $oradix"
index 3dfecdd021fd650e6ec5c47c861c34477a0e1e66..f475633df3cf6a50d57e4dfa0d18eb8f0bc8245a 100644 (file)
@@ -34,7 +34,7 @@ save_vars { GDBFLAGS } {
     }
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 681cdb52a8436f22c57abd0a28fee2f6caee8bc8..3505f440b0687223b51bc9d5d905f70116817734 100644 (file)
@@ -38,7 +38,7 @@ proc recurse_tests {} {
         gdb_test_no_output "set can-use-hw-watchpoints 0" ""
     }
 
-    if [runto recurse] then {
+    if {[runto recurse]} {
        # First we need to step over the assignment of b, so it has a known
        # value.
        gdb_test "next" "if \\(a == 1\\)" "next over b = 0 in first instance"
index f3fc7d7fe0e3ddfd0002b7f8828e86b27da0b6c8..db48705000b5bb3eca88ceaf1ee44ed8bc52dd77 100644 (file)
@@ -23,7 +23,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 80185052f205a1ff4110633675b0c704c98c2855..98f43791d5cd701d82f4e1a281b8341daef63dca 100644 (file)
@@ -33,7 +33,7 @@ if { [prepare_for_testing "failed to prepare" relational int-type.c {debug nowar
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 579dcd40e5c18e02814c0ef24948c3e6d0d88a41..4c2bc251d470a1b312b8a476c0338d6bceb07b2f 100644 (file)
@@ -22,7 +22,7 @@ if {[target_info gdb_protocol] != "remote"
 standard_testfile .c
 
 set result [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}]
-if {$result != "" } then {
+if {$result != ""} {
     untested "failed to compile"
     return -1
 }
@@ -87,7 +87,7 @@ proc gdb_load_timed {executable class writesize} {
     global test gdb_prompt
     set test "timed download `[file tail $executable]' - $class, $writesize"
 
-    if {$writesize != ""} then {
+    if {$writesize != ""} {
        gdb_test_no_output "set remote memory-write-packet-size $writesize" \
            "$test - set packet size"
 
@@ -121,7 +121,7 @@ proc gdb_load_timed {executable class writesize} {
     set load_begin_time [clock clicks]
     set result [gdb_load $executable]
     set load_end_time [clock clicks]
-    if { $result != 0 } then {
+    if {$result != 0} {
        fail "$test - loading executable"
        return
     }
@@ -158,7 +158,7 @@ set sizeof_random_data [get_sizeof "random_data" 48*1024]
 #
 # Part THREE: Check the upload behavour
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -171,7 +171,7 @@ gdb_test "x/8ub random_data" \
 gdb_test "x/8ub random_data + 400 - 4" \
        "<random_data\\+396>:\[ \t\]+185\[ \t\]+255\[ \t\]+50\[ \t\]+140\[ \t\]+237\[ \t\]+172\[ \t\]+143\[ \t\]+93"
 
-if {$sizeof_random_data > 16380 } then {
+if {$sizeof_random_data > 16380} {
     gdb_test "x/8ub random_data + 16384 - 4" \
        "<random_data\\+16380>:\[ \t\]+178\[ \t\]+180\[ \t\]+135\[ \t\]+93\[ \t\]+70\[ \t\]+62\[ \t\]+205\[ \t\]+76"
 }
index e3e47026fd1221a6b3e15e228fb58c19b3dfd461..82a18fa82394e070f2b9b8750681daaa8ae3e656 100644 (file)
@@ -25,7 +25,7 @@ proc return_tests { } {
     global gdb_prompt skip_float_test
 
 
-    if { ! [ runto func1 ] } then { return 0 }
+    if {![runto func1]} { return 0 }
 
     gdb_test_multiple "return" "simple return" {
        -re "Make .* return now.*y or n. $" {
index b5eb2da44067040fd12708a8b5234420077d64f8..50c67e51a5cfdec458f926ac46e97eb161a412d8 100644 (file)
@@ -81,7 +81,7 @@ proc return_void { } {
 proc return2_tests { } {
     global gdb_prompt skip_float_test
 
-    if { ! [ runto_main ] } then {
+    if {![runto_main]} {
        return -1
     }
 
@@ -90,12 +90,12 @@ proc return2_tests { } {
     return_1 "short"
     return_1 "int"
     return_1 "long"
-    if { ! [istarget "m6811-*-*"] && ![istarget "h8300*-*"] } then {
+    if {![istarget "m6811-*-*"] && ![istarget "h8300*-*"]} {
         return_1 "long_long"
     }
     if {!$skip_float_test} {
        return_1 "float"
-       if { ! [istarget "m6811-*-*"] } then {
+       if {![istarget "m6811-*-*"]} {
            return_1 "double"
        }
     }
index 8cd7138426109e0fea6718d1eafaa0a27b800a07..b092cb9a49d3a4218b1cf608ffa9e231f4633be1 100644 (file)
@@ -41,7 +41,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 clean_restart ${binfile}
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 1854496289304c26f48671f09e233dc996b12fe3..e807a1cdd96223b591c0075b35aa9c5d42b125ae 100644 (file)
@@ -52,40 +52,40 @@ proc_with_prefix test_at_main {} {
     # Print scope0.c::filelocal_ro, which is 201
 
     # No clue why the rs6000 fails this test.
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print filelocal_ro" "\\\$$decimal = 201" "print filelocal_ro in test_at_main"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope0.c'::filelocal_ro" "\\\$$decimal = 201" "print 'scope0.c'::filelocal_ro"
 
     # Print scope1.c::filelocal, which is 2
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal" "\\\$$decimal = 2" "print 'scope1.c'::filelocal"
 
     # Print scope1.c::filelocal_bss, which is 102
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal_bss" "\\\$$decimal = 102" "print 'scope1.c'::filelocal_bss"
 
     # Print scope1.c::filelocal_ro, which is 202
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal_ro" "\\\$$decimal = 202" "print 'scope1.c'::filelocal_ro"
 
     # Print scope1.c::foo::funclocal, which is 3
     gdb_test "print foo::funclocal" "\\\$$decimal = 3"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal" "\\\$$decimal = 3" "print 'scope1.c'::foo::funclocal"
 
     # Print scope1.c::foo::funclocal_ro, which is 203
     gdb_test "print foo::funclocal_ro" "\\\$$decimal = 203"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal_ro" "\\\$$decimal = 203" "print 'scope1.c'::foo::funclocal_ro"
 
     # Print scope1.c::bar::funclocal, which is 4
     gdb_test "print bar::funclocal" "\\\$$decimal = 4"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::bar::funclocal" "\\\$$decimal = 4" "print 'scope1.c'::bar::funclocal"
 }
 
@@ -105,27 +105,27 @@ proc_with_prefix test_at_foo {} {
     gdb_test "print 'scope0.c'::filelocal_bss" "\\\$$decimal = 101" "print 'scope0.c'::filelocal_bss in test_at_foo"
 
     # Print scope0.c::filelocal_ro, which is 201
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope0.c'::filelocal_ro" "\\\$$decimal = 201" "print 'scope0.c'::filelocal_ro"
 
 
     # Print scope1.c::filelocal, which is 2
     gdb_test "print filelocal" "\\\$$decimal = 2" "print filelocal at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal" "\\\$$decimal = 2" "print 'scope1.c'::filelocal at foo"
 
     gdb_test "print filelocal_bss" "\\\$$decimal = 102" \
        "print filelocal_bss at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal_bss" "\\\$$decimal = 102" "print 'scope1.c'::filelocal_bss at foo"
 
 
     gdb_test "print filelocal_ro" "\\\$$decimal = 202" \
        "print filelocal_ro at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal_ro" "\\\$$decimal = 202" "print 'scope1.c'::filelocal_ro at foo"
 
 
@@ -136,7 +136,7 @@ proc_with_prefix test_at_foo {} {
     gdb_test "print foo::funclocal" "\\\$$decimal = 3" \
        "print foo::funclocal at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal" "\\\$$decimal = 3" "print 'scope1.c'::foo::funclocal at foo"
 
 
@@ -148,7 +148,7 @@ proc_with_prefix test_at_foo {} {
     gdb_test "print foo::funclocal_bss" "\\\$$decimal = 103" \
        "print foo::funclocal_bss at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal_bss" "\\\$$decimal = 103" "print 'scope1.c'::foo::funclocal_bss at foo"
 
 
@@ -160,7 +160,7 @@ proc_with_prefix test_at_foo {} {
     gdb_test "print foo::funclocal_ro" "\\\$$decimal = 203" \
        "print foo::funclocal_ro at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal_ro" "\\\$$decimal = 203" "print 'scope1.c'::foo::funclocal_ro at foo"
 
 
@@ -169,7 +169,7 @@ proc_with_prefix test_at_foo {} {
     gdb_test "print bar::funclocal" "\\\$$decimal = 4" \
        "print bar::funclocal at foo"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::bar::funclocal" "\\\$$decimal = 4" "print 'scope1.c'::bar::funclocal at foo"
 
 }
@@ -190,57 +190,57 @@ proc_with_prefix test_at_bar {} {
     gdb_test "print 'scope0.c'::filelocal_bss" "\\\$$decimal = 101" "print 'scope0.c'::filelocal_bss in test_at_bar"
 
     # Print scope0.c::filelocal_ro, which is 201
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope0.c'::filelocal_ro" "\\\$$decimal = 201" "print 'scope0.c'::filelocal_ro at bar"
 
     # Print scope1.c::filelocal, which is 2
     gdb_test "print filelocal" "\\\$$decimal = 2" "print filelocal at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal" "\\\$$decimal = 2" "print 'scope1.c'::filelocal at bar"
 
     # Print scope1.c::filelocal_bss, which is 102
     gdb_test "print filelocal_bss" "\\\$$decimal = 102" "print filelocal_bss at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal_bss" "\\\$$decimal = 102" "print 'scope1.c'::filelocal_bss at bar"
 
     # Print scope1.c::filelocal_ro, which is 202
     gdb_test "print filelocal_ro" "\\\$$decimal = 202" "print filelocal_ro in test_at_bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::filelocal_ro" "\\\$$decimal = 202" "print 'scope1.c'::filelocal_ro at bar"
 
     # Print scope1.c::foo::funclocal, which is 3
     gdb_test "print foo::funclocal" "\\\$$decimal = 3" "print foo::funclocal at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal" "\\\$$decimal = 3" "print 'scope1.c'::foo::funclocal at bar"
 
     # Print scope1.c::foo::funclocal_bss, which is 103
     gdb_test "print foo::funclocal_bss" "\\\$$decimal = 103" "print foo::funclocal_bss at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal_bss" "\\\$$decimal = 103" "print 'scope1.c'::foo::funclocal_bss at bar"
 
     # Print scope1.c::foo::funclocal_ro, which is 203
     gdb_test "print foo::funclocal_ro" "\\\$$decimal = 203" "print foo::funclocal_ro at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::foo::funclocal_ro" "\\\$$decimal = 203" "print 'scope1.c'::foo::funclocal_ro at bar"
 
     # Print scope1.c::bar::funclocal, which is 4
     gdb_test "print funclocal" "\\\$$decimal = 4" "print funclocal at bar"
     gdb_test "print bar::funclocal" "\\\$$decimal = 4" "print bar::funclocal at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::bar::funclocal" "\\\$$decimal = 4" "print 'scope1.c'::bar::funclocal at bar"
 
     # Print scope1.c::bar::funclocal_bss, which is 104
     gdb_test "print funclocal_bss" "\\\$$decimal = 104" "print funclocal_bss at bar"
     gdb_test "print bar::funclocal_bss" "\\\$$decimal = 104" "print bar::funclocal_bss at bar"
 
-    if { [test_compiler_info gcc-*-*] } then { setup_xfail "rs6000-*-*" }
+    if {[test_compiler_info gcc-*-*]} { setup_xfail "rs6000-*-*" }
     gdb_test "print 'scope1.c'::bar::funclocal_bss" "\\\$$decimal = 104" "print 'scope1.c'::bar::funclocal_bss at bar"
 }
 
@@ -347,8 +347,8 @@ gdb_test "print 'scope0.c'::filelocal_bss" "= 0" \
 gdb_test "print 'scope0.c'::filelocal" "= 1" \
     "print 'scope0.c'::filelocal before run"
 
-if [runto_main] then { test_at_main }
-if [runto foo] then { test_at_foo }
-if [runto bar] then { test_at_bar }
-if [runto localscopes] then { test_at_localscopes }
-if [runto autovars] then { test_at_autovars }
+if {[runto_main]} { test_at_main }
+if {[runto foo]} { test_at_foo }
+if {[runto bar]} { test_at_bar }
+if {[runto localscopes]} { test_at_localscopes }
+if {[runto autovars]} { test_at_autovars }
index 8e778eec716da96d71de11132844efa18a018773..c0e2cd003733f2dbcf821363671caa7f9e650cb1 100644 (file)
@@ -22,7 +22,7 @@ if { [prepare_for_testing "failed to prepare" "sect-cmd" \
 set srcfile break.c
 set srcfile1 break1.c
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index 20a7f346994da5673c18c321dece2b76624b18a4..75811ff93bde1aa52c46ff6106265e80244a463d 100644 (file)
@@ -62,7 +62,7 @@ remote_exec build "rm -rf [file dirname $new_name]"
 remote_exec build "mkdir [file dirname $new_name]"
 remote_exec build "ln -s ${binfile}${EXEEXT} $new_name"
 clean_restart ${testfile}${EXEEXT}
-if { $gdb_file_cmd_debug_info != "debug" } then {
+if {$gdb_file_cmd_debug_info != "debug"} {
     fail "no debug information found."
 }
 
@@ -70,7 +70,7 @@ if { $gdb_file_cmd_debug_info != "debug" } then {
 set subdir ${old_subdir}
 
 clean_restart ${testfile}${EXEEXT}
-if { $gdb_file_cmd_debug_info != "debug" } then {
+if {$gdb_file_cmd_debug_info != "debug"} {
     fail "no debug information found."
 }
 
@@ -278,7 +278,7 @@ gdb_test "info break" "Num     Type.*Disp Enb Address.*What.*\[\r\n\]
 # inappropriately.  (There are no calls to those system functions
 # in this test program.)
 #
-if ![runto_main] then { 
+if {![runto_main]} {
     return
 }
 
@@ -329,7 +329,7 @@ gdb_test "break" \
 # Verify that a "silent" breakpoint can be set, and that GDB is indeed
 # "silent" about its triggering.
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -451,7 +451,7 @@ gdb_test_multiple "finish" "finish from called function" {
 # Verify that GDB responds gracefully to a "finish" command with
 # arguments.
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -669,7 +669,7 @@ if {[build_executable sepdebug.exp sepdebug2 sepdebug2.c debug] != -1
 # NT_GNU_BUILD_ID / .note.gnu.build-id test:
 
 set build_id_debug_filename [build_id_debug_filename_get $binfile$EXEEXT]
-if ![string compare $build_id_debug_filename ""] then {
+if {![string compare $build_id_debug_filename ""]} {
     unsupported "build-id is not supported by the compiler"
 
     # Spare debug files may confuse testsuite runs in the future.
@@ -678,10 +678,10 @@ if ![string compare $build_id_debug_filename ""] then {
     set build_id_debugself_filename [build_id_debug_filename_get $debugfile]
     set test "build-id support by binutils"
     set xfail 0
-    if ![string compare $build_id_debugself_filename ""] then {
+    if {![string compare $build_id_debugself_filename ""]} {
        unsupported $test
        set xfail 1
-    } elseif {[string compare $build_id_debugself_filename $build_id_debug_filename] != 0} then {
+    } elseif {[string compare $build_id_debugself_filename $build_id_debug_filename] != 0} {
        fail $test
     } else {
        pass $test
index c3f2a0cce8905912bc07fdffee82d00f5cc32860..f91ca5d417b2af4c44b5ad284ac83e59b9b26fb1 100644 (file)
@@ -18,7 +18,7 @@ if { [prepare_for_testing "failed to prepare" "set-noassign" start.c {debug nowa
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index c90d13292dfa540cc41059a3d8e8d31591fecc25..26fad28d89aa509b9e24eef6321b332adfcf502a 100644 (file)
@@ -31,7 +31,7 @@ proc_with_prefix test_setshow_annotate {} {
     # Start with a fresh gdb
     clean_restart $::binfile
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -80,7 +80,7 @@ proc_with_prefix test_setshow_annotate {} {
 proc_with_prefix test_setshow_args {} {
     clean_restart $::binfile
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
index 3d6c5700a490b92290f710c3f302e0acfa3a1ec5..673df0a188bc3e857733c9cf823bb9609128624f 100644 (file)
@@ -35,7 +35,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 #
 gdb_test_no_output "set print sevenbit-strings" 
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
@@ -389,7 +389,7 @@ set timeout $prev_timeout
 # Test printing of enumeration bitfields.
 # GNU C supports them, some other compilers don't.
 
-if {[is_c_compiler_gcc]} then {
+if {[is_c_compiler_gcc]} {
     gdb_test "print sef.field=sm1" ".*.\[0-9\]* = sm1"
     gdb_test "print sef.field" ".*.\[0-9\]* = sm1" "print sef.field (sm1)"
     gdb_test "print sef.field=s1" ".*.\[0-9\]* = s1"
index c692570967601d307f72695d0895ce6b0ded7cd7..5db6b128536e4db8c10e757f965a51d3de3186a3 100644 (file)
@@ -61,7 +61,7 @@ gdb_load_shlib $lib1
 gdb_load_shlib $lib2
 
 
-if ![runto_main] then {
+if {![runto_main]} {
         perror "C function calling tests suppressed"
 }
 
index 25032400ee611d07c181f5b365150b20ba9751f1..ad10d5f34972007730c212a4191c92536b281389 100644 (file)
@@ -72,7 +72,7 @@ gdb_load_shlib $lib1_sl
 gdb_load_shlib $lib2_sl
 
 # Load up the shared objects
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index f2c99c8a2a6bf36b37c522a5b895e242a215f3a0..923499bffc5a2fdb25e31c83cb5cd32c4b74cb9b 100644 (file)
@@ -37,7 +37,7 @@ proc test_one_sig {nextsig} {
 
     set need_another_continue 1
     set missed_handler 0
-    if $this_sig_supported then {
+    if {$this_sig_supported} {
        set esig $thissig
 
        if { $thissig == "IO" } {
@@ -45,7 +45,7 @@ proc test_one_sig {nextsig} {
        }
        # On Linux SPARC64 systems SIGLOST==SIGPWR and gdb identifies
        # the raised signal as PWR.
-       if { $thissig == "LOST" && [ istarget "sparc64-*-linux*" ] } then {
+       if {$thissig == "LOST" && [istarget "sparc64-*-linux*"]} {
            set esig "PWR"
        }
 
@@ -54,7 +54,7 @@ proc test_one_sig {nextsig} {
                "get signal $esig"
     }
 
-    if $need_another_continue then {
+    if {$need_another_continue} {
        if { $thissig == "URG" } {
            setup_xfail "i*86-pc-linuxoldld-gnu" "i*86-pc-linuxaout-gnu"
        }
@@ -73,7 +73,7 @@ proc test_one_sig {nextsig} {
        }
     }
 
-    if { $missed_handler == "0" } then {
+    if {$missed_handler == "0"} {
        gdb_test_multiple "signal 0" "advance to $nextsig" {
            -re "Breakpoint.*gen_$nextsig.*\r\n\[0-9\]+\[ \t\]+kill \\(.*\r\n$gdb_prompt $" {
                pass "advance to $nextsig"
index 6bddc0749a930b00b828db54a3bf04612bb55a16..e0d351348d7c5560670b1af80b9e8b723c6890c7 100644 (file)
@@ -44,7 +44,7 @@ gdb_test "handle SIGVTALRM print pass nostop"
 gdb_test "handle SIGPROF print pass nostop"
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index eb44533eacd7abe5b7ace146724590808d14c84f..0fa63f166f6ed6f6f25b88a0adabcbbe68292241 100644 (file)
@@ -44,7 +44,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 961bc54bcb1faadd9fb7ac519f95fe6f2dbd385b..8d6c6418cbf4d90f2b88935ace6e10b340fba260 100644 (file)
@@ -34,7 +34,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 gdb_test "display/i \$pc"
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 221f0e9a2944c1f527ea818e5452e9dc898376f1..5efff68d72140e24fa4e40b901d6c5bd831929a6 100644 (file)
@@ -39,7 +39,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 }
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
@@ -99,7 +99,7 @@ with_test_prefix "validate siginfo fields" {
 # the changed value.
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 8ced509cd5f5040369dfb87a5b27020eb4e4bb99..abd328a2a785e4e21090b1d13055c8cebb93cbc9 100644 (file)
@@ -36,7 +36,7 @@ if  {[gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" \
 clean_restart $binfile
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 2a1ceb89bdce833260154ff8d9c0b527551db428..075b9979e8b0f8fb55cc55fd4440b98506c17035 100644 (file)
@@ -38,7 +38,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 gdb_test "display/i \$pc"
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index e1d7f3a844c17329bef8f6ae24d4e47dc2cd9cc5..53ec5e36e00138814123d4a1d6a0c0261e7939cf 100644 (file)
@@ -81,7 +81,7 @@ remote_close target
 
 clean_restart $binfile
 
-if { ! [ runto_main ] } then {
+if {![runto_main]} {
     return -1
 }
 
index 2b3fc1c66629fe34f46f5ff1e32f0fba9fc65bc4..997024842e5bd66aa6cffbfa58f25ee3e68a4fa5 100644 (file)
@@ -63,7 +63,7 @@ test_handle_all_print
 
 clean_restart $binfile
 
-if [runto_main] then {
+if {[runto_main]} {
 
     # Since count is a static variable outside main, runto_main is no
     # guarantee that count will be 0 at this point.
index ccbe7700d292e635126bc68a4cd290530b42d06b..14302d2a567fefafaf6a5360c3e5a10927fc22b2 100644 (file)
@@ -26,7 +26,7 @@ if [prepare_for_testing "failed to prepare" ${testfile} ${srcfile} {debug}] {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return -1
 }
 
index e7ce39d3669399096fcb79b88006472c2a736a88..7a33b61868c412a8aa5d341f90dfbe2672ee37f3 100644 (file)
@@ -46,7 +46,7 @@ clean_restart ${binfile}
 # Run to `main' where we begin our tests.
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index ab1ce9beb269f5c8f9df4f5ce637d6dc29702e9e..0bcd46b5d790bb1bbaf5b3f80c4e0893ff8092c1 100644 (file)
@@ -36,7 +36,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {deb
 clean_restart ${binfile}
 
 # Advance to main
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 3237c212a0a7b24b217ae5671b5f25475221d1fe..7cdbace56c51e2a93ff6935549d96654e3a424c3 100644 (file)
@@ -29,7 +29,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 3c5d1070a5b8f5c7c1f35a593f89a45f013f492e..1822c03138cd3d6a054cef727ca6bdc0747c3e6e 100644 (file)
@@ -40,7 +40,7 @@ gdb_load_shlib $lib_sl
 
 # This program implicitly loads SOM shared libraries.
 #
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index 8f686ce6e89fa149b84eed41a55f5a6657642764..2433425bf00e95110f64c7c57ad691cdea32224b 100644 (file)
@@ -51,7 +51,7 @@ gdb_reinitialize_dir $srcdir/$subdir
 gdb_load ${binfile}
 gdb_load_shlib $libobj
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 4c4da6aee9de53b0eb5a10e44a8e6443a7dcea24..6da4b8d4d4aa7611f7d57387b0983b0ae15f7343 100644 (file)
@@ -85,7 +85,7 @@ foreach libsepdebug {NO IN SEP} { with_test_prefix "$libsepdebug" {
 
     clean_restart $executable
 
-    if ![runto_main] then {
+    if {![runto_main]} {
       return 0
     }
 
index 02b2028ce9225f72225be526f1d1604ceb09951d..ff812a7e87bfc044fd204a46e83ab5e7ee21ab54 100644 (file)
@@ -49,7 +49,7 @@ gdb_test "br foo2" \
 
 delete_breakpoints
 
-if ![runto_main] then {
+if {![runto_main]} {
   return 0
 }
 
index fff1c1f8717d5d85c91d75d59df3f392ab359e06..e0aa7b4f64ccbb43974b412a40d969ca52121718 100644 (file)
@@ -119,7 +119,7 @@ proc test_truncated_comp_dir {} {
     gdb_test "show directories" \
        "\r\nSource directories searched: \\\$cdir\[:;\]\\\$cwd"
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
index 31aa4832b28e9a194b247e26ef365b6f4adb8676..f9c889b4d2bf1905d9404c2f664694f1263834d0 100644 (file)
@@ -23,7 +23,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 2e479d6ebe201fe30c25f41a621999972f04be5e..600dc52bbf595e9dba2efb2eb50c8866976fe3ac 100644 (file)
@@ -24,7 +24,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 9f85302f13c3f94c17d9cf4016fb3bb0a2e20cd0..ce6d2d0cca7cc41c9c247adf8be1d7a1e137b4d1 100644 (file)
@@ -23,7 +23,7 @@ if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [con
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index e63b55729eb4fcc9bcabcdd39c2dfeac9e633290..782d12a03feff123637371e1098578fca67f32a0 100644 (file)
@@ -44,7 +44,7 @@ proc simple_func_break_test { protection } {
        return -1
     }
 
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return -1
     }
 
index 61675e598f2a6b4051bd510e07548be7e2500792..a5c96f33b990b9bb39e742591e4dc96bd64e95cf 100644 (file)
@@ -21,7 +21,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
      return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index 807475a29b60a8829b4263a07fd7f47f143e6595..cbd73cfb463045ec44f96b86ab12b9ed46e5af28 100644 (file)
@@ -27,7 +27,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
index 530765c3dabda8480f8bb5de91d95658b9bc56a5..4192fa108263900501b8fbf5d7d383409bff7685 100644 (file)
@@ -115,7 +115,7 @@ proc setup { syscall } {
 
     clean_restart $testfile
 
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return -1
     }
 
@@ -386,7 +386,7 @@ step_over_syscall "clone"
 
 set testfile "step-over-fork"
 clean_restart $testfile
-if { ![runto_main] } then {
+if {![runto_main]} {
     return -1
 }
 
index e73539b22ec3365e20caae71e73c67a304044229..8dfbe86c79ded76cbe4db1b6a4516cda5287ea1d 100644 (file)
@@ -26,7 +26,7 @@ if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debu
 
 clean_restart ${binfile}
 
-if ![runto_main] then {
+if {![runto_main]} {
    return 0
 }
 
@@ -143,7 +143,7 @@ gdb_test_multiple "finish" "$test" {
     }
     -re ".*(Program received|$inferior_exited_re).*$gdb_prompt $" {
        # Oops... We ran to the end of the program...  Better reset     
-       if {![runto_main]} then {
+       if {![runto_main]} {
            return 0
        }
        if {![runto step-test.c:45]} {
index 65847c7fbd597e897af4c236adaf1daa40bda7eb..6fc22f27c55aee89e1d4db47c26c2760ef0164bb 100644 (file)
@@ -26,7 +26,7 @@ if { [prepare_for_testing "failed to prepare" $executable $srcfile] } {
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index 71c07ebe3680a9effb72382801a78b6918009623..ed10d24ba57654bd6ef6425224706d776ef13172 100644 (file)
@@ -70,7 +70,7 @@ proc start_structs_test { types } {
     gdb_test_no_output "set print elements 300"
 
     # Advance to main
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return
     }
 
index eecc19bba3e54133e173f12efc01ec93f00187a0..eb4a89b477fe2602cd42e5b605283ad10a6e016a 100644 (file)
@@ -19,7 +19,7 @@ if { [prepare_for_testing "failed to prepare" $binfile $srcfile {debug}] } {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 66273f19e9c0f494b596bf338b14e340dafe3cb3..c73facb16582a44373eaa5fa71824f220c79266f 100644 (file)
@@ -71,7 +71,7 @@ if {[prepare_for_testing "failed to prepare"  $binfile "$srcfile $srcfile2" $exe
 
 gdb_load_shlib ${lib_so}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -82,7 +82,7 @@ gdb_continue_to_breakpoint gdb_add_symbol_file
 
 # 2) Set a pending breakpoint at bar in $srcfile3.
 set result [gdb_breakpoint bar allow-pending]
-if {!$result} then {
+if {!$result} {
    return
 }
 
@@ -93,7 +93,7 @@ set result [gdb_test "add-symbol-file ${lib_syms} addr" \
                     "add symbol table from file \".*${lib_basename}\\.so\"\
  at.*\\(y or n\\) " \
                     "y"]
-if {$result != 0} then {
+if {$result != 0} {
    return
 }
 
@@ -109,7 +109,7 @@ gdb_continue_to_breakpoint bar ".*${lib_basename}\\.c:$lnum_bar.*"
 
 # 6) Set a breakpoint at foo in $srcfile3.
 set result [gdb_breakpoint foo]
-if {!$result} then {
+if {!$result} {
     return
 }
 
@@ -121,7 +121,7 @@ gdb_continue_to_breakpoint foo ".*${lib_basename}\\.c:$lnum_foo.*"
 # 8) Set a breakpoint at gdb_remove_symbol_file in $srcfile for
 #    removing the library's symbols.
 set result [gdb_breakpoint gdb_remove_symbol_file]
-if {!$result} then {
+if {!$result} {
     return
 }
 
@@ -135,7 +135,7 @@ set result [gdb_test "remove-symbol-file -a addr" \
                     "Remove symbol table from file \".*${lib_basename}\\.so\"\\?\
 .*\\(y or n\\) " \
                     "y"]
-if {$result != 0} then {
+if {$result != 0} {
     return
 }
 
index 9cc26bb66e54f6dafe6d692c94c77942d7a1d1b8..ac0aba986c0dbc66d63e82df763d80cb70e852e6 100644 (file)
@@ -23,7 +23,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} [list $srcfile $srcfil
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index 21a0c1a979df3004c5e86db592ddb87d86085c41..e9310045ac3c72e7f4c28181b00e7c8e12b891b9 100644 (file)
@@ -25,7 +25,7 @@ gdb_test "info terminal" \
     "No saved terminal information.*" \
     "test info terminal pre-execution"
 
-if ![runto break_here] then {
+if {![runto break_here]} {
     return 0
 }
 
index c1296992229192c3957d033e5189cc04280f83e8..f7671e1a2ae9e77fd408850dae828594e13dcb85 100644 (file)
@@ -34,7 +34,7 @@ if  { [gdb_compile $local_srcfile "${binfile}" executable $options] != "" } {
 
 clean_restart $binfile
 
-if [runto_main] then {
+if {[runto_main]} {
     # Test that GDB can still detect whether we have line numbers
     # even if we're executing code in an include file.
 
index 6370c99565780037307f79952c07fc43f721d4b9..35ccfb7ec68118f2e8bae4f24a450697023b1255 100644 (file)
@@ -37,7 +37,7 @@ if { [gdb_compile_shlib $libsrc $libobj {debug}] != ""
 clean_restart ${binfile}
 gdb_load_shlib ${libobj}
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 40945e3374f6df2d9c2f7ea57c7edb7af124f7b7..87ebe39c895e80890d1897132684deca14d126f1 100644 (file)
@@ -25,7 +25,7 @@ set bp_location19 [gdb_get_line_number "set breakpoint 19 here"]
 set bp_location20 [gdb_get_line_number "set breakpoint 20 here"]
 set bp_location21 [gdb_get_line_number "set breakpoint 21 here"]
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index d8863ad28955323fac7ac5f4b1551f231390c249..58fb1267aa471c724ccf0b9df2c58062f75df772 100644 (file)
@@ -31,7 +31,7 @@ if {[prepare_for_testing_full "failed to prepare" \
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 7fa464f09ade71a55b198c49e718a8db80f9edbe..a7807ea469d6343fdd0d58d77958faf957173383 100644 (file)
@@ -58,7 +58,7 @@ if [gdb_skip_stdio_test "varargs.exp"] {
     return
 }
 
-if ![runto test] then {
+if {![runto test]} {
     perror "couldn't run to breakpoint"
     return
 }
index 4c56eb40fe97cf6f6eb2baf05a4854d0085bd8b8..7fda22647c39a575232bb02907f005324aec5c11 100644 (file)
@@ -42,7 +42,7 @@ if { [prepare_for_testing "failed to prepare" volatile constvars.c] } {
 #
 # set it up at a breakpoint so we can play with the variable values
 #
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
@@ -52,7 +52,7 @@ get_debug_format
 # Many tests xfail with gcc 2 -gstabs+.
 
 proc local_compiler_xfail_check { } {
-    if { [test_compiler_info gcc-2-*] } then {
+    if {[test_compiler_info gcc-2-*]} {
        if { ![test_debug_format "HP"] \
                && ![test_debug_format "DWARF \[0-9\]"] } then {
            setup_xfail "*-*-*" 
index 5dc0ebc2bb4ad266fe08bc05ec7ce7ed469bb742..026f12b2d83e13925eeac8bb8bc1fb4d9f4e279d 100644 (file)
@@ -41,7 +41,7 @@ proc test_watchpoint { hw } {
 
     clean_restart ${testfile}
 
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return
     }
 
index 18fabfc69c95990b28f973acf5188bf22d7e80bf..379fdc71ce40f573f120ccc737a3d2d1c843a26f 100644 (file)
@@ -36,7 +36,7 @@ if {$skip_hw_watchpoint_tests_p} {
     gdb_test_no_output "set can-use-hw-watchpoints 0" ""
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -55,7 +55,7 @@ if {$skip_hw_watchpoint_tests_p} {
     gdb_test_no_output "set can-use-hw-watchpoints 0" ""
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
@@ -74,7 +74,7 @@ if {$skip_hw_watchpoint_tests_p} {
     gdb_test_no_output "set can-use-hw-watchpoints 0" ""
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index 61497f87eaf716d5031a0167e1c7097b0293edcc..02d87ed99441fcaa7c0bc37a427b5c27184de60c 100644 (file)
@@ -23,7 +23,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return
 }
 
index 4b8c87f193d721414cea0885f1c78b16506bf678..d73bc5c4ab348073c7a50d0c697b696dc6d1dd0a 100644 (file)
@@ -32,7 +32,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
     return -1
 }
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return
 }
 
index 0ba5af021a840ec1903de342fd22ac3c96f7ae7e..2771e4efddbd51c720d442241b1493ae1fc08034 100644 (file)
@@ -27,7 +27,7 @@ proc test_watchpoint_across_vfork { hw teststr } {
 
     clean_restart ${testfile}
 
-    if { ![runto_main] } then {
+    if {![runto_main]} {
        return
     }
 
index 7d4f19ddc0614b375a470920c6eb59f8d66b8d27..da921a72346a69e4f20349b5040e60231a3a4629 100644 (file)
@@ -35,7 +35,7 @@ if {[gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" "${binfile}" executab
 
 clean_restart ${binfile}
 
-if { ![runto_main] } then {
+if {![runto_main]} {
    return
 }
 
index 7ea6a7467c9bf45e3a132579b9b82432d31f13fd..431f78856d878e88eb510f2bb45932b07cbb73e5 100644 (file)
@@ -38,7 +38,7 @@ if {[prepare_for_testing "failed to prepare" $testfile $srcfile debug]} {
     return -1
 }
 
-if ![runto_main] then {
+if {![runto_main]} {
     return 0
 }
 
index 6c445d4cbd8e26bf44e54f704995c94e6c4526de..0364591d745a04609293f0f5696af76388dcfa52 100644 (file)
@@ -79,7 +79,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
     return -1
 }
 
-if { ![runto_main] } then {
+if {![runto_main]} {
     return
 }
 
@@ -133,7 +133,7 @@ proc test {always_inserted} {
 
        clean_restart $binfile
 
-       if { ![runto_main] } then {
+       if {![runto_main]} {
            return
        }
 
index a8c98578cb9ba3d838f7383ff3b4ba7747182240..7c379cb316f3c8b8363f74a84ca359ea49c1d002 100644 (file)
@@ -272,7 +272,7 @@ proc test_disabling_watchpoints {} {
 proc test_stepping {} {
     global gdb_prompt
 
-    if [runto marker1] then {
+    if {[runto marker1]} {
        gdb_test "watch ival2" ".*\[Ww\]atchpoint \[0-9\]*: ival2"
 
        # Well, let's not be too mundane.  It should be a *bit* of a challenge
@@ -351,7 +351,7 @@ proc test_watchpoint_triggered_in_syscall {} {
     set x 0
     set y 0
     set testname "watch buffer passed to read syscall"
-    if [runto marker2] then {
+    if {[runto marker2]} {
        gdb_test "watch buf\[0\]" ".*\[Ww\]atchpoint \[0-9\]*: buf\\\[0\\\]"
        gdb_test "watch buf\[1\]" ".*\[Ww\]atchpoint \[0-9\]*: buf\\\[1\\\]"
        gdb_test "watch buf\[2\]" ".*\[Ww\]atchpoint \[0-9\]*: buf\\\[2\\\]"
@@ -409,7 +409,7 @@ proc test_watchpoint_triggered_in_syscall {} {
        }
 
        # Did we find what we were looking for?  If not, flunk it.
-       if [expr $x==$y] then { pass $testname } else { fail "$testname (only triggered $x watchpoints, expected $y)"}
+       if {[expr $x==$y]} { pass $testname } else { fail "$testname (only triggered $x watchpoints, expected $y)"}
 
        # Continue until we hit the finishing marker function.
        # Make sure we hit no more watchpoints.
@@ -431,7 +431,7 @@ proc test_watchpoint_triggered_in_syscall {} {
 proc test_complex_watchpoint {} {
     global gdb_prompt
 
-    if [runto marker4] then {
+    if {[runto marker4]} {
        gdb_test "watch ptr1->val" ".*\[Ww\]atchpoint \[0-9\]*: ptr1->val"
        gdb_test "break marker5" ".*Breakpoint.*"
 
@@ -559,7 +559,7 @@ proc test_watchpoint_and_breakpoint {} {
     # This is a test for PR breakpoints/7143, which involves setting a
     # watchpoint right after you've reached a breakpoint.
 
-    if [runto func3] then {
+    if {[runto func3]} {
        gdb_breakpoint [gdb_get_line_number "second x assignment"]
        gdb_continue_to_breakpoint "second x assignment"
        gdb_test "watch x" ".*atchpoint \[0-9\]+: x"
@@ -730,7 +730,7 @@ proc test_inaccessible_watchpoint {} {
     # This is a test for watchpoints on currently inaccessible (but later
     # valid) memory.
 
-    if [runto func4] then {
+    if {[runto func4]} {
        # Make sure we only allow memory access errors.
        set msg "watchpoint refused to insert on nonexistent struct member"
        gdb_test_multiple "watch struct1.nosuchmember" $msg {
@@ -811,7 +811,7 @@ proc test_no_hw_watchpoints {} {
     # (This proves rather little on kernels that don't support
     # fast watchpoints, but still...)
     #
-    if ![runto_main] then {
+    if {![runto_main]} {
        return
     }
 
@@ -936,7 +936,7 @@ proc do_tests {} {
            "disable fast watches, 1"
     }
 
-    if [initialize] then {
+    if {[initialize]} {
 
        test_simple_watchpoint
 
@@ -960,7 +960,7 @@ proc do_tests {} {
     # elsewhere.
     # On sparc-sun-sunos4.1.3, GDB was running all the way to the marker4 
     # breakpoint before stopping for the watchpoint.  I don't know why.
-    if {[istarget "hppa*-*-*"]} then {
+    if {[istarget "hppa*-*-*"]} {
        test_watchpoint_triggered_in_syscall
     }
 
index f6dce17238ce1b7f1fea21ca8722e25ff0562513..d42a41aa398ad1f79c186d7f7eed139a0023ca52 100644 (file)
@@ -20,7 +20,7 @@ if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } {
 }
 
 set bp_location [gdb_get_line_number "START"]
-if ![runto "wchar.c:$bp_location" ] then {
+if {![runto "wchar.c:$bp_location"]} {
   return -1
 }
 
index 655fbb8e8fa9b23e073d155d7f2543e6a5fbe1af..132eee7f5a187caa9c5374de14204032fc233549 100644 (file)
@@ -37,7 +37,7 @@ if { [prepare_for_testing "failed to prepare" whatis-exp int-type.c {debug nowar
 # set it up at a breakpoint so we can play with the variable values
 #
 
-if ![runto_main] then {
+if {![runto_main]} {
     perror "couldn't run to breakpoint"
     return
 }
index be76183ca79e7db269ae0cb26a1fc192fe8af424..615b159a69590ae3dcb3ab0a42041fa384f33f0f 100644 (file)
@@ -48,7 +48,7 @@ proc prepare {lang} {
        return 0
     }
 
-    if ![runto_main] then {
+    if {![runto_main]} {
        return 0
     }
 
@@ -301,7 +301,7 @@ proc run_tests {lang} {
 }
 
 foreach_with_prefix lang {"c" "c++"} {
-    if { [prepare $lang] } then {
+    if {[prepare $lang]} {
        run_tests $lang
     }
 }
index 5a3ff536bd981504f16172d370e64eddbcf6e873..c455c675051e5e8a38e53d3112b96d47250ab144 100644 (file)
@@ -100,14 +100,14 @@ proc do_test {dir options} {
 
     setup_xfail_on_long_vs_int
     # AIX xlc gets this wrong and unsigned long right.  Go figure.
-    if {!$gcc_compiled} then {setup_xfail "rs6000-*-aix*"}
+    if {!$gcc_compiled} {setup_xfail "rs6000-*-aix*"}
     gdb_test "whatis v_long" \
        "type = (long|long int)" \
        "whatis long"
 
     setup_xfail_on_long_vs_int
     # AIX xlc gets this wrong and unsigned long right.  Go figure.
-    if {!$gcc_compiled} then {setup_xfail "rs6000-*-aix*"}
+    if {!$gcc_compiled} {setup_xfail "rs6000-*-aix*"}
     gdb_test "whatis v_signed_long" \
        "type = (signed |)(long|long int)" \
        "whatis signed long"
@@ -475,7 +475,7 @@ proc do_test {dir options} {
        }
 
        # Sun /bin/cc calls this a function returning double.
-       if {!$gcc_compiled} then {setup_xfail "*-sun-sunos4*"}
+       if {!$gcc_compiled} {setup_xfail "*-sun-sunos4*"}
        gdb_test "whatis v_float_func" \
            "type = float \\($void\\)" \
            "whatis float function"