[PATCH] gdb.mi/mi-var-display.exp: fix racy tests (PR testsuite/12649)

Marek Polacek mpolacek@redhat.com
Tue Jun 21 13:57:00 GMT 2011


Another one where we need to replace gdb_test_multiple with mi_gdb_test
to prevent failures with read1().  Note that when setting `fp', we need
to s/1/3/ since the index 1 and 2 is used by mi_gdb_test.  Tested with 
both read{,1}.  OK to apply?

2011-06-21  Marek Polacek  <mpolacek@redhat.com>

        * gdb.mi/mi-var-display.exp: Replace gdb_test_multiple with mi_gdb_test.

--- gdb/gdb/testsuite/gdb.mi/mi-var-display.exp.mp      2011-06-21 13:05:15.679575309 +0200
+++ gdb/gdb/testsuite/gdb.mi/mi-var-display.exp 2011-06-21 13:34:06.561961094 +0200
@@ -605,12 +605,8 @@ mi_gdb_test "-var-list-children anone" \
 
 
 # Record fp
-
-gdb_test_multiple "p/x \$fp" "print FP register" {
-    -re ".*($hex).*\\^done\r\n$mi_gdb_prompt$" { 
-       pass "print FP register"
-       set fp $expect_out(1,string) 
-    }
+if ![mi_gdb_test "p/x \$fp" ".*($hex).*\\^done" "Xprint FP register"] {
+       set fp $expect_out(3,string) 
 }
 
 mi_continue_to "incr_a"



More information about the Gdb-patches mailing list