[PATCH] Print thread name when executing thread commands

[Pat Pannuto <pat.pannuto@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 870 bytes --]

Currently the thread family of commands only prints the thread ID and
PID. GDB also has access to the thread's name, which is often an easier
way of quickly identifying a thread. This simple patch uses the same
mechanism as `info threads' to get the name of a thread and add it to
the thread identifier line when it's printed.

I wrapped the name in []'s to offset it and make it visually distinct
from the ()'s that wrap the other thread information. I'm not at all
attached to this look, just looked best to me.

Diff should be from current cvs. This is my first patch to GDB, if I
messed something up don't hesitate to yell at me.

Bonus: solves my old SO question:
http://stackoverflow.com/questions/12679339/make-gdb-show-thread-names-on-apply-all-operations

-Pat

-- 
Pat Pannuto
NSF/NDSEG/Qualcomm Fellow
Computer Engineering
University of Michigan
248.990.4548

[-- Attachment #2: print_thread_names.ChangeLog --]
[-- Type: application/octet-stream, Size: 156 bytes --]

2013-09-18  Pat Pannuto <pat.pannuto@gmail.com>

	* thread.c: Print thread name as well as ID number and PID when
	executing the thread family of commands


[-- Attachment #3: print_thread_names.diff --]
[-- Type: application/octet-stream, Size: 1513 bytes --]

Index: gdb/thread.c
===================================================================
RCS file: /cvs/src/src/gdb/thread.c,v
retrieving revision 1.157
diff -u -p -r1.157 thread.c
--- gdb/thread.c	17 Sep 2013 18:26:39 -0000	1.157
+++ gdb/thread.c	18 Sep 2013 06:33:29 -0000
@@ -1255,10 +1255,14 @@ thread_apply_all_command (char *cmd, int
       for (k = 0; k != i; k++)
         if (thread_alive (tp_array[k]))
           {
+            char* name;
             switch_to_thread (tp_array[k]->ptid);
-            printf_filtered (_("\nThread %d (%s):\n"), 
+            name = tp_array[k]->name ? tp_array[k]->name
+                    : target_thread_name (tp_array[k]);
+            printf_filtered (_("\nThread %d (%s) [%s]:\n"), 
 			     tp_array[k]->num,
-			     target_pid_to_str (inferior_ptid));
+			     target_pid_to_str (inferior_ptid),
+			     name);
             execute_command (cmd, from_tty);
 
             /* Restore exact command used previously.  */
@@ -1308,10 +1312,12 @@ thread_apply_command (char *tidlist, int
 	warning (_("Thread %d has terminated."), start);
       else
 	{
+	  char *name;
 	  switch_to_thread (tp->ptid);
 
-	  printf_filtered (_("\nThread %d (%s):\n"), tp->num,
-			   target_pid_to_str (inferior_ptid));
+	  name = tp->name ? tp->name : target_thread_name (tp);
+	  printf_filtered (_("\nThread %d (%s) [%s]:\n"), tp->num,
+			   target_pid_to_str (inferior_ptid), name);
 	  execute_command (cmd, from_tty);
 
 	  /* Restore exact command used previously.  */