This is an automated email from Gerrit.

Hsiangkai Wang (hsiang...@gmail.com) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/1372

-- gerrit

commit ff9f6ef85ae9fad5d367ff18832909918f85c8b0
Author: Hsiangkai Wang <hsiang...@gmail.com>
Date:   Wed May 1 10:23:38 2013 +0800

    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 <hsiang...@gmail.com>

diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c
index e7f5428..cd72658 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);

-- 

------------------------------------------------------------------------------
Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
Get 100% visibility into your production application - at no cost.
Code-level diagnostics for performance bottlenecks with <2% overhead
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap1
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to