This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFC] Trailing spaces in solaris_pid_to_str
- From: dje at google dot com (Doug Evans)
- To: gdb-patches at sourceware dot org
- Date: Fri, 1 May 2009 14:32:18 -0700 (PDT)
- Subject: [RFC] Trailing spaces in solaris_pid_to_str
I can imagine that the trailing spaces are so the output of "info threads"
looks pretty, but these trailing spaces would mess up the text when used
elsewhere.
[Assuming the trailing spaces are indeed for prettier "info threads" ...]
If one wants the output of "info threads" to be prettier in this way
then it seems like the right thing to do is have the "info threads"
printer add the spaces. (a) info threads presumably knows more about
how much alignment is needed, and (b) all ports would win.
If the spaces are to line these all up:
"Thread %ld (defunct)"
"Thread %ld (LWP %ld)"
"Thread %ld "
"LWP %ld "
"process %d "
they're already in need of some TLC.
Doesn't matter to me whether to check this in or not,
just thought I'd pass it on. It is odd. Is there another reason
for these spaces that I'm missing?
[The spaces in "LWP %ld" are to line up the text with "Thread %ld".
They don't bother me as much as the trailing spaces so I left them in,
but they could just as well be deleted.]
2009-05-01 Doug Evans <dje@google.com>
* sol-thread.c (solaris_pid_to_str): Remove trailing spaces in result.
Index: sol-thread.c
===================================================================
RCS file: /cvs/src/src/gdb/sol-thread.c,v
retrieving revision 1.72
diff -u -p -r1.72 sol-thread.c
--- sol-thread.c 23 Feb 2009 00:03:50 -0000 1.72
+++ sol-thread.c 1 May 2009 21:19:50 -0000
@@ -1152,12 +1152,12 @@ solaris_pid_to_str (struct target_ops *o
sprintf (buf, "Thread %ld (LWP %ld)",
GET_THREAD (ptid), GET_LWP (lwp));
else
- sprintf (buf, "Thread %ld ", GET_THREAD (ptid));
+ sprintf (buf, "Thread %ld", GET_THREAD (ptid));
}
else if (GET_LWP (ptid) != 0)
- sprintf (buf, "LWP %ld ", GET_LWP (ptid));
+ sprintf (buf, "LWP %ld", GET_LWP (ptid));
else
- sprintf (buf, "process %d ", PIDGET (ptid));
+ sprintf (buf, "process %d", PIDGET (ptid));
return buf;
}