diff options
| author | Pedro Alves <palves@redhat.com> | 2013-10-28 16:39:05 +0000 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2013-10-28 16:39:05 +0000 |
| commit | f05e4c1115dec06d408394e2d9a529d1475f96ee (patch) | |
| tree | f071a475c13747c83b4b63e29e66516936f3501d /gdb | |
| parent | ca20d46296a700f493edaa05401b75b1759e78c8 (diff) | |
| download | gdb-f05e4c1115dec06d408394e2d9a529d1475f96ee.zip gdb-f05e4c1115dec06d408394e2d9a529d1475f96ee.tar.gz gdb-f05e4c1115dec06d408394e2d9a529d1475f96ee.tar.bz2 | |
infrun.c:handle_inferior_event: Remove some more dead code.
'ecs' is always memset before being passed to handle_inferior_event.
The stop func is only filled in later in the flow. And since "Remove
dead sets/clears of ecs->random signal", nothing ever sets
ecs->random_signal before this part is reached either.
(Also tested with some added assertions in place.)
gdb/
2013-10-28 Pedro Alves <palves@redhat.com>
* infrun.c (clear_stop_func): Delete.
(handle_inferior_event): Don't call clear_stop_func and don't
clear 'ecs->random_signal'.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rw-r--r-- | gdb/infrun.c | 13 |
2 files changed, 6 insertions, 13 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index b33fc37..1a0328b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2013-10-28 Pedro Alves <palves@redhat.com> + + * infrun.c (clear_stop_func): Delete. + (handle_inferior_event): Don't call clear_stop_func and don't + clear 'ecs->random_signal'. + 2013-10-27 Yao Qi <yao@codesourcery.com> * varobj.c (struct varobj_root) <lang>: Rename to 'lang_ops'. diff --git a/gdb/infrun.c b/gdb/infrun.c index 4f7068c..8b1b668 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -3120,17 +3120,6 @@ handle_syscall_event (struct execution_control_state *ecs) return 1; } -/* Clear the supplied execution_control_state's stop_func_* fields. */ - -static void -clear_stop_func (struct execution_control_state *ecs) -{ - ecs->stop_func_filled_in = 0; - ecs->stop_func_start = 0; - ecs->stop_func_end = 0; - ecs->stop_func_name = NULL; -} - /* Lazily fill in the execution_control_state's stop_func_* fields. */ static void @@ -4089,12 +4078,10 @@ Cannot fill $_exitsignal with the correct signal number.\n")); return; } - clear_stop_func (ecs); ecs->event_thread->stepping_over_breakpoint = 0; bpstat_clear (&ecs->event_thread->control.stop_bpstat); ecs->event_thread->control.stop_step = 0; stop_print_frame = 1; - ecs->random_signal = 0; stopped_by_random_signal = 0; /* Hide inlined functions starting here, unless we just performed stepi or |