This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PING: [RFA/doco] document "set/show interactive-mode"


> 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?


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]