gdb/

	No functionality change.
	* breakpoint.c (bpstat_clear_actions): Remove the BS parameter, make
	function comment a reference, new variables tp and bs, move here code
	from throw_exception.
	* breakpoint.h (bpstat_clear_actions): Remove the BS parameter,
	describe it in the comment.
	* exceptions.c (throw_exception): Remove variable tp, move the code for
	bpstat_clear_actions to bpstat_clear_actions.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1da536e7fdc3a2dcdb116f60c4f1e2f06e24c905..aad31ec95ff5c7ef2407e4ce250ceee3f679b1c9 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,14 @@
+2011-08-26  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       No functionality change.
+       * breakpoint.c (bpstat_clear_actions): Remove the BS parameter, make
+       function comment a reference, new variables tp and bs, move here code
+       from throw_exception.
+       * breakpoint.h (bpstat_clear_actions): Remove the BS parameter,
+       describe it in the comment.
+       * exceptions.c (throw_exception): Remove variable tp, move the code for
+       bpstat_clear_actions to bpstat_clear_actions.
+
 2011-08-24  Luis Machado  <lgustavo@codesourcery.com>
 
        * linux-nat.h (linux_proc_get_tgid): Remove declaration.

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index d1d325ebca182773cc2b1f997cf1506201e1531c..6ccdb6d015f61f71675cb9c777c081ed6bc91b26 100644 (file)
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -3186,12 +3186,22 @@ bpstat_num (bpstat *bsp, int *num)
   return 1;
 }
 
-/* Modify BS so that the actions will not be performed.  */
+/* See breakpoint.h.  */
 
 void
-bpstat_clear_actions (bpstat bs)
+bpstat_clear_actions (void)
 {
-  for (; bs != NULL; bs = bs->next)
+  struct thread_info *tp;
+  bpstat bs;
+
+  if (ptid_equal (inferior_ptid, null_ptid))
+    return;
+
+  tp = find_thread_ptid (inferior_ptid);
+  if (tp == NULL)
+    return;
+
+  for (bs = tp->control.stop_bpstat; bs != NULL; bs = bs->next)
     {
       decref_counted_command_line (&bs->commands);
 

diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
index 146ed7e656042a9f7c9f8da9c03b013120822f71..5e5d1b9028d5fb6a0956a4cd7dc4d89623b2e160 100644 (file)
--- a/gdb/breakpoint.h
+++ b/gdb/breakpoint.h
@@ -895,8 +895,9 @@ extern int bpstat_num (bpstat *, int *);
    command loop).  */
 extern void bpstat_do_actions (void);
 
-/* Modify BS so that the actions will not be performed.  */
-extern void bpstat_clear_actions (bpstat);
+/* Modify all entries of STOP_BPSTAT of INFERIOR_PTID so that the actions will
+   not be performed.  */
+extern void bpstat_clear_actions (void);
 
 /* Implementation:  */
 

diff --git a/gdb/exceptions.c b/gdb/exceptions.c
index 559e8ede32f5d24f83afc9dc3c8a4d59ca85bae6..e9fdb265553c59de09db466f4c82f17978ac32ed 100644 (file)
--- a/gdb/exceptions.c
+++ b/gdb/exceptions.c
@@ -207,21 +207,12 @@ exceptions_state_mc_action_iter_1 (void)
 void
 throw_exception (struct gdb_exception exception)
 {
-  struct thread_info *tp = NULL;
-
   quit_flag = 0;
   immediate_quit = 0;
 
-  if (!ptid_equal (inferior_ptid, null_ptid))
-    tp = find_thread_ptid (inferior_ptid);
-
   /* Perhaps it would be cleaner to do this via the cleanup chain (not sure
      I can think of a reason why that is vital, though).  */
-  if (tp != NULL)
-    {
-      /* Clear queued breakpoint commands.  */
-      bpstat_clear_actions (tp->control.stop_bpstat);
-    }
+  bpstat_clear_actions ();
 
   do_cleanups (ALL_CLEANUPS);