winedbg: Rewrite and simplify step / continue handlers.

The vCont handler used some overcomplicated logic, we only need to iterate over the actions and apply them on the matching threads that didn't match yet. Thanks to DBG_REPLY_LATER we can now continue/step any thread regardless of whether it is the one that raised the debug event. Just suspend all active threads after debug event is raised and resume them one by one, according to the gdb request. If the thread that raised the debug event should not be resumed, reply with DBG_REPLY_LATER. Signed-off-by: Rémi Bernon <rbernon@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winedbg: Rewrite and simplify step / continue handlers.
983d02c0 · Rémi Bernon · Alexandre Julliard · 7af48607 · 983d02c0 · 983d02c0
Commit 983d02c0 authored Apr 01, 2020 by Rémi Bernon Committed by Alexandre Julliard Apr 01, 2020
Expand all Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

debugger.h programs/winedbg/debugger.h +1 -0

gdbproxy.c programs/winedbg/gdbproxy.c +0 -0

winedbg.c programs/winedbg/winedbg.c +1 -0

No files found.
--- a/programs/winedbg/debugger.h
+++ b/programs/winedbg/debugger.h
@@ -207,6 +207,7 @@ struct dbg_thread
    }*                          frames;
    int                         num_frames;
    int                         curr_frame;
+    BOOL                        suspended;
 };

 struct dbg_delayed_bp

--- a/programs/winedbg/gdbproxy.c
+++ b/programs/winedbg/gdbproxy.c
--- a/programs/winedbg/winedbg.c
+++ b/programs/winedbg/winedbg.c
@@ -500,6 +500,7 @@ struct dbg_thread* dbg_add_thread(struct dbg_process* p, DWORD tid,
    t->num_frames = 0;
    t->curr_frame = -1;
    t->addr_mode = AddrModeFlat;
+    t->suspended = FALSE;

    snprintf(t->name, sizeof(t->name), "%04x", tid);