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] PR 12999 info macros command.


>>>>> "Matt" == Matt Rice <ratmice@gmail.com> writes:

Matt> Updated the docs, and the 'help' command text for 'info definitions'.
Matt> I added @cindex macros, from debug info
Matt> so it would be near the existing @cindex macros, user-defined

Two more nits...

Matt> +  if (! name || ! *name)
Matt> +    error (_("The `info definitions' command requires a macro name as an \
Matt> +argument.\n"
Matt> +           "Usage: info definitions MACRO\n"));

I think "usage" lines should be in the help text, but not in errors.
So, please remove it here.
Also, errors should not end with \n.

Matt> +  add_cmd ("macros", no_class, info_macros_command,
Matt> +	   _("Show the definitions of all macros at LINESPEC, or the current \
Matt> +source location.\n\
Matt> +Usage: info source [LINESPEC]"),

Typo: s/source/macros/

This is ok with those changes.

Tom


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