gdb:

	Fix PR gdb/14617.
	* breakpoint.c (trace_pass_set_count): Call
	observer_notify_breakpoint_modified instead of
	observer_notify_tracepoint_modified.
	* mi/mi-cmd-break.c (mi_cmd_break_passcount): Likewise.

gdb/doc:

	* observer.texi (GDB Observers): Remove observer
	'tracepoint_modified', 'tracepoint_created' and
	'tracepoint_deleted'.

gdb/testsuite:

	Fix PR gdb/14617.
	* gdb.mi/mi-breakpoint-changed.exp (test_insert_delete_modify):
	Remove setup_kfail, and update test.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0fc9a86816cac934a884e933e9f09e958a502173..f7808a4c9bb0c8495fba554d19ced7d45296f662 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2012-11-03  Yao Qi  <yao@codesourcery.com>
+
+       Fix PR gdb/14617.
+       * breakpoint.c (trace_pass_set_count): Call
+       observer_notify_breakpoint_modified instead of
+       observer_notify_tracepoint_modified.
+       * mi/mi-cmd-break.c (mi_cmd_break_passcount): Likewise.
+
 2012-11-02  Tom Tromey  <tromey@redhat.com>
 
        * breakpoint.c (catch_syscall_completer): Pass 'word' as second

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index 0023ba5a374a8e899ab33f83693972c0367760d2..851dd8624a4e405e36b46bffb09e21fafe718e10 100644 (file)
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -15245,7 +15245,7 @@ static void
 trace_pass_set_count (struct tracepoint *tp, int count, int from_tty)
 {
   tp->pass_count = count;
-  observer_notify_tracepoint_modified (tp->base.number);
+  observer_notify_breakpoint_modified (&tp->base);
   if (from_tty)
     printf_filtered (_("Setting tracepoint %d's passcount to %d\n"),
                     tp->base.number, count);

diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog
index 82b1543b1856c3236694ddda4c9d3c5607893daa..a717c5ad1744d25cb5e6ba22fcbf7c3ffcab7059 100644 (file)
--- a/gdb/doc/ChangeLog
+++ b/gdb/doc/ChangeLog
@@ -1,3 +1,9 @@
+2012-11-03  Yao Qi  <yao@codesourcery.com>
+
+       * observer.texi (GDB Observers): Remove observer
+       'tracepoint_modified', 'tracepoint_created' and
+       'tracepoint_deleted'.
+
 2012-10-26  Tom Tromey  <tromey@redhat.com>
 
        * gdb.texinfo (General Query Packets): Use @itemx for QAgent.

diff --git a/gdb/doc/observer.texi b/gdb/doc/observer.texi
index 106475bd1be7b292d0b91b3d22ee32f3321d7cf7..50038ac36cfa13f0f1a0472270c1efee9d8d7769 100644 (file)
--- a/gdb/doc/observer.texi
+++ b/gdb/doc/observer.texi
@@ -179,21 +179,6 @@ A breakpoint has been modified in some way.  The argument @var{b}
 is the modified breakpoint.
 @end deftypefun
 
-@deftypefun void tracepoint_created (int @var{tpnum})
-A new tracepoint has been created.  The argument @var{tpnum} is the
-number of the newly-created tracepoint.
-@end deftypefun
-
-@deftypefun void tracepoint_deleted (int @var{tpnum})
-A tracepoint has been destroyed.  The argument @var{tpnum} is the
-number of the newly-destroyed tracepoint.
-@end deftypefun
-
-@deftypefun void tracepoint_modified (int @var{tpnum})
-A tracepoint has been modified in some way.  The argument @var{tpnum}
-is the number of the modified tracepoint.
-@end deftypefun
-
 @deftypefun void traceframe_changed (int @var{tfnum}, int @var{tpnum})
 The trace frame is changed to @var{tfnum} (e.g., by using the
 @code{tfind} command).  If @var{tfnum} is negative, it means

diff --git a/gdb/mi/mi-cmd-break.c b/gdb/mi/mi-cmd-break.c
index 2fe84dcfeccee0f7bfe302c69862065e2a6777ca..487d42d5472c70163ca6d9ab14515db835aa24da 100644 (file)
--- a/gdb/mi/mi-cmd-break.c
+++ b/gdb/mi/mi-cmd-break.c
@@ -200,7 +200,7 @@ mi_cmd_break_passcount (char *command, char **argv, int argc)
   if (t)
     {
       t->pass_count = p;
-      observer_notify_tracepoint_modified (n);
+      observer_notify_breakpoint_modified (&t->base);
     }
   else
     {

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index e3376d131665573406a5b32ed2849a2500871e42..ad0407dd3d612faf884f486146c4ff41cfc7fbf7 100644 (file)
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2012-11-03  Yao Qi  <yao@codesourcery.com>
+
+       Fix PR gdb/14617.
+       * gdb.mi/mi-breakpoint-changed.exp (test_insert_delete_modify):
+       Remove setup_kfail, and update test.
+
 2012-11-02  Tom Tromey  <tromey@redhat.com>
 
        * gdb.base/catch-syscall.exp (do_syscall_tests): Add completion

diff --git a/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp b/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp
index 773f588d5cda48d1b3b67db0a1b0cecf8b9ca835..ec10032eda9755c7b882668f53eb429106f4a211 100644 (file)
--- a/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp
+++ b/gdb/testsuite/gdb.mi/mi-breakpoint-changed.exp
@@ -120,10 +120,9 @@ proc test_insert_delete_modify { } {
        {.*=breakpoint-modified,bkpt=\{number="5",.*,ignore=\"1\".*\}.*\n\^done} \
        $test
     # 5. when modifying pass count.
-    setup_kfail gdb/14617  *-*-*
     set test "passcount 1 4"
     mi_gdb_test $test \
-       {.*=breakpoint-modified,bkpt=\{number="4",.*\}.*\n\^done} \
+       {.*=breakpoint-modified,bkpt=\{number="4",.*pass="1".*\}.*\n\^done} \
        $test
 
     # Delete some breakpoints and verify that '=breakpoint-deleted