[PATCH] gdb.threads/watchpoint-fork.exp: Use with_test_prefix.

[Pedro Alves <palves at redhat dot com>]

This gets rid of the manual pf_prefix handling from
gdb.threads/watchpoint-fork.exp.

Tested on x86_64 Fedora 16.  Will check in in a bit.

2012-02-21  Pedro Alves  <palves@redhat.com>

	* gdb.threads/watchpoint-fork.exp (test): Use with_test_prefix.
---
 gdb/testsuite/gdb.threads/watchpoint-fork.exp |  217 ++++++++++++-------------
 1 files changed, 103 insertions(+), 114 deletions(-)

diff --git a/gdb/testsuite/gdb.threads/watchpoint-fork.exp b/gdb/testsuite/gdb.threads/watchpoint-fork.exp
index 086c9bd..c1ef063 100644
--- a/gdb/testsuite/gdb.threads/watchpoint-fork.exp
+++ b/gdb/testsuite/gdb.threads/watchpoint-fork.exp
@@ -23,128 +23,117 @@ if [is_remote target] {
 }
 
 proc test {type symbol} { with_test_prefix "$type" {
-    global testfile objdir subdir srcdir gdb_prompt pf_prefix
+    global testfile objdir subdir srcdir gdb_prompt
 
     set srcfile_type ${srcdir}/${subdir}/${testfile}-${type}.c
 
     # no threads
 
-    set prefix_test $pf_prefix
-    append pf_prefix " singlethreaded:"
-
-    set executable ${testfile}-${type}-st
-    set srcfile_main ${srcdir}/${subdir}/${testfile}-st.c
-    if { [gdb_compile "${srcfile_main} ${srcfile_type}" ${objdir}/${subdir}/${executable} executable [list debug additional_flags=-D$symbol]] != "" } {
-	untested ${testfile}.exp
-	set pf_prefix $prefix_test
-	return
-    }
-    clean_restart $executable
-
-    if [target_info exists gdb,no_hardware_watchpoints] {
-	# The software watchpoint functionality is in GDB an unrelated test.
-	gdb_test_no_output "set can-use-hw-watchpoints 0"
-    }
-
-    gdb_test "show detach-on-fork" "Whether gdb will detach the child of a fork is on\\."
-    gdb_test_no_output "set follow-fork-mode $type"
-    gdb_test "show follow-fork-mode" "Debugger response to a program call of fork or vfork is \"$type\"\\."
-    # Testcase uses it for the `follow-fork-mode child' type.
-    gdb_test "handle SIGUSR1 nostop noprint pass" "No\[ \t\]+No\[ \t\]+Yes.*"
-
-    if ![runto_main] {
-	set pf_prefix $prefix_test
-	return
+    with_test_prefix "singlethreaded" {
+	set executable ${testfile}-${type}-st
+	set srcfile_main ${srcdir}/${subdir}/${testfile}-st.c
+	if { [gdb_compile "${srcfile_main} ${srcfile_type}" ${objdir}/${subdir}/${executable} executable [list debug additional_flags=-D$symbol]] != "" } {
+	    untested ${testfile}.exp
+	    return
+	}
+	clean_restart $executable
+
+	if [target_info exists gdb,no_hardware_watchpoints] {
+	    # The software watchpoint functionality is in GDB an unrelated test.
+	    gdb_test_no_output "set can-use-hw-watchpoints 0"
+	}
+
+	gdb_test "show detach-on-fork" "Whether gdb will detach the child of a fork is on\\."
+	gdb_test_no_output "set follow-fork-mode $type"
+	gdb_test "show follow-fork-mode" "Debugger response to a program call of fork or vfork is \"$type\"\\."
+	# Testcase uses it for the `follow-fork-mode child' type.
+	gdb_test "handle SIGUSR1 nostop noprint pass" "No\[ \t\]+No\[ \t\]+Yes.*"
+
+	if ![runto_main] {
+	    return
+	}
+
+	gdb_test "watch var" "atchpoint \[0-9\]+: var" "Set the watchpoint"
+
+	# It is never hit but it should not be left over in the fork()ed-off child.
+	if [skip_hw_breakpoint_tests] {
+	    set hbreak "break"
+	} else {
+	    set hbreak "hbreak"
+	}
+	gdb_test "$hbreak marker"
+
+	gdb_breakpoint "mark_exit"
+
+	gdb_test "continue" \
+	    "reakpoint \[0-9\]+, marker.*" "hardware breakpoints work"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 0.*New value = 1.*forkoff *\\(1\\).*" "watchpoints work"
+	gdb_test "continue" \
+	    "reakpoint \[0-9\]+, marker.*" "breakpoint after the first fork"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 1.*New value = 2.*forkoff *\\(2\\).*" "watchpoint after the first fork"
+	gdb_test "continue" \
+	    "reakpoint \[0-9\]+, marker.*" "breakpoint after the second fork"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 2.*New value = 3.*mark_exit \\(\\);" "watchpoint after the second fork"
+	gdb_test "continue" "Continuing\\..*\r\nBreakpoint \[0-9\]+, mark_exit .*" "finish"
+
+
+	# threads
+
+	if [target_info exists gdb,no_hardware_watchpoints] {
+	    # Watchpoint hits would get detected in unexpected threads.
+	    return
+	}
     }
 
-    gdb_test "watch var" "atchpoint \[0-9\]+: var" "Set the watchpoint"
-
-    # It is never hit but it should not be left over in the fork()ed-off child.
-    if [skip_hw_breakpoint_tests] {
-	set hbreak "break"
-    } else {
-	set hbreak "hbreak"
-    }
-    gdb_test "$hbreak marker"
-
-    gdb_breakpoint "mark_exit"
-
-    gdb_test "continue" \
-	     "reakpoint \[0-9\]+, marker.*" "hardware breakpoints work"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 0.*New value = 1.*forkoff *\\(1\\).*" "watchpoints work"
-    gdb_test "continue" \
-	     "reakpoint \[0-9\]+, marker.*" "breakpoint after the first fork"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 1.*New value = 2.*forkoff *\\(2\\).*" "watchpoint after the first fork"
-    gdb_test "continue" \
-	     "reakpoint \[0-9\]+, marker.*" "breakpoint after the second fork"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 2.*New value = 3.*mark_exit \\(\\);" "watchpoint after the second fork"
-    gdb_test "continue" "Continuing\\..*\r\nBreakpoint \[0-9\]+, mark_exit .*" "finish"
-
-
-    # threads
-
-    if [target_info exists gdb,no_hardware_watchpoints] {
-	# Watchpoint hits would get detected in unexpected threads.
-	set pf_prefix $prefix_test
-	return
+    with_test_prefix "multithreaded" {
+	set executable ${testfile}-${type}-mt
+	set srcfile_main ${srcdir}/${subdir}/${testfile}-mt.c
+	if { [gdb_compile_pthreads "${srcfile_main} ${srcfile_type}" ${objdir}/${subdir}/${executable} executable [list debug "additional_flags=-D$symbol -DTHREAD"]] != "" } {
+	    untested ${testfile}.exp
+	    return
+	}
+	clean_restart $executable
+
+	gdb_test_no_output "set follow-fork-mode $type"
+	# Testcase uses it for the `follow-fork-mode child' type.
+	gdb_test "handle SIGUSR1 nostop noprint pass" "No\[ \t\]+No\[ \t\]+Yes.*"
+
+	if ![runto_main] {
+	    return
+	}
+
+	gdb_test "watch var" "atchpoint \[0-9\]+: var" "Set the watchpoint"
+
+	# It should not be left over in the fork()ed-off child.
+	gdb_test "$hbreak marker" {reakpoint [0-9]+.*}
+
+	gdb_breakpoint "mark_exit"
+
+	gdb_test "continue" \
+	    "reakpoint \[0-9\]+, marker.*" "hardware breakpoints work"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 0.*New value = 1.*validity-first.*" "singlethread watchpoints work"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 1.*New value = 2.*validity-thread-A.*" "multithreaded watchpoints work at A"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 2.*New value = 3.*validity-thread-B.*" "multithreaded watchpoints work at B"
+	gdb_test "continue" \
+	    "reakpoint \[0-9\]+, marker.*" "breakpoint (A) after the first fork"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 3.*New value = 4.*after-fork1-A.*" "watchpoint A after the first fork"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 4.*New value = 5.*after-fork1-B.*" "watchpoint B after the first fork"
+	gdb_test "continue" \
+	    "reakpoint \[0-9\]+, marker.*" "breakpoint (A) after the second fork"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 5.*New value = 6.*after-fork2-A.*" "watchpoint A after the second fork"
+	gdb_test "continue" \
+	    "atchpoint \[0-9\]+: var.*Old value = 6.*New value = 7.*after-fork2-B.*" "watchpoint B after the second fork"
+	gdb_test "continue" "Continuing\\..*\r\nBreakpoint \[0-9\]+, mark_exit .*" "finish"
     }
-
-    set pf_prefix $prefix_test
-    append pf_prefix " multithreaded:"
-
-    set executable ${testfile}-${type}-mt
-    set srcfile_main ${srcdir}/${subdir}/${testfile}-mt.c
-    if { [gdb_compile_pthreads "${srcfile_main} ${srcfile_type}" ${objdir}/${subdir}/${executable} executable [list debug "additional_flags=-D$symbol -DTHREAD"]] != "" } {
-	untested ${testfile}.exp
-	set pf_prefix $prefix_test
-	return
-    }
-    clean_restart $executable
-
-    gdb_test_no_output "set follow-fork-mode $type"
-    # Testcase uses it for the `follow-fork-mode child' type.
-    gdb_test "handle SIGUSR1 nostop noprint pass" "No\[ \t\]+No\[ \t\]+Yes.*"
-
-    if ![runto_main] {
-	set pf_prefix $prefix_test
-	return
-    }
-
-    gdb_test "watch var" "atchpoint \[0-9\]+: var" "Set the watchpoint"
-
-    # It should not be left over in the fork()ed-off child.
-    gdb_test "$hbreak marker" {reakpoint [0-9]+.*}
-
-    gdb_breakpoint "mark_exit"
-
-    gdb_test "continue" \
-	     "reakpoint \[0-9\]+, marker.*" "hardware breakpoints work"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 0.*New value = 1.*validity-first.*" "singlethread watchpoints work"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 1.*New value = 2.*validity-thread-A.*" "multithreaded watchpoints work at A"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 2.*New value = 3.*validity-thread-B.*" "multithreaded watchpoints work at B"
-    gdb_test "continue" \
-	     "reakpoint \[0-9\]+, marker.*" "breakpoint (A) after the first fork"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 3.*New value = 4.*after-fork1-A.*" "watchpoint A after the first fork"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 4.*New value = 5.*after-fork1-B.*" "watchpoint B after the first fork"
-    gdb_test "continue" \
-	     "reakpoint \[0-9\]+, marker.*" "breakpoint (A) after the second fork"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 5.*New value = 6.*after-fork2-A.*" "watchpoint A after the second fork"
-    gdb_test "continue" \
-	     "atchpoint \[0-9\]+: var.*Old value = 6.*New value = 7.*after-fork2-B.*" "watchpoint B after the second fork"
-    gdb_test "continue" "Continuing\\..*\r\nBreakpoint \[0-9\]+, mark_exit .*" "finish"
-
-
-    # cleanup
-    set pf_prefix $prefix_test
 }}
 
 test parent FOLLOW_PARENT