gdb: change 'maint info section' to use command options
authorAndrew Burgess <andrew.burgess@embecosm.com>
Fri, 5 Feb 2021 13:51:34 +0000 (13:51 +0000)
committerAndrew Burgess <andrew.burgess@embecosm.com>
Thu, 11 Feb 2021 10:26:19 +0000 (10:26 +0000)
commitbf3386f0c13f52cf379440b4d06ed849a9f5706b
tree26cd23b8e3508cc80be1aa2e823dd778ac96f368
parent4790db149699b21113c566b8b9249953038fd3a7
gdb: change 'maint info section' to use command options

The 'maintenance info sections' command currently takes a list of
filters on the command line.  It can also accept the magic string
'ALLOBJ' which acts more like a command line flag, telling the command
to print information about all objfiles.

The manual has this to say about the options and filters:

  ... In addition, 'maint info sections' provides the following
  command options (which may be arbitrarily combined): ...

Implying (to me at least) that I can do this:

  (gdb) maint info sections ALLOBJ READONLY

to list all the read-only sections from all currently loaded object
files.

Unfortunately, this doesn't work.  The READONLY filter will work, but
ALLOBJ will not be detected correctly.

It would be fairly simple to fix the ALLOBJ detection.  However, I
dislike this mixing of command options (ALLOBJ) with command data (the
filters, e.g. READONLY, etc).

As this is a maintenance command, so not really intended for end
users, I think we can be a little more aggressive in "fixing" the
option parsing.  So that's what I do in this commit.

The ALLOBJ mechanism is replaced with a real command
option (-all-objects).  The rest of the command operates just as
before.  The example above would now become:

  (gdb) maint info sections -all-objects READONLY

The manual has been updated, and I added a NEWS entry to document the
change.

gdb/ChangeLog:

* NEWS: Mention changes to 'maint info sections'.
* maint.c (match_substring): Return a bool, fix whitespace issue.
(struct single_bfd_flag_info): New struct.
(bfd_flag_info): New static global.
(match_bfd_flags): Return a bool, use bfd_flag_info.
(print_bfd_flags): Use bfd_flag_info.
(maint_print_section_info): Delete trailing whitespace.
(struct maint_info_sections_opts): New struct.
(maint_info_sections_option_defs): New static global.
(maint_info_sections_completer): New function.
(maintenance_info_sections): Use option parsing mechanism.
(_initialize_maint_cmds): Register command completer.

gdb/doc/ChangeLog:

* gdb.texinfo (Files): Update documentation for 'maint info
sections'.

gdb/testsuite/ChangeLog:

* gdb.base/maint-info-sections.exp: Update expected output, and
add additional tests. Again.
gdb/ChangeLog
gdb/NEWS
gdb/doc/ChangeLog
gdb/doc/gdb.texinfo
gdb/maint.c
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/maint-info-sections.exp