This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Display configuration details in --help
- From: Doug Evans <dje at google dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: Eli Zaretskii <eliz at gnu dot org>, Tom Tromey <tromey at redhat dot com>, Joel Brobecker <brobecker at adacore dot com>, gdb-patches <gdb-patches at sourceware dot org>
- Date: Tue, 9 Apr 2013 12:26:25 -0700
- Subject: Re: [PATCH] Display configuration details in --help
- References: <83r4jaynqq dot fsf at gnu dot org> <CADPb22R4o4gkQk8oeqGvmWZerVf7gdkw8XOKz9mYxD6ubFPRig at mail dot gmail dot com> <514B4AC3 dot 1090103 at redhat dot com> <83zjxwy4ux dot fsf at gnu dot org> <20130321191051 dot GF5447 at adacore dot com> <83r4j8y1kh dot fsf at gnu dot org> <20130321203919 dot GG5447 at adacore dot com> <87r4j8zd0k dot fsf at fleche dot redhat dot com> <83ehf85u7x dot fsf at gnu dot org> <83wqsbadfe dot fsf at gnu dot org> <20836 dot 17485 dot 525118 dot 795474 at ruffy2 dot mtv dot corp dot google dot com> <83txnfa9m7 dot fsf at gnu dot org> <CADPb22SQaWrBd4MOmMqFaAMc=KnX15hd=be3Ta_u3wGg=HGmXQ at mail dot gmail dot com> <51645982 dot 8050904 at redhat dot com>
On Tue, Apr 9, 2013 at 11:10 AM, Pedro Alves <palves@redhat.com> wrote:
>>> >>> --core=COREFILE Analyze the core dump COREFILE.
>
> Remove the ones above related to --command, and this one's
> actually in place.
>
>>> >>> --pid=PID Attach to running process PID.
>
> --pid is kind of a cousin of --core, due to how BAR is
> handled in "gdb FOO BAR".
And --exec is a cousin of --se/--symbols but they're apart and in
general alphabetical order.
Unless there is a physical form to the grouping (e.g., further
indentation, blank lines, or whatever), I'd much prefer a full
alphabetical sort.
>
>>> --dbx DBX compatibility mode.
>>> --directory=DIR Search for source files in DIR.
>>> --epoch Output information used by epoch emacs-GDB interface.
>>> --exec=EXECFILE Use EXECFILE as the executable.
>>> --fullname Output information used by emacs-GDB interface.
>>> --help Print this message.
>>> --interpreter=INTERP
>>> Select a specific interpreter / user interface
>>> -l TIMEOUT Set timeout in seconds for remote debugging.
>>> --nw Do not use a window interface.
>>> --nx Do not read .gdbinit file.
>>> --quiet Do not print version number on startup.
>>> --readnow Fully read symbol files on first access.
>>> --se=FILE Use FILE as symbol file and executable file.
>>> --symbols=SYMFILE Read symbols from SYMFILE.
>>> --tty=TTY Use TTY for input/output by the program being debugged.
>>> --version Print version information and then exit.
>>> -w Use a window interface.
>>> --write Set writing into executable and core files.
>>> --xdb XDB compatibility mode.
>>>
>>> Again, I'm not really wedded to the place where I inserted the new
>>> option, but please tell me near which 'c' option to put it instead ;-).
>>
>> Right after --command works for me.
>
> So if you imagine a right indent in the grouped options, the
> list is actually sorted, so that'd break the existing logic. I'd suggest
> either right after --version; right after --help; right before --core, or
> a wholesale reevaluation of the current grouping (either sort all
> alphabetically, or break "Options:" into groups).
Wholesale reevaluation.