From 7eb0e3105908dae36841daf1dca0b1a4be50e4d3 Mon Sep 17 00:00:00 2001 From: Hsiangkai Wang Date: Wed, 1 May 2013 10:23:38 +0800 Subject: [PATCH] gdb_server: remove target_handle_event from gdb event callback In target_call_event_callbacks(), it will execute 1. target_handle_event (use Jim_EvalObj() to evaluate event statements in config files) 2. call user registered callbacks Before calling user registered callbacks, target_handle_event has been executed. So, there is no need to call target_handle_event() in gdb event callback. It will execute event statements in config files twice. Change-Id: I84629e324fa3eb909907badf2319b4138ba89f07 Signed-off-by: Hsiangkai Wang Reviewed-on: http://openocd.zylin.com/1372 Tested-by: jenkins Reviewed-by: Spencer Oliver Reviewed-by: Freddie Chopin --- src/server/gdb_server.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index e7f54281f8..cd72658024 100644 --- a/src/server/gdb_server.c +++ b/src/server/gdb_server.c @@ -719,7 +719,6 @@ static int gdb_target_callback_event_handler(struct target *target, int retval; struct connection *connection = priv; - target_handle_event(target, event); switch (event) { case TARGET_EVENT_GDB_HALT: gdb_frontend_halted(target, connection); -- 2.30.2