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'.
This commit is contained in:
parent
ca20d46296
commit
f05e4c1115
2 changed files with 6 additions and 13 deletions
|
@ -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'.
|
||||
|
|
13
gdb/infrun.c
13
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
|
||||
|
|
Loading…
Reference in a new issue