diff options
author | Hsiangkai Wang <hsiangkai@gmail.com> | 2013-05-01 10:23:38 +0800 |
---|---|---|
committer | Freddie Chopin <freddie.chopin@gmail.com> | 2013-05-06 14:44:41 +0000 |
commit | 7eb0e3105908dae36841daf1dca0b1a4be50e4d3 (patch) | |
tree | c5f1d5dae0ccbcea2389d21473412c3a57b4e622 | |
parent | c765ef0ff6706736da4df1e832063b7ffefe984b (diff) |
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 <hsiangkai@gmail.com>
Reviewed-on: http://openocd.zylin.com/1372
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
-rw-r--r-- | src/server/gdb_server.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index e7f54281..cd726580 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); |