[RFA/commit] thread-specific breakpoints not saved properly by save-breakpoint

[Joel Brobecker <brobecker at adacore dot com>]

Initially, I noticed that the save command was often missing
new lines in the file that it generated.  For instance, consider:

        % gdb save-bp
        (gdb) b break_me
        (gdb) b save-bp.c:27
        (gdb) save breakpoints bps

The contents of the bps file would be:

        % cat bps
        break break_mebreak save-bp.c:27

Looking further into the problem, I realized that the missing newlines
are just a consequence of a missing call to print_recreate_thread.
After having generated the breakpoint location in the break command,
we cannot put a new line until we have looked at whether we need to
add a 'thread NUM' argument.

I was quite surprised to see that we actually do not have a testcase
for the "save breakpoint" command??? So I added one.

gdb/ChangeLog:

        * breakpoint.c (bkpt_print_recreate): Add call to
        print_recreate_thread.

gdb/testsuite/ChangeLog:

        * gdb.base/save.bp.exp, gdb.base/save-bp.c: New files.

Tested on x86_64-linux.  The new testcase fails if the patch is
not applied.

I will commit in a few days if there are no objections.

---
 gdb/breakpoint.c                   |    1 +
 gdb/testsuite/gdb.base/save-bp.c   |   36 ++++++++++++++++
 gdb/testsuite/gdb.base/save-bp.exp |   80 ++++++++++++++++++++++++++++++++++++
 3 files changed, 117 insertions(+), 0 deletions(-)
 create mode 100644 gdb/testsuite/gdb.base/save-bp.c
 create mode 100644 gdb/testsuite/gdb.base/save-bp.exp

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index f995b14..08ff69b 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -11047,6 +11047,7 @@ bkpt_print_recreate (struct breakpoint *tp, struct ui_file *fp)
 		    _("unhandled breakpoint type %d"), (int) tp->type);
 
   fprintf_unfiltered (fp, " %s", tp->addr_string);
+  print_recreate_thread (tp, fp);
 }
 
 /* Virtual table for internal breakpoints.  */
diff --git a/gdb/testsuite/gdb.base/save-bp.c b/gdb/testsuite/gdb.base/save-bp.c
new file mode 100644
index 0000000..3469124
--- /dev/null
+++ b/gdb/testsuite/gdb.base/save-bp.c
@@ -0,0 +1,36 @@
+/* Copyright 2011 Free Software Foundation, Inc.
+
+   This file is part of GDB.
+
+   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
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+void
+break_me (void)
+{
+}
+
+int
+main (void)
+{
+  int i;
+  break_me (); /* BREAK HERE.  */
+  break_me (); /* Try a thread-specific breakpoint.  */
+
+  for (i = 0; i < 5; i++)
+    break_me (); /* Try a condition-specific breakpoint.  */
+
+  break_me (); /* Finally, try a breakpoint with commands.  */
+  return 0;
+}
+
diff --git a/gdb/testsuite/gdb.base/save-bp.exp b/gdb/testsuite/gdb.base/save-bp.exp
new file mode 100644
index 0000000..d64a0ce
--- /dev/null
+++ b/gdb/testsuite/gdb.base/save-bp.exp
@@ -0,0 +1,80 @@
+# Copyright (C) 2011 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
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+set testfile save-bp
+set srcfile ${testfile}.c
+
+if { [prepare_for_testing ${testfile}.exp ${testfile}] } {
+    return -1
+}
+
+if ![runto_main] {
+    untested ${testfile}.exp
+    return -1
+}
+# Delete all breakpoints so that the "runto_main" breakpoint above
+# does not interfere with our testing.
+delete_breakpoints
+
+# Insert a bunch of breakpoints... The goal is to create breakpoints
+# that we are going to try to save in a file and then reload.  So
+# try to create a good variety of them.
+gdb_breakpoint "break_me"
+
+gdb_breakpoint ${srcfile}:[gdb_get_line_number "BREAK HERE"]
+
+set loc [gdb_get_line_number "thread-specific"]
+gdb_breakpoint "${srcfile}:${loc} thread 1"
+
+set loc [gdb_get_line_number "condition"]
+gdb_breakpoint "${srcfile}:${loc} if i == 1"
+
+gdb_breakpoint ${srcfile}:[gdb_get_line_number "with commands"]
+gdb_test "commands\nsilent\nend" "End with.*" "add breakpoint commands"
+
+set test_name "info break before saving to file"
+gdb_test_multiple "info break" $test_name {
+    -re "(.*)$gdb_prompt " {
+        set info_break "$expect_out(1,string)"
+        pass $test_name
+    }
+}
+
+# Now, save the breakpoints into a file...
+gdb_test "save breakpoint bps"
+
+# Now start a new debugger session...
+clean_restart $testfile
+if ![runto_main] {
+    fail "cannot restart program"
+    return -1
+}
+# Delete all breakpoints so that the "runto_main" breakpoint above
+# does not interfere with our testing.
+delete_breakpoints
+
+# ... and restore the breakpoints.
+gdb_test "source bps"
+
+# Now, verify that the output of 'info break' is identical.
+set test_name "info break after restoring breakpoints"
+set info_break_pattn [string_to_regexp $info_break]
+gdb_test_multiple "info break" $test_name {
+    -re "${info_break_pattn}$gdb_prompt " {
+        set info_break "$expect_out(1,string)"
+        pass $test_name
+    }
+}
+
-- 
1.7.1