[RFA] Thread exit messages on MS-Windows
Joel Brobecker
brobecker@adacore.com
Mon Apr 29 17:31:00 GMT 2013
> No one replied, so I'm now converting this into an RFA. The patch
> below causes GDB on Windows to display thread exit messages like this:
>
> [Thread 5920.0x13e4 exited with code 0]
> [Thread 5920.0x12d0 exited with code 0]
> [Thread 5920.0x1cbc exited with code 0]
> 2013-04-27 Eli Zaretskii <eliz@gnu.org>
>
> * windows-nat.c (windows_delete_thread): Accept an additional
> argument, the thread's exit code, and announce thread death when
> print_thread_events is non-zero and we are deleting a thread that
> is not the main thread.
> (get_windows_debug_event): Pass thread exit code to
> windows_delete_thread.
Looks good to me, modulo the comments already made. It's a little
unusual to see an exit code for a thread, but it could be useful
information, and it does not unnecessarily clutter the output.
> @@ -1513,7 +1517,7 @@ get_windows_debug_event (struct target_o
> current_process_handle = current_event.u.CreateProcessInfo.hProcess;
> if (main_thread_id)
> windows_delete_thread (ptid_build (current_event.dwProcessId, 0,
> - main_thread_id));
> + main_thread_id), 0);
One tiny nitpick, very possibly influenced by personal preferences,
so feel free to ignore... I think that the code would be faster
to read if the added parameter was moved to the next line. That way,
all parameters in call to windows_delete_thread would have the same
indentation level.
--
Joel
More information about the Gdb-patches
mailing list