2011-02-14 Michael Snyder <msnyder@vmware.com>

	* command.h (enum command_class): New class 'no_set_class', for
	"show" commands without a corresponding "set" command.
	* value.c (_initialize_values): Use 'no_set_class' for "show values".
	* copying.c (_initialize_copying): Ditto for "show copying" and
	"show warranty".
	* cli/cli-cmds.c (init_cli_cmds): Ditto for "show commands" and
	"show version".
	* cli/cli-setshow.c (cmd_show_list): Skip "show" commands for
	which there is no corresponding "set" command (eg. "show copying").

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c88a9f4f7283ecf438666ea6c08a5b760633ef2e..fc013d56f3dde71ccae5820cf11ce40256fec229 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,15 @@
+2011-02-14  Michael Snyder  <msnyder@vmware.com>
+
+       * command.h (enum command_class): New class 'no_set_class', for
+       "show" commands without a corresponding "set" command.
+       * value.c (_initialize_values): Use 'no_set_class' for "show values".
+       * copying.c (_initialize_copying): Ditto for "show copying" and
+       "show warranty".
+       * cli/cli-cmds.c (init_cli_cmds): Ditto for "show commands" and
+       "show version".
+       * cli/cli-setshow.c (cmd_show_list): Skip "show" commands for
+       which there is no corresponding "set" command (eg. "show copying").
+
 2011-02-14  Pedro Alves  <pedro@codesourcery.com>
            Jan Kratochvil  <jan.kratochvil@redhat.com>
 

diff --git a/gdb/cli/cli-cmds.c b/gdb/cli/cli-cmds.c
index c11b257eeaeaaa78a26ac799ab50d02d48d71952..e3cccdfc2be57bd1f09259af9dc76a126b333c01 100644 (file)
--- a/gdb/cli/cli-cmds.c
+++ b/gdb/cli/cli-cmds.c
@@ -1533,13 +1533,13 @@ Generic command for showing things about the debugger."),
   /* Another way to get at the same thing.  */
   add_info ("set", show_command, _("Show all GDB settings."));
 
-  add_cmd ("commands", no_class, show_commands, _("\
+  add_cmd ("commands", no_set_class, show_commands, _("\
 Show the history of commands you typed.\n\
 You can supply a command number to start with, or a `+' to start after\n\
 the previous command number shown."),
           &showlist);
 
-  add_cmd ("version", no_class, show_version,
+  add_cmd ("version", no_set_class, show_version,
           _("Show what version of GDB this is."), &showlist);
 
   add_com ("while", class_support, while_command, _("\

diff --git a/gdb/cli/cli-setshow.c b/gdb/cli/cli-setshow.c
index c676f3bf3bcdacd2082e99942992fe9199718c08..6acdd9156c659e3af91db06bedbe4a22471dc201 100644 (file)
--- a/gdb/cli/cli-setshow.c
+++ b/gdb/cli/cli-setshow.c
@@ -434,18 +434,21 @@ cmd_show_list (struct cmd_list_element *list, int from_tty, char *prefix)
        }
       else
        {
-         struct cleanup *option_chain
-           = make_cleanup_ui_out_tuple_begin_end (uiout, "option");
-
-         ui_out_text (uiout, prefix);
-         ui_out_field_string (uiout, "name", list->name);
-         ui_out_text (uiout, ":  ");
-         if (list->type == show_cmd)
-           do_setshow_command ((char *) NULL, from_tty, list);
-         else
-           cmd_func (list, NULL, from_tty);
-          /* Close the tuple.  */
-         do_cleanups (option_chain);
+         if (list->class != no_set_class)
+           {
+             struct cleanup *option_chain
+               = make_cleanup_ui_out_tuple_begin_end (uiout, "option");
+
+             ui_out_text (uiout, prefix);
+             ui_out_field_string (uiout, "name", list->name);
+             ui_out_text (uiout, ":  ");
+             if (list->type == show_cmd)
+               do_setshow_command ((char *) NULL, from_tty, list);
+             else
+               cmd_func (list, NULL, from_tty);
+             /* Close the tuple.  */
+             do_cleanups (option_chain);
+           }
        }
     }
   /* Close the tuple.  */

diff --git a/gdb/command.h b/gdb/command.h
index d2f5ca501f75b8cd156b1a177da6cfb777a40673..0e2547e7e34771d6c2831db245f75fa2340640c7 100644 (file)
--- a/gdb/command.h
+++ b/gdb/command.h
@@ -34,7 +34,9 @@ enum command_class
   no_class = -1, class_run = 0, class_vars, class_stack, class_files,
   class_support, class_info, class_breakpoint, class_trace,
   class_alias, class_bookmark, class_obscure, class_maintenance,
-  class_pseudo, class_tui, class_user, class_xdb
+  class_pseudo, class_tui, class_user, class_xdb,
+  no_set_class /* Used for "show" commands that have no corresponding
+                  "set" command.  */
 };
 
 /* FIXME: cagney/2002-03-17: Once cmd_type() has been removed, ``enum

diff --git a/gdb/copying.c b/gdb/copying.c
index 6be9b8fb5b00cc6fb6fa20880e2db8331f9a9a5b..d608a63669e4cc924ac290fa15403fa768f8a594 100644 (file)
--- a/gdb/copying.c
+++ b/gdb/copying.c
@@ -649,10 +649,10 @@ show_warranty_command (char *ignore, int from_tty)
 void
 _initialize_copying (void)
 {
-  add_cmd ("copying", no_class, show_copying_command,
+  add_cmd ("copying", no_set_class, show_copying_command,
           _("Conditions for redistributing copies of GDB."),
           &showlist);
-  add_cmd ("warranty", no_class, show_warranty_command,
+  add_cmd ("warranty", no_set_class, show_warranty_command,
           _("Various kinds of warranty you do not have."),
           &showlist);
 

diff --git a/gdb/value.c b/gdb/value.c
index 116eee269daee5ee645b6f291fed8fb79a591842..2e07a85cdd120e2526255cb644b458fb5d27a479 100644 (file)
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -3074,7 +3074,7 @@ A few convenience variables are given values automatically:\n\
 \"$__\" holds the contents of the last address examined with \"x\"."),
           &showlist);
 
-  add_cmd ("values", no_class, show_values, _("\
+  add_cmd ("values", no_set_class, show_values, _("\
 Elements of value history around item number IDX (or last ten)."),
           &showlist);