From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8219 invoked by alias); 28 Mar 2004 12:32:29 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 8212 invoked from network); 28 Mar 2004 12:32:27 -0000 Received: from unknown (HELO kraid.nerim.net) (62.4.16.100) by sources.redhat.com with SMTP; 28 Mar 2004 12:32:27 -0000 Received: from nerim.fr (stcarrez.net1.nerim.net [62.212.108.40]) by kraid.nerim.net (Postfix) with ESMTP id ED13F4191B; Sun, 28 Mar 2004 14:32:25 +0200 (CEST) Message-ID: <4066C5D4.5010109@nerim.fr> Date: Sun, 28 Mar 2004 12:32:00 -0000 From: Stephane Carrez User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624 Netscape/7.1 MIME-Version: 1.0 To: Eli Zaretskii Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA]: Document tui reg commands References: <405318FE.8000904@nerim.fr> <6480-Sat13Mar2004182210+0200-eliz@elta.co.il> In-Reply-To: <6480-Sat13Mar2004182210+0200-eliz@elta.co.il> X-Enigmail-Version: 0.76.8.0 X-Enigmail-Supports: pgp-inline, pgp-mime Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig13B30F4B5A16C219AF295AF6" X-SW-Source: 2004-03/txt/msg00701.txt.bz2 This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig13B30F4B5A16C219AF295AF6 Content-Type: multipart/mixed; boundary="------------030205090200010104020501" This is a multi-part message in MIME format. --------------030205090200010104020501 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Content-length: 2080 Hi! Eli Zaretskii wrote: >>Date: Sat, 13 Mar 2004 15:21:50 +0100 >>From: Stephane Carrez >> >>This is the documentation part of the new TUI reg commands. > > > Thanks. > > >>Ok to commit? > > > Yes, but please fix the following small gotchas: > > >>+@item tui reg float >>+@kindex tui reg float >>+Show the floating point registers in the register window. >>+ >>+@item tui reg general >>+@kindex tui reg general >>+Show the general registers in the register window. >>+ >>+@item tui reg next >>+@kindex tui reg next > > > There's no need to make a @kindex entry for all the 3 variants of this > command; a single "@kindex tui reg" will do. In general, it's not > useful to have several index entries that all begin with the same > string and all point to the same page, because that doesn't add any > information to someone who is looking through the index. > Ok. I've committed the attached patch in 6_1 and main. > >>+ [...] The predefined register groups are the >>+following: @code{general}, @code{float}, @code{system}, @code{vector}, >>+@code{all}, @code{save}, @code{restore}. >>+ >>+@item tui reg system >>+@kindex tui reg system >>+Show the system registers in the register window. > > > I think we should have a description of at least some of these > predefined register groups. For example, I could probably easily > guess what "reg float" or "reg vector" mean, but I have no idea about > "reg system" or "reg save/restore". I'm afraid a description such as > "Show the system registers in the register window." doesn't help me to > unlock this mystery. > I agree. The semantics of 'system' is unclear to me. For 'save/restore' may be the TUI should skip them silently as they are related to gdb internals (registers that are saved/restored by gdb when it calls inferior functions). From my point of view, if these groups should ever be documented, they should be documented outside of the TUI chapter (but today this is the only place 'register group' is mentionned...; apart from mainainters commands...) Stephane --------------030205090200010104020501 Content-Type: text/plain; name="gdb.texinfo.diffs" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="gdb.texinfo.diffs" Content-length: 998 Index: gdb.texinfo =================================================================== RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v retrieving revision 1.201 diff -u -p -r1.201 gdb.texinfo --- gdb.texinfo 26 Mar 2004 16:15:57 -0000 1.201 +++ gdb.texinfo 28 Mar 2004 12:21:22 -0000 @@ -14236,6 +14236,22 @@ can be affected to another window. @item refresh @kindex refresh Refresh the screen. This is similar to using @key{C-L} key. + +@item tui reg float +@kindex tui reg +Show the floating point registers in the register window. + +@item tui reg general +Show the general registers in the register window. + +@item tui reg next +Show the next register group. The list of register groups as well as +their order is target specific. The predefined register groups are the +following: @code{general}, @code{float}, @code{system}, @code{vector}, +@code{all}, @code{save}, @code{restore}. + +@item tui reg system +Show the system registers in the register window. @item update @kindex update --------------030205090200010104020501-- --------------enig13B30F4B5A16C219AF295AF6 Content-Type: application/pgp-signature Content-length: 253 -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) Comment: Using GnuPG with Netscape - http://enigmail.mozdev.org iD8DBQFAZsXaNyQxO2LzKT0RAkEXAKCtcuczSNShU3jBrK6NCYqzHgXG0gCgumFn ga5mpkeIIJC6N2vbfv5yt8M= =UI+x -----END PGP SIGNATURE----- --------------enig13B30F4B5A16C219AF295AF6--