From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-66662-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 12468 invoked by alias); 9 Sep 2009 19:17:54 -0000
Received: (qmail 12434 invoked by uid 22791); 9 Sep 2009 19:17:53 -0000
X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 	tests=AWL,BAYES_00
X-Spam-Check-By: sourceware.org
Received: from mtaout6.012.net.il (HELO mtaout6.012.net.il) (84.95.2.16)     by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 09 Sep 2009 19:17:44 +0000
Received: from conversion-daemon.i-mtaout6.012.net.il by i-mtaout6.012.net.il (HyperSendmail v2007.08) id <0KPP00700VIPZ800@i-mtaout6.012.net.il> for gdb-patches@sourceware.org; Wed, 09 Sep 2009 22:17:39 +0300 (IDT)
Received: from HOME-C4E4A596F7 ([84.228.50.163]) by i-mtaout6.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0KPP00HC9XLFYTF0@i-mtaout6.012.net.il>; Wed, 09 Sep 2009 22:17:39 +0300 (IDT)
Date: Wed, 09 Sep 2009 19:17:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: PING: [RFA/doco] document "set/show interactive-mode"
In-reply-to: <20090909180355.GE20694@adacore.com>
To: Joel Brobecker <brobecker@adacore.com>
Cc: gdb-patches@sourceware.org
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <8363bs2al9.fsf@gnu.org>
References: <20090623181341.GK5085@adacore.com> <20090909180355.GE20694@adacore.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: 2009-09/txt/msg00257.txt.bz2

> Date: Wed, 9 Sep 2009 11:03:55 -0700
> From: Joel Brobecker <brobecker@adacore.com>
> 
> Looks like this one fell through the cracks.

Yes, sorry.  It was sitting in my inbox all this time.

> +@cindex Miscellaneous Settings

Index entries should not use capital letters in simple words.  Our
convention is to use only lower-case text.

> +  add_setshow_auto_boolean_cmd ("interactive-mode", class_support,
> +                                &interactive_mode, _("\
> +Set whether gdb should run in interactive mode or not"), _("\

"gdb" or "GDB"?

> +If on, gdb runs in interactive mode and waits for the user to answer\n\
> +all its queries.  If off, gdb runs in non-interactive mode and\n\
> +automatically assumes the default answer to all its queries.  If auto\n\
> +(which is the default), automatically determine which mode to use based\n\
> +on the standard input settings"),

A couple of minor stylistic comments:

 . For "on" and "off", you use "gdb runs", but for "auto" you say
   "automatically determine", which is inconsistent.  "gdb determines"
   in the latter case or "run" in the former case is more consistent.
   (I like the second alternative better, as it's more concise.)

 . I would lose the "automatically" part in the last sentence, since
   "auto" already says that.

Finally, what about NEWS?