RFA: Undocument set remotedebug

RFA: Undocument set remotedebug

[Daniel Jacobowitz]

This patch:

date: 2005/02/09 21:56:12;  author: cagney;  state: Exp;  lines: +0 -9
2005-02-09  Andrew Cagney  <cagney@gnu.org>

        * NEWS: Mention removed commands.
        * m68hc11-tdep.c (show_regs): Delete, unused.
        (_initialize_m68hc11_tdep): Delete deprecated "regs" command.
        * gdb-events.sh: Delete deprecated "set eventdebug" and "show
        eventdebug" commands.
        * gdb-events.c: Regenerate.
        * gdbarch.sh: Delete deprecated "set archdebug" and "show
        archdebug" commands.
        * gdbarch.c: Regenerate.
        * cli/cli-cmds.c (init_cli_cmds): Delete deprecated "set
        remotedebug" and "show remotedebug".
        * arm-tdep.c (_initialize_arm_tdep): Delete deprecated "set arm
        disassembly-flavor", "show arm disassembly-flavor", and
        "othernames" commands.
        (arm_othernames): Delete unused function.

removed "set remotedebug".  But a coworker discovered that it was still
documented in the manual.  The others all seem to have been removed (or else
were undocumented).

Eli, I preserved the index entries, since they all seemed reasonable.  But
now "set debug remote" has five, which seems like a bit much.  What do you
think?

-- 
Daniel Jacobowitz
CodeSourcery

2006-05-02  Daniel Jacobowitz  <dan@codesourcery.com>

	* gdb.texinfo (Remote Configuration): Remove "set remotedebug"
	and "show remotedebug".
	(Debugging Output): Add additional index entries.

Index: gdb.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v
retrieving revision 1.325
diff -u -p -r1.325 gdb.texinfo
--- gdb.texinfo	27 Apr 2006 23:03:42 -0000	1.325
+++ gdb.texinfo	2 May 2006 17:27:26 -0000
@@ -12452,17 +12452,6 @@ expect to see @samp{Ctrl-C} as the inter
 Show whether @value{GDBN} sends @code{BREAK} or @samp{Ctrl-C} to
 interrupt the remote program.
 
-@item set remotedebug
-@cindex debug remote protocol
-@cindex remote protocol debugging
-@cindex display remote packets
-Control the debugging of the remote protocol.  When enabled, each
-packet sent to or received from the remote target is displayed.  The
-defaults is off.
-
-@item show remotedebug
-Show the current setting of the remote protocol debugging.
-
 @item set remotedevice @var{device}
 @cindex serial port name
 Set the name of the serial port through which to communicate to the
@@ -15947,6 +15936,9 @@ Displays the current state of displaying
 debugging info.
 @cindex packets, reporting on stdout
 @cindex serial connections, debugging
+@cindex debug remote protocol
+@cindex remote protocol debugging
+@cindex display remote packets
 @item set debug remote
 Turns on or off display of reports on all packets sent back and forth across
 the serial line to the remote machine.  The info is printed on the

Re: RFA: Undocument set remotedebug

[Eli Zaretskii]

> Date: Tue, 2 May 2006 13:29:20 -0400
> From: Daniel Jacobowitz <drow@false.org>
> 
> Eli, I preserved the index entries, since they all seemed reasonable.

Yes, they are.

> But now "set debug remote" has five, which seems like a bit much.

Are you talking about these 5 @cindex entries:

>  @cindex packets, reporting on stdout
>  @cindex serial connections, debugging
> +@cindex debug remote protocol
> +@cindex remote protocol debugging
> +@cindex display remote packets
>  @item set debug remote

If so, there's nothing wrong with extra index entries, as long as we
believe that these are reasonable subjects to look up when one is
looking for the information described in this section.

This patch is approved.  Thanks.

Re: RFA: Undocument set remotedebug

[Daniel Jacobowitz]

On Tue, May 02, 2006 at 09:55:43PM +0300, Eli Zaretskii wrote:
> If so, there's nothing wrong with extra index entries, as long as we
> believe that these are reasonable subjects to look up when one is
> looking for the information described in this section.

OK then.  I've committed it - thanks for the review.

-- 
Daniel Jacobowitz
CodeSourcery