diff options
author | Jan Kratochvil <jan.kratochvil@redhat.com> | 2011-08-26 09:28:27 +0000 |
---|---|---|
committer | Jan Kratochvil <jan.kratochvil@redhat.com> | 2011-08-26 09:28:27 +0000 |
commit | e93ca0199f5ea2bec534cf051c2007641f952f82 (patch) | |
tree | f9264079097737f3630b5b9c0f3a16a3af579d84 | |
parent | bdb5404946bc009d72a4e3d3c1ed4c00b8215d89 (diff) | |
download | binutils-e93ca0199f5ea2bec534cf051c2007641f952f82.zip binutils-e93ca0199f5ea2bec534cf051c2007641f952f82.tar.gz binutils-e93ca0199f5ea2bec534cf051c2007641f952f82.tar.bz2 |
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.
-rw-r--r-- | gdb/ChangeLog | 11 | ||||
-rw-r--r-- | gdb/breakpoint.c | 16 | ||||
-rw-r--r-- | gdb/breakpoint.h | 5 | ||||
-rw-r--r-- | gdb/exceptions.c | 11 |
4 files changed, 28 insertions, 15 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1da536e..aad31ec 100644 --- 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 d1d325e..6ccdb6d 100644 --- 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 146ed7e..5e5d1b9 100644 --- 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 559e8ed..e9fdb26 100644 --- 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); |