Don't handle timeout inside gdb_test_multiple
authorLuis Machado <lgustavo@codesourcery.com>
Fri, 2 Dec 2016 04:07:02 +0000 (22:07 -0600)
committerLuis Machado <lgustavo@codesourcery.com>
Fri, 2 Dec 2016 04:07:02 +0000 (22:07 -0600)
This fixes a few cases where the testcase is explicitly handling timeouts
inside gdb_test_multiple when it is not necessary.

It also converts two gdb_test_multiple calls to gdb_test_no_output calls
(also removing the timeout handling).

gdb/testsuite/ChangeLog:

2016-12-01  Luis Machado  <lgustavo@codesourcery.com>

* gdb.base/maint.exp: Remove timeout handling for gdb_test_multiple.
* gdb.cp/gdb2495.exp: Likewise and convert gdb_test_multiple into
gdb_test_no_output for a couple of cases.
* gdb.cp/ovldbreak.exp: Remove timeout handling for gdb_test_multiple.

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/maint.exp
gdb/testsuite/gdb.cp/gdb2495.exp
gdb/testsuite/gdb.cp/ovldbreak.exp

index 4060ea6409295e3fbffc9d705ac696542b373a6e..0ec07ae3af32a15c955bee09840c5e9ffe52b699 100644 (file)
@@ -1,3 +1,10 @@
+2016-12-01  Luis Machado  <lgustavo@codesourcery.com>
+
+       * gdb.base/maint.exp: Remove timeout handling for gdb_test_multiple.
+       * gdb.cp/gdb2495.exp: Likewise and convert gdb_test_multiple into
+       gdb_test_no_output for a couple of cases.
+       * gdb.cp/ovldbreak.exp: Remove timeout handling for gdb_test_multiple.
+
 2016-12-01  Luis Machado  <lgustavo@codesourcery.com>
 
        Fix test names and standardize compilation error messages throughout
index b07b370663ae62adeb2b4b1a206561091805edd6..17c606b7b1a2f25dd5dcf24e722487bd44865534 100644 (file)
@@ -297,9 +297,6 @@ gdb_test_multiple "maint print msymbols msymbols_output2 ${testfile}" "maint pri
                    -re ".*$gdb_prompt $" {
                        fail "maint print msymbols"
                    }
-                   timeout {
-                       fail "(timeout) maint print msymbols"
-                   }
                }
                gdb_test "shell rm -f msymbols_output2" ".*" \
                    "shell rm -f msymbols_output2"
@@ -307,17 +304,11 @@ gdb_test_multiple "maint print msymbols msymbols_output2 ${testfile}" "maint pri
            -re ".*$gdb_prompt $" {
                fail "maint print msymbols"
            }
-           timeout {
-               fail "(timeout) maint print msymbols"
-           }
        }
     }
     -re ".*$gdb_prompt $" {
        fail "maint print msymbols"
     }
-    timeout {
-       fail "(timeout) maint print msymbols"
-    }
 }
 gdb_test "cd ${mydir}" \
     "Working directory [string_to_regexp ${mydir}]\..*" \
index b4304fb7f1e37163941e2b088ea2e6107d5cea26..ac7438283475d0137f0374582ea14e4266abffb4 100644 (file)
@@ -117,10 +117,7 @@ if ![runto_main] then {
 # behaviour; it should not.  Test both on and off states.
 
 # Turn on unwind on signal behaviour.
-gdb_test_multiple "set unwindonsignal on" "turn unwindonsignal on" {
-    -re "$gdb_prompt $" {pass "set unwindonsignal on"}
-    timeout {fail "(timeout) set unwindonsignal on"}
-}
+gdb_test_no_output "set unwindonsignal on"
 
 # Check that it is turned on.
 gdb_test "show unwindonsignal" \
@@ -133,10 +130,7 @@ gdb_test "p exceptions.raise_signal(1)" \
     "To change this behavior use \"set unwindonsignal off\".*"
 
 # And reverse - turn off again.
-gdb_test_multiple "set unwindonsignal off" "turn unwindonsignal off" {
-    -re "$gdb_prompt $" {pass "set unwindonsignal off"}
-    timeout {fail "(timeout) set unwindonsignal off"}
-}
+gdb_test_no_output "set unwindonsignal off"
 
 # Check that it is actually turned off.
 gdb_test "show unwindonsignal" \
index 96d3bd3d2abe4bd1fbe8a28e8ce73e6a9c4ad2cb..b6c5efd7cc2a07161aaf8470b43b42f898726f3d 100644 (file)
@@ -58,10 +58,6 @@ proc take_gdb_out_of_choice_menu {} {
     gdb_test_multiple " " " " {
         -re ".*$gdb_prompt $" {
         }
-        timeout {
-            perror "could not resynchronize to command prompt (timeout)"
-            continue
-        }
     }
 }