This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: PING: [RFA/doco] document "set/show interactive-mode"
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Wed, 09 Sep 2009 22:15:46 +0300
- Subject: Re: PING: [RFA/doco] document "set/show interactive-mode"
- References: <20090623181341.GK5085@adacore.com> <20090909180355.GE20694@adacore.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> 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?