Re: RFC: Clean up "show remote"

[Daniel Jacobowitz <drow at false dot org>]

On Mon, Jan 23, 2006 at 06:38:37AM +0200, Eli Zaretskii wrote:
> > Also shorten the messages a bit by removing redundancy; either the user
> > typed config->title, or it was prefixed to the output already.
> 
> I'm not quite sure I get this.  With the current code, "show remote"
> outputs lines like this:
> 
>  Support for remote protocol `Z0' (software-breakpoint) packet is auto-detected,
>  currently unknown.
> 
> With your change, the part in parentheses will not be displayed, so
> how would the user know that Z0 is a packet that sets software
> breakpoints?

Instead, you'll get something that looks like the current output of
"show print":

software-breakpoint-packet: Support for the `Z0' packet is auto-detected, currently unknown

I'm trying to cut down on how badly this wraps, since we can't
word-wrap in translated messages (is there any way to do that?
GCC seems to wrap using spaces even in i18n output; maybe we
could do the same, if I am interpreting that right?).

> > An alternative if the code in show_remote_cmd is too ugly would be to delete
> > "show remote Z-packet".  I think we have to leave "set remote Z-packet" -
> > it's five years obsoleted but never marked as deprecated and I know people
> > are continuing to use it - but the show command is completely redundant.
> 
> I think what you did is fine, but perhaps explain a bit more in
> comments about why Z-packet is redundant, and that maybe it should be
> removed later on.

Willdo (although I don't know if we can reasonably remove the "set"
variant - I keep finding new docs suggesting it.)

-- 
Daniel Jacobowitz
CodeSourcery