This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA] gdb/win32-nat.c Step the correct thread
- From: "Pierre Muller" <muller at ics dot u-strasbg dot fr>
- To: <gdb-patches at sourceware dot org>
- Date: Thu, 29 Nov 2007 16:45:17 +0100
- Subject: [RFA] gdb/win32-nat.c Step the correct thread
While trying to understand the SuspendThread
counting problems, I tried to single step after changing
thread, and I noticed that the next event always
brought me back to the main thread.
There is a bug in the current win32-nat.c source
which places the trace bit always in the
main thread, while we should step the thread identified by
inferior_ptid.
This simple patch corrects this.
The testsuite runs with and without this patch
show no difference.
OK to check in?
2007-11-29 Pierre Muller <muller@ics.u-strasbg.fr>
* win32-nat.c (win32_resume): Set the trace bit
in the thread identified by inferior_ptid.
Index: gdb/win32-nat.c
===================================================================
RCS file: /cvs/src/src/gdb/win32-nat.c,v
retrieving revision 1.140
diff -u -p -r1.140 win32-nat.c
--- gdb/win32-nat.c 24 Nov 2007 12:13:28 -0000 1.140
+++ gdb/win32-nat.c 29 Nov 2007 13:52:18 -0000
@@ -1225,7 +1225,7 @@ win32_resume (ptid_t ptid, int step, enu
pid, step, sig));
/* Get context for currently selected thread */
- th = thread_rec (current_event.dwThreadId, FALSE);
+ th = thread_rec (PIDGET (inferior_ptid), FALSE);
if (th)
{
if (step)