This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] Expand "show disassembler-options" output
- From: Tom Tromey <tromey at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 16 Sep 2018 12:26:20 -0000
- Subject: [binutils-gdb] Expand "show disassembler-options" output
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f4bab6ff22a0ddb799c1c492117a15fe0501ff1e
commit f4bab6ff22a0ddb799c1c492117a15fe0501ff1e
Author: Tom Tromey <tom@tromey.com>
Date: Fri Sep 14 07:34:05 2018 -0600
Expand "show disassembler-options" output
I typed this:
(gdb) help set disassembler-options
Set the disassembler options.
Usage: set disassembler-options OPTION [,OPTION]...
See: 'show disassembler-options' for valid option values.
... so I tried what it said and got:
(gdb) show disassembler-options
The current disassembler options are ''
This surprised me a little, so this patch adds some text to explain
the situation when an architecture does not have disassembler options.
While there I noticed one more spot where gdb was not using the GNU
style for metasyntactic variables. This patch fixes this as well.
gdb/ChangeLog
2018-09-16 Tom Tromey <tom@tromey.com>
* disasm.c (show_disassembler_options_sfunc): Use GNU style for
metasyntactic variables. Print message if no disassembler options
are available.
Diff:
---
gdb/ChangeLog | 6 ++++++
gdb/disasm.c | 13 +++++++++----
2 files changed, 15 insertions(+), 4 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index da9b450..095c6b5 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2018-09-16 Tom Tromey <tom@tromey.com>
+
+ * disasm.c (show_disassembler_options_sfunc): Use GNU style for
+ metasyntactic variables. Print message if no disassembler options
+ are available.
+
2018-09-15 Tom Tromey <tom@tromey.com>
* infcmd.c (get_inferior_args): Return const char *.
diff --git a/gdb/disasm.c b/gdb/disasm.c
index 11793df..de1c7dc 100644
--- a/gdb/disasm.c
+++ b/gdb/disasm.c
@@ -1008,19 +1008,24 @@ show_disassembler_options_sfunc (struct ui_file *file, int from_tty,
if (options == NULL)
options = "";
- fprintf_filtered (file, _("The current disassembler options are '%s'\n"),
+ fprintf_filtered (file, _("The current disassembler options are '%s'\n\n"),
options);
valid_options_and_args = gdbarch_valid_disassembler_options (gdbarch);
if (valid_options_and_args == NULL)
- return;
+ {
+ fputs_filtered (_("There are no disassembler options available "
+ "for this architecture.\n"),
+ file);
+ return;
+ }
valid_options = &valid_options_and_args->options;
- fprintf_filtered (file, _("\n\
+ fprintf_filtered (file, _("\
The following disassembler options are supported for use with the\n\
-'set disassembler-options <option>[,<option>...]' command:\n"));
+'set disassembler-options OPTION [,OPTION]...' command:\n"));
if (valid_options->description != NULL)
{