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: [patch 2/6] set auto-load * main part


> Date: Sat, 24 Mar 2012 19:37:47 +0100
> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> 
> --- a/gdb/NEWS
> +++ b/gdb/NEWS

This part is OK.

> +Enable or disable auto-loading canned sequences of commands scripts."), _("\
> +Show enablement of auto-loading canned sequences of commands scripts."), _("\

Ouch, "enablement"...  How about "state" instead?

> +@item set auto-load local-gdbinit [yes|no]
> +Enable or disable the auto-loading of
> +@xref{Sequences, Canned Sequences of Commands}, found in init file in the
> +current directory.

This clever way of cross-references doesn't work.  First @xref
produces a capitalized "See", which is inappropriate in the middle of
a sentence.  But even if you replace it with @ref, there's still that
"*note" that will look awkward.  And finally, the "Sequences" part is
displayed by Info readers, unlike HTML where the corresponding part of
a reference would not be displayed.  So I suggest this instead:

  Enable or disable the auto-loading of canned sequences of commands
  (@pxref{Sequences}) found in init file in the current directory.

> +@cindex print commands from current directory init file have been auto-loaded

This index entry is way too long.  How about

 @cindex report auto-loading from current directory

>  @smallexample
> -$ gdb -iex "set auto-load-scripts off" myprogram
> +$ gdb -iex "set auto-load python-scripts" myprogram

Did you really mean to omit the "off" part?

> +@item set auto-load no
> +Global disable of any auto-loading sub-commands of @value{GDBN}.  You may want
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  Globally disable any auto-loading ...

> +Similar to the @ref{objfile-gdb.py file} file @value{GDBN} also tries to load
> +an @file{@var{objfile}-gdb.rc} file containing
> +@xref{Sequences, Canned Sequences of Commands}, as long as
> +@samp{set auto-load gdb-scripts} is set to @samp{yes}.

This @xref needs the same surgery as I show above.

OK with those changes.


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