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] Display configuration details in --help


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.


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