* gdb.base/bitfields.exp: Simplify by using delete_breakpoints and

  gdb_test.
* lib/gdb.exp (gdb_test): if match times out, don't call fail if
  message is "".

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 94d3ef02a405b6ad3b7d9d2bdd0ef371b37cf8ec..687231e71fd31ace9cea79a3c46e055c5b8de582 100644 (file)
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,11 @@
+Tue Jul 19 10:26:32 1994  J.T. Conklin  (jtc@phishhead.cygnus.com)
+
+       * gdb.base/bitfields.exp: Simplify by using delete_breakpoints and
+         gdb_test.
+
+       * lib/gdb.exp (gdb_test): if match times out, don't call fail if
+         message is "".
+
 Mon Jul 18 12:18:07 1994  J.T. Conklin  (jtc@phishhead.cygnus.com)
 
        * gdb.base/{a2-run,callfuncs,funcargs,interrupt,mips_pro,nodebug,

diff --git a/gdb/testsuite/gdb.base/bitfields.exp b/gdb/testsuite/gdb.base/bitfields.exp
index ab92fd1b262d626223873f224cd2b4359c14bc6a..94ef01b984d7f25a52e7f037233021f763efe821 100644 (file)
--- a/gdb/testsuite/gdb.base/bitfields.exp
+++ b/gdb/testsuite/gdb.base/bitfields.exp
@@ -1,4 +1,4 @@
-# Copyright (C) 1992 Free Software Foundation, Inc.
+# Copyright (C) 1992, 1994 Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -47,10 +47,9 @@ proc bitfield_uniqueness {} {
     global srcfile
     global det_file
 
-    send "break break1\n"
-    expect {
-       -re "Break.* at $hex: file .*$srcfile, line $decimal.\r\n$prompt $" {}
-       timeout { fail "setting breakpoint at break1" ; return }
+    if [gdb_test "break break1" "Break.* at $hex: file .*$srcfile, line $decimal" ""] {
+       fail "setting breakpoint at break1"
+       return
     }
 
     send "run\n"
@@ -63,113 +62,84 @@ proc bitfield_uniqueness {} {
        timeout { fail "running to break1" ; return }
     }
 
-    send "print flags\n"
-    expect {
-       -re ".*uc = 1 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (uc)" ; return }
+    if [gdb_test "print flags" ".*uc = 1 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0" ""] {
+       fail "bitfield uniqueness (uc)"
+       return 
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return 
     }
-    send "print flags\n"
     # Note that we check for s1 as either 1 or -1, so that failure to
     # treat it correctly as a signed 1bit field (values 0 or -1) while
     # printing its value does not cause a spurious failure.  We do the
     # signedness preservation test later.
-    expect {
-       -re ".*uc = 0 .*, s1 = (1|-1), u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (s1)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = (1|-1), u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0" ""] {
+       fail "bitfield uniqueness (s1)"
+       return 
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return 
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 1, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (u1)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 1, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0" ""] {
+       fail "bitfield uniqueness (u1)"
+       return
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1" 
+       return 
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 1, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (s2)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 1, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0" ""] {
+       fail "bitfield uniqueness (s2)" 
+       return 
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 1, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (u2)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 1, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 0" ""] {
+       fail "bitfield uniqueness (u2)"
+       return
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 1, u3 = 0, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (s3)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 1, u3 = 0, s9 = 0, u9 = 0, sc = 0" ""] {
+       fail "bitfield uniqueness (s3)"
+       return
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 1, s9 = 0, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (u3)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 1, s9 = 0, u9 = 0, sc = 0" "" ] {
+       fail "bitfield uniqueness (u3)"
+       return
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
        timeout { fail "continuing to break1" ; return }
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 1, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (s9)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 1, u9 = 0, sc = 0" "" ] {
+       fail "bitfield uniqueness (s9)"
+       return
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 1, sc = 0 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (u9)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 1, sc = 0" "" ] {
+       fail "bitfield uniqueness (u9)"
+       return
     }
-
-    send "cont\n"
-    expect {
-       -re "Break.*break1 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break1" ; return }
+    if [gdb_test "cont" "Break.*break1 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break1"
+       return
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 1 .*$prompt $" {}
-       timeout { fail "bitfield uniqueness (sc)" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 0, u2 = 0, s3 = 0, u3 = 0, s9 = 0, u9 = 0, sc = 1" "" ] {
+       fail "bitfield uniqueness (sc)"
+       return
     }
 
     pass "bitfield locating and uniqueness"
@@ -189,22 +159,11 @@ proc bitfield_containment {} {
     global srcfile
     global det_file
 
-    send "delete\n"
-    expect {
-       -re "Delete all breakpoints.*y or n. $" {
-           send "y\n"
-           expect {
-               -re "$prompt $" {}
-               timeout { fail "deleting breakpoints" ; return }
-           }
-       }
-       timeout { fail "deleting breakpoints" ; return }
-    }
+    delete_breakpoints
 
-    send "break break2\n"
-    expect {
-       -re "Break.* at $hex: file .*$srcfile, line $decimal.\r\n$prompt $" {}
-       timeout { fail "setting breakpoint at break2" ; return }
+    if [gdb_test "break break2" "Break.* at $hex: file .*$srcfile, line $decimal" ""] {
+       fail "setting breakpoint at break2"
+       return
     }
 
     send "run\n"
@@ -217,25 +176,22 @@ proc bitfield_containment {} {
        timeout { fail "running to break2" ; return }
     }
 
-    send "print/x flags\n"
-    expect {
-       -re ".*uc = 0xff, s1 = 0x0, u1 = 0x1, s2 = 0x0, u2 = 0x3, s3 = 0x0, u3 = 0x7, s9 = 0x0, u9 = 0x1ff, sc = 0x0.*$prompt $" {}
-       timeout { fail "bitfield containment" ; return }
+    if [gdb_test "print/x flags" ".*uc = 0xff, s1 = 0x0, u1 = 0x1, s2 = 0x0, u2 = 0x3, s3 = 0x0, u3 = 0x7, s9 = 0x0, u9 = 0x1ff, sc = 0x0" ""] {
+       fail "bitfield containment"
+       return
     }
 
-    send "cont\n"
-    expect {
-       -re "Break.*break2 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break2" ; return }
+    if [gdb_test "cont" "Break.*break2 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break2"
+       return
     }
 
     # If program is compiled with Sun CC, then these print out as their
     # actual sizes; if compiled with gcc, they print out as 0xffffffff
     # (which strikes me as bogus, but accept it at least for now).
-    send "print/x flags\n"
-    expect {
-       -re ".*uc = 0x0, s1 = 0x(1|f*), u1 = 0x0, s2 = 0x(3|f*), u2 = 0x0, s3 = 0x(7|f*), u3 = 0x0, s9 = 0x(1ff|f*), u9 = 0x0, sc = 0xff.*$prompt $" {}
-       timeout { fail "bitfield containment" ; return }
+    if [gdb_test "print/x flags" ".*uc = 0x0, s1 = 0x(1|f*), u1 = 0x0, s2 = 0x(3|f*), u2 = 0x0, s3 = 0x(7|f*), u3 = 0x0, s9 = 0x(1ff|f*), u9 = 0x0, sc = 0xff" ""] {
+       fail "bitfield containment"
+       return
     }
 
     pass "bitfield containment"
@@ -255,22 +211,11 @@ proc bitfield_unsignedness {} {
     global srcfile
     global det_file
 
-    send "delete\n"
-    expect {
-       -re "Delete all breakpoints.*y or n. $" {
-           send "y\n"
-           expect {
-               -re "$prompt $" {}
-               timeout { fail "deleting breakpoints" ; return }
-           }
-       }
-       timeout { fail "deleting breakpoints" ; return }
-    }
+    delete_breakpoints
 
-    send "break break3\n"
-    expect {
-       -re "Break.* at $hex: file .*$srcfile, line $decimal.\r\n$prompt $" {}
-       timeout { fail "setting breakpoint at break3" ; return }
+    if [gdb_test "break break3" "Break.* at $hex: file .*$srcfile, line $decimal" ""] {
+       fail "setting breakpoint at break3"
+       return
     }
 
     send "run\n"
@@ -283,10 +228,9 @@ proc bitfield_unsignedness {} {
        timeout { fail "running to break3" ; return }
     }
 
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 1, s2 = 0, u2 = 3, s3 = 0, u3 = 7, s9 = 0, u9 = 511, sc = 0 .*$prompt $" {}
-       timeout { fail "unsigned bitfield ranges" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 1, s2 = 0, u2 = 3, s3 = 0, u3 = 7, s9 = 0, u9 = 511, sc = 0" "" ] {
+       fail "unsigned bitfield ranges"
+       return
     }
 
     pass "unsigned bitfield ranges and unsigned-ness"
@@ -306,22 +250,11 @@ proc bitfield_signedness {} {
     global srcfile
     global det_file
 
-    send "delete\n"
-    expect {
-       -re "Delete all breakpoints.*y or n. $" {
-           send "y\n"
-           expect {
-               -re "$prompt $" {}
-               timeout { warning "deleting breakpoints (timedout)" ; return }
-           }
-       }
-       timeout { warning "deleting breakpoints (timedout)" ; return }
-    }
+    delete_breakpoints
 
-    send "break break4\n"
-    expect {
-       -re "Break.* at $hex: file .*$srcfile, line $decimal.\r\n$prompt $" {}
-       timeout { fail "setting breakpoint at break4" ; return }
+    if [gdb_test "break break4" "Break.* at $hex: file .*$srcfile, line $decimal" ""] {
+       fail "setting breakpoint at break4"
+       return
     }
 
     send "run\n"
@@ -334,16 +267,14 @@ proc bitfield_signedness {} {
        timeout { fail "running to break4" ; return }
     }
 
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 1, u2 = 0, s3 = 3, u3 = 0, s9 = 255, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "signed bitfields, max positive values" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = 0, u1 = 0, s2 = 1, u2 = 0, s3 = 3, u3 = 0, s9 = 255, u9 = 0, sc = 0" ""] {
+       fail "signed bitfields, max positive values"
+       return
     }
 
-    send "cont\n"
-    expect {
-       -re "Break.*break4 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break4" ; return }
+    if [gdb_test "cont" "Break.*break4 \[)(\]+ at .*$srcfile:$decimal" "" ] {
+       fail "continuing to break4"
+       return
     }
 
     # Determine if the target has signed bitfields so we can xfail the
@@ -352,28 +283,25 @@ proc bitfield_signedness {} {
     expect {
        -re ".* = -256.*$prompt $" {}
        -re ".* = 256.*$prompt $" {setup_xfail "*-*-*"}
-       -re ".*$prompt $" { fail "\
-determining signed-ness of bitfields (known bug in sparc gcc 2.4.5)"
+       -re ".*$prompt $" { fail "determining signed-ness of bitfields (known bug in sparc gcc 2.4.5)"
            return
        }
        default { fail "determining signed-ness of bitfields" ; return }
     }
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = -1, u1 = 0, s2 = -2, u2 = 0, s3 = -4, u3 = 0, s9 = -256, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "signed bitfields, max negative values" ; return }
+
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = -1, u1 = 0, s2 = -2, u2 = 0, s3 = -4, u3 = 0, s9 = -256, u9 = 0, sc = 0" ""] {
+        fail "signed bitfields, max negative values"
+        return
     }
 
-    send "cont\n"
-    expect {
-       -re "Break.*break4 \[)(\]+ at .*$srcfile:$decimal.*$prompt $" {}
-       timeout { fail "continuing to break4" ; return }
+    if [gdb_test "cont" "Break.*break4 \[)(\]+ at .*$srcfile:$decimal" ""] {
+       fail "continuing to break4"
+       return
     }
 
-    send "print flags\n"
-    expect {
-       -re ".*uc = 0 .*, s1 = -1, u1 = 0, s2 = -1, u2 = 0, s3 = -1, u3 = 0, s9 = -1, u9 = 0, sc = 0 .*$prompt $" {}
-       timeout { fail "signed bitfields with -1" ; return }
+    if [gdb_test "print flags" ".*uc = 0 .*, s1 = -1, u1 = 0, s2 = -1, u2 = 0, s3 = -1, u3 = 0, s9 = -1, u9 = 0, sc = 0" ""] {
+       fail "signed bitfields with -1"
+       return
     }
 
     pass "signed bitfield ranges and signed-ness"
@@ -395,11 +323,6 @@ if [istarget "mips-idt-*"] then {
     gdb_reinitialize_dir $srcdir/$subdir
     gdb_load $objdir/$subdir/$binfile
 }
-if [istarget "a29k-*-udi"] then {
-    # FIXME: If PR 2415 is fixed, this is not needed.
-    gdb_target_udi
-    gdb_load $objdir/$subdir/$binfile
-}
 bitfield_containment
 if [istarget "mips-idt-*"] then {
     # Restart because IDT/SIM runs out of file descriptors.
@@ -408,11 +331,6 @@ if [istarget "mips-idt-*"] then {
     gdb_reinitialize_dir $srcdir/$subdir
     gdb_load $objdir/$subdir/$binfile
 }
-if [istarget "a29k-*-udi"] then {
-    # FIXME: If PR 2415 is fixed, this is not needed.
-    gdb_target_udi
-    gdb_load $objdir/$subdir/$binfile
-}
 bitfield_unsignedness
 if [istarget "mips-idt-*"] then {
     # Restart because IDT/SIM runs out of file descriptors.
@@ -421,14 +339,4 @@ if [istarget "mips-idt-*"] then {
     gdb_reinitialize_dir $srcdir/$subdir
     gdb_load $objdir/$subdir/$binfile
 }
-if [istarget "a29k-*-udi"] then {
-    # FIXME: If PR 2415 is fixed, this is not needed.
-    gdb_target_udi
-    gdb_load $objdir/$subdir/$binfile
-}
 bitfield_signedness
-
-if [istarget "a29k-*-udi"] then {
-    # FIXME: If PR 2415 is fixed, this is not needed.
-    gdb_target_udi
-}

diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 9e42606fbd0bbe12202cc4339afd72cdf2d5e984..438eb633dfd9e4233c1b2c19a66a4896b0866ccf 100644 (file)
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -25,7 +25,7 @@
 
 global GDB
 if ![info exists GDB] then {
-    set GDB [findfile $base_dir/../gdb "gdb" [transform gdb ]]
+    set GDB [findfile $base_dir/../gdb $base_dir/../gdb [transform gdb ]]
 }
 
 global GDBFLAGS
@@ -138,9 +138,10 @@ proc runto { function } {
     }
 
     send "break $function\n"
-    # The first regexp is what we get with -g, the second without -g.
+    # The first two regexps are what we get with -g, the third is without -g.
     expect {
-       -re "Break.* at .*: file .*, line $decimal.\r\n$prompt $" {}
+       -re "Breakpoint \[0-9\]* at 0x\[0-9a-f\]*: file .*, line $decimal.\r\n$prompt $" {}
+       -re "Breakpoint \[0-9\]*: file .*, line $decimal.\r\n$prompt $" {}
        -re "Breakpoint \[0-9\]* at 0x\[0-9a-f\]*.*$prompt $" {}
        -re "$prompt $" { fail "setting breakpoint at $function" ; return 0 }
        timeout { fail "setting breakpoint at $function (timeout)" ; return 0 }
@@ -252,7 +253,9 @@ proc gdb_test { args } {
            perror "internal buffer is full."
        }
        timeout {
-           fail "(timeout) $message"
+           if ![string match "" $message] then {
+               fail "(timeout) $message"
+           }
            set result 1
        }
     }
@@ -261,8 +264,10 @@ proc gdb_test { args } {
 
 # Given an input string, adds backslashes as needed to create a
 # regexp that will match the string.
+
 proc string_to_regexp {str} {
-    regsub -all {[]*+.|()^$[]} $str {\\&} result
+    set result $str
+    regsub -all {[]*+.|()^$\[]} $str {\\&} result
     return $result
 }
 
@@ -406,27 +411,22 @@ proc default_gdb_start { } {
     global spawn_id
     global timeout
     verbose "Spawning $GDB $GDBFLAGS"
+
+    if { [which $GDB] == 0 } then {
+       perror "$GDB does not exist."
+       exit 1
+    }
     
     set oldtimeout $timeout
     set timeout [expr "$timeout + 60"]
     if [ llength $GDBFLAGS ] then {
-       if {[which $GDB] != 0} then {
            spawn $GDB $GDBFLAGS
-       } else {
-           perror "$GDB does not exist."
-           exit 1
-       }
     } else {
-       if {[which $GDB] != 0} then {
            spawn $GDB
-       } else {
-           perror "$GDB does not exist."
-           exit 1
-       }
     }
     expect {
        -re ".*\r\n$prompt $" {
-           verbose "GDB initialized for native mode"
+           verbose "GDB initialized."
        }
        -re "$prompt $" {
            perror "GDB never initialized."