From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-48873-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 2416 invoked by alias); 11 May 2007 17:58:26 -0000
Received: (qmail 2406 invoked by uid 22791); 11 May 2007 17:58:25 -0000
X-Spam-Check-By: sourceware.org
Received: from nitzan.inter.net.il (HELO nitzan.inter.net.il) (213.8.233.22)     by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 11 May 2007 17:58:23 +0000
Received: from HOME-C4E4A596F7 (IGLD-84-228-51-101.inter.net.il [84.228.51.101]) 	by nitzan.inter.net.il (MOS 3.7.3a-GA) 	with ESMTP id GSM07368 (AUTH halo1); 	Fri, 11 May 2007 20:58:16 +0300 (IDT)
Date: Fri, 11 May 2007 17:58:00 -0000
Message-Id: <uirazfdjd.fsf@gnu.org>
From: Eli Zaretskii <eliz@gnu.org>
To: Bob Wilson <bwilson@tensilica.com>
CC: gdb-patches@sources.redhat.com
In-reply-to: <46423A94.2090607@tensilica.com> (message from Bob Wilson on Wed, 	09 May 2007 14:18:12 -0700)
Subject: Re: [PATCH] edit documentation for TUI mode
Reply-to: Eli Zaretskii <eliz@gnu.org>
References:  <46423A94.2090607@tensilica.com>
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2007-05/txt/msg00196.txt.bz2

> Date: Wed, 09 May 2007 14:18:12 -0700
> From: Bob Wilson <bwilson@tensilica.com>
> 
> I noticed that the TUI-related sections of the GDB docs are a little rough.  I'm 
> no expert on this feature, but I thought I'd have a go at cleaning up the docs 
> anyway.  I discovered a few minor discrepancies between the current behavior and 
> the document, along with numerous issues of style, formatting and grammar. 
> Here's what I came up with.  Is it OK?

Thanks!  This is approved, provided that you take care of a couple
minor comments below:

> +@pindex @value{GDBTUI}
> +The TUI mode is enabled by default when you invoke @value{GDBN} as
> +either @samp{@value{GDBTUI}} or @samp{@value{GDBP} -tui}.
> +You can also switch in and out of TUI mode while @value{GDBN} runs.

The last sentence begs a question: ``how''?  We should either answer
that right there, or have an xref to where the appropriate command is
described.

> -@table @key
> +@table @asis
>  @kindex PgUp
> -@item PgUp
> +@item @key{PgUp}
>  Scroll the active window one page up.

Why did you switch to `@table @asis' and an explicit @key in each
item?  What was wrong with the previous method?