From f05e4c1115dec06d408394e2d9a529d1475f96ee Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Mon, 28 Oct 2013 16:39:05 +0000 Subject: [PATCH] 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 * infrun.c (clear_stop_func): Delete. (handle_inferior_event): Don't call clear_stop_func and don't clear 'ecs->random_signal'. --- gdb/ChangeLog | 6 ++++++ gdb/infrun.c | 13 ------------- 2 files changed, 6 insertions(+), 13 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index b33fc374a0e..1a0328b5a91 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2013-10-28 Pedro Alves + + * 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 * varobj.c (struct varobj_root) : Rename to 'lang_ops'. diff --git a/gdb/infrun.c b/gdb/infrun.c index 4f7068c8996..8b1b668a41c 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 -- 2.30.2