[RFC] Trailing spaces in solaris_pid_to_str

Joel Brobecker brobecker@adacore.com
Tue May 12 09:19:00 GMT 2009


> 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?

Can't figure it out. I did a bit of archeology, and this pre-dates
the public CVS.

> 2009-05-01  Doug Evans  <dje@google.com>
> 
> 	* sol-thread.c (solaris_pid_to_str): Remove trailing spaces in result.

I'm not objecting, but if it's just a visual annoyance and no one else
provides feedback, then perhaps it's better to leave things as is
(taste varies from person to person). If you need that function
elsewhere and it's causing trouble, on the other hand...

> [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.]

If you do remove the trailing spaces, might as well remove the leading
ones too. There're there for formatting, so if we remove the formatting,
we might as well remove all of it (IMO).

-- 
Joel



More information about the Gdb-patches mailing list