2010-05-28 Michael Snyder <msnyder@msnyder-server.eng.vmware.com>

* gdb.reverse/break-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/break-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/consecutive-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/consecutive-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/finish-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/finish-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/i386-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/i386-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/i386-sse-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/i387-env-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/i387-stack-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/machinestate-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/machinestate.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/sigall-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/sigall-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/solib-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/solib-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/step-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/step-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/until-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/until-reverse.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/watch-precsave.exp:
	Replace uses of send_gdb / gdb_expect.
	* gdb.reverse/watch-reverse.exp:
	Replace uses of send_gdb / gdb_expect.

gdb/testsuite/ChangeLog

@@ -1,3 +1,52 @@
+2010-05-28  Michael Snyder  <msnyder@msnyder-server.eng.vmware.com>
+
+	* gdb.reverse/break-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/break-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/consecutive-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/consecutive-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/finish-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/finish-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/i386-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/i386-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/i386-sse-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/i387-env-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/i387-stack-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/machinestate-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/machinestate.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/sigall-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/sigall-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/solib-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/solib-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/step-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/step-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/until-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/until-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/watch-precsave.exp: 
+	Replace uses of send_gdb / gdb_expect.
+	* gdb.reverse/watch-reverse.exp: 
+	Replace uses of send_gdb / gdb_expect.
+
 2010-05-28  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
 	* gdb.base/sepdebug.exp (info silent break bp_location1): Fix racy

gdb/testsuite/gdb.reverse/break-precsave.exp

@@ -37,8 +37,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 gdb_test "break $end_location" \
@@ -77,8 +76,7 @@ gdb_test_multiple "continue" "go to end of main forward" {
     }
 }
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction reverse" "" "set reverse"
+gdb_test_no_output "set exec-direction reverse" "set reverse"
 
 gdb_continue_to_breakpoint "bar backward"  ".*/$srcfile:$bar_location.*"
 gdb_continue_to_breakpoint "foo backward"  ".*/$srcfile:$foo_location.*"
@@ -92,7 +90,7 @@ gdb_test_multiple "continue" "main backward" {
     }
 }
 
-gdb_test "set exec-direction forward" "" "set forward"
+gdb_test_no_output "set exec-direction forward" "set forward"
 
 gdb_continue_to_breakpoint "foo" ".*/$srcfile:$foo_location.*"
 gdb_continue_to_breakpoint "bar" ".*/$srcfile:$bar_location.*"

gdb/testsuite/gdb.reverse/break-reverse.exp

@@ -36,8 +36,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 gdb_test "break foo" \
@@ -56,8 +55,7 @@ gdb_continue_to_breakpoint "foo" ".*/$srcfile:$foo_location.*"
 gdb_continue_to_breakpoint "bar" ".*/$srcfile:$bar_location.*"
 gdb_continue_to_breakpoint "end" ".*/$srcfile:$end_location.*"
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction reverse" "" "set reverse"
+gdb_test_no_output "set exec-direction reverse" "set reverse"
 
 gdb_continue_to_breakpoint "bar backward"  ".*/$srcfile:$bar_location.*"
 gdb_continue_to_breakpoint "foo backward"  ".*/$srcfile:$foo_location.*"
@@ -71,7 +69,7 @@ gdb_test_multiple "continue" "main backward" {
     }
 }
 
-gdb_test "set exec-direction forward" "" "set forward"
+gdb_test_no_output "set exec-direction forward" "set forward"
 
 gdb_continue_to_breakpoint "foo" ".*/$srcfile:$foo_location.*"
 gdb_continue_to_breakpoint "bar" ".*/$srcfile:$bar_location.*"

gdb/testsuite/gdb.reverse/consecutive-precsave.exp

@@ -32,8 +32,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 set end_location  [gdb_get_line_number "end of main"  ]
@@ -95,9 +94,8 @@ gdb_test_multiple "step" $testmsg {
 ###
 
 # Set reverse execution direction
-# FIXME: command needs to acknowledge, so we can test if it succeeded.
 
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 # Now step backward and hope to hit the first breakpoint.
 

gdb/testsuite/gdb.reverse/consecutive-reverse.exp

@@ -31,8 +31,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 gdb_breakpoint foo
@@ -76,9 +75,8 @@ gdb_test_multiple "step" $testmsg {
 ###
 
 # Set reverse execution direction
-# FIXME: command needs to acknowledge, so we can test if it succeeded.
 
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 # Now step backward and hope to hit the first breakpoint.
 

gdb/testsuite/gdb.reverse/finish-precsave.exp

@@ -31,8 +31,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # Run until end, then save execution log. 
@@ -175,7 +174,7 @@ gdb_test_multiple "finish" "$test_msg" {
 ###
 
 # Now switch to reverse
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 # Test reverse finish from long long func
 

gdb/testsuite/gdb.reverse/finish-reverse.exp

@@ -31,8 +31,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # Test finish from void func
@@ -155,7 +154,7 @@ gdb_test_multiple "finish" "$test_msg" {
 ###
 
 # Now switch to reverse
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 # Test reverse finish from long long func
 

gdb/testsuite/gdb.reverse/i386-precsave.exp

@@ -63,8 +63,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 global hex

gdb/testsuite/gdb.reverse/i386-reverse.exp

@@ -62,8 +62,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 global hex

gdb/testsuite/gdb.reverse/i386-sse-reverse.exp

@@ -63,8 +63,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 global hex

gdb/testsuite/gdb.reverse/i387-env-reverse.exp

@@ -52,17 +52,17 @@ runto main
 # Record to end of main
 
 set location [gdb_get_line_number "END I387-FLOAT-REVERSE"]
-gdb_test "record" "" "Turn on process record"
+gdb_test_no_output "record"  "Turn on process record"
 gdb_test "until $location" ".*/$srcfile:$location.*" \
     "record to end of main"
 
 # Now rewind to beginning so we can begin testing.
 
 set location [gdb_get_line_number "BEGIN I387-FLOAT-REVERSE"]
-gdb_test "set exec-dir reverse" "" "set reverse direction"
+gdb_test_no_output "set exec-dir reverse" "set reverse direction"
 gdb_test "until $location" ".*/$srcfile:$location.*" \
     "rewind to beginning of main"
-gdb_test "set exec-dir forward" "" "set forward direction"
+gdb_test_no_output "set exec-dir forward" "set forward direction"
 
 # Test FPU env particularly ftag and fstatus reigters.
 

gdb/testsuite/gdb.reverse/i387-stack-reverse.exp

@@ -52,17 +52,17 @@ runto main
 # Record to end of main
 
 set location [gdb_get_line_number "END I387-FLOAT-REVERSE"]
-gdb_test "record" "" "Turn on process record"
+gdb_test_no_output "record" "Turn on process record"
 gdb_test "until $location" ".*/$srcfile:$location.*" \
     "record to end of main"
 
 # Now rewind to beginning so we can begin testing.
 
 set location [gdb_get_line_number "BEGIN I387-FLOAT-REVERSE"]
-gdb_test "set exec-dir reverse" "" "set reverse direction"
+gdb_test_no_output "set exec-dir reverse" "set reverse direction"
 gdb_test "until $location" ".*/$srcfile:$location.*" \
     "rewind to beginning of main"
-gdb_test "set exec-dir forward" "" "set forward direction"
+gdb_test_no_output "set exec-dir forward" "set forward direction"
 
 # Test FPU stack. FPU stack includes st0, st1, st2, st3, st4, 
 # st5, st6, st7. We push 8 values to FPU stack in record mode

gdb/testsuite/gdb.reverse/machinestate-precsave.exp

@@ -59,8 +59,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 gdb_test "break $endmain" \

gdb/testsuite/gdb.reverse/machinestate.exp

@@ -58,8 +58,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # Proceed to end of main

gdb/testsuite/gdb.reverse/sigall-precsave.exp

@@ -174,8 +174,7 @@ runto gen_ABRT
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # Run until end, then save execution log. 

gdb/testsuite/gdb.reverse/sigall-reverse.exp

@@ -174,8 +174,7 @@ runto gen_ABRT
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 test_one_sig HUP

gdb/testsuite/gdb.reverse/solib-precsave.exp

@@ -56,8 +56,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 set end_of_main [gdb_get_line_number "end of main" ]

gdb/testsuite/gdb.reverse/solib-reverse.exp

@@ -55,8 +55,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 #

gdb/testsuite/gdb.reverse/step-precsave.exp

@@ -36,8 +36,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 set end_of_main [gdb_get_line_number "end of main" ]
@@ -158,9 +157,8 @@ gdb_test_multiple "stepi" "$test_message" {
 ###
 
 # Set reverse execution direction
-# FIXME: command needs to acknowledge, so we can test if it succeeded.
 
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 # stepi backward thru return and into a function
 
@@ -268,6 +266,6 @@ gdb_test "next" ".*NEXT TEST 1.*" "reverse next test 2"
 
 # Finish test by running forward to the end.
 # FIXME return to this later...
-# gdb_test "set exec-dir forward" "" "set forward execution"
+# gdb_test_no_output "set exec-dir forward" "set forward execution"
 # gdb_continue_to_end "step-reverse.exp"
 

gdb/testsuite/gdb.reverse/step-reverse.exp

@@ -35,8 +35,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # plain vanilla step/next (no count)
@@ -139,9 +138,8 @@ gdb_test_multiple "stepi" "$test_message" {
 ###
 
 # Set reverse execution direction
-# FIXME: command needs to acknowledge, so we can test if it succeeded.
 
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 # stepi backward thru return and into a function
 
@@ -249,6 +247,6 @@ gdb_test "next" ".*NEXT TEST 1.*" "reverse next test 2"
 
 # Finish test by running forward to the end.
 # FIXME return to this later...
-# gdb_test "set exec-dir forward" "" "set forward execution"
+# gdb_test_no_output "set exec-dir forward" "set forward execution"
 # gdb_continue_to_end "step-reverse.exp"
 

gdb/testsuite/gdb.reverse/until-precsave.exp

@@ -40,8 +40,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 set end_of_main [gdb_get_line_number "set breakpoint 10a here" ]
@@ -113,9 +112,8 @@ gdb_test_multiple "until marker3" "$test_msg" {
 ###
 
 # Set reverse execution direction
-# FIXME: command needs to acknowledge, so we can test if it succeeded.
 
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 #
 # We should now be at main, after the return from marker2.

gdb/testsuite/gdb.reverse/until-reverse.exp

@@ -39,8 +39,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # Verify that plain vanilla "until <location>" works.
@@ -94,9 +93,8 @@ gdb_test_multiple "until marker3" "$test_msg" {
 ###
 
 # Set reverse execution direction
-# FIXME: command needs to acknowledge, so we can test if it succeeded.
 
-gdb_test "set exec-dir reverse" "" "set reverse execution"
+gdb_test_no_output "set exec-dir reverse" "set reverse execution"
 
 #
 # We should now be at main, after the return from marker2.

gdb/testsuite/gdb.reverse/watch-precsave.exp

@@ -34,8 +34,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 set end_location  [gdb_get_line_number "end of main"  ]
@@ -110,8 +109,7 @@ gdb_continue_to_breakpoint "marker2" ".*/$srcfile:.*"
 ###
 ###
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction reverse" "" "set reverse"
+gdb_test_no_output "set exec-direction reverse" "set reverse"
 
 # Reverse until the previous change, from 4 to 3
 # Note that this one is outside the loop
@@ -147,8 +145,7 @@ gdb_test "set can-use-hw-watchpoints 1" "" "enable hw watchpoints"
 ###
 ###
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction forward" "" "set forward"
+gdb_test_no_output "set exec-direction forward" "set forward"
 
 # Continue until first change, from -1 to 0
 
@@ -187,8 +184,7 @@ gdb_test "continue" "marker2 .*" "replay forward to marker2"
 ###
 ###
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction reverse" "" "set reverse"
+gdb_test_no_output "set exec-direction reverse" "set reverse"
 
 # Reverse until the previous change, from 4 to 3
 # Note that this one is outside the loop

gdb/testsuite/gdb.reverse/watch-reverse.exp

@@ -34,8 +34,7 @@ runto main
 
 if [target_info exists gdb,use_precord] {
     # Activate process record/replay
-    gdb_test "record" "" "Turn on process record"
-    # FIXME: command ought to acknowledge, so we can test if it succeeded.
+    gdb_test_no_output "record" "Turn on process record"
 }
 
 # Test software watchpoints
@@ -92,8 +91,7 @@ gdb_continue_to_breakpoint "marker2" ".*/$srcfile:.*"
 ###
 ###
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction reverse" "" "set reverse"
+gdb_test_no_output "set exec-direction reverse" "set reverse"
 
 # Reverse until the previous change, from 4 to 3
 # Note that this one is outside the loop
@@ -129,8 +127,7 @@ gdb_test "set can-use-hw-watchpoints 1" "" "enable hw watchpoints"
 ###
 ###
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction forward" "" "set forward"
+gdb_test_no_output "set exec-direction forward" "set forward"
 
 # Continue until first change, from -1 to 0
 
@@ -169,8 +166,7 @@ gdb_test "continue" "marker2 .*" "replay forward to marker2"
 ###
 ###
 
-# FIXME 'set exec-dir' command should give some output so we can test.
-gdb_test "set exec-direction reverse" "" "set reverse"
+gdb_test_no_output "set exec-direction reverse" "set reverse"
 
 # Reverse until the previous change, from 4 to 3
 # Note that this one is outside the loop