Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFA] Add constant CMD_LIST_AMBIGUOUS to replace (struct cmd_list_element *) -1
@ 2011-03-11 21:29 Michael Snyder
  2011-03-11 21:44 ` Tom Tromey
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Snyder @ 2011-03-11 21:29 UTC (permalink / raw)
  To: gdb-patches, Joel Brobecker

[-- Attachment #1: Type: text/plain, Size: 5 bytes --]

OK?


[-- Attachment #2: ambiguous.txt --]
[-- Type: text/plain, Size: 3853 bytes --]

2011-03-11  Michael Snyder  <msnyder@vmware.com>

	* cli/cli-decode.h (CMD_LIST_AMBIGUOUS): Define.
	* cli/cli-decode.c (lookup_cmd_1): Use CMD_LIST_AMBIGUOUS.
	(lookup_cmd): Test for CMD_LIST_AMBIGUOUS.
	* completer.c (complete_line_internal): Use CMD_LIST_AMBIGUOUS.
	* top.c (set_verbose): Use CMD_LIST_AMBIGUOUS.

Index: completer.c
===================================================================
RCS file: /cvs/src/src/gdb/completer.c,v
retrieving revision 1.44
diff -u -p -r1.44 completer.c
--- completer.c	26 Feb 2011 02:07:07 -0000	1.44
+++ completer.c	11 Mar 2011 20:55:25 -0000
@@ -590,7 +590,7 @@ complete_line_internal (const char *text
     {
       /* An empty line we want to consider ambiguous; that is, it
 	 could be any command.  */
-      c = (struct cmd_list_element *) -1;
+      c = CMD_LIST_AMBIGUOUS;
       result_list = 0;
     }
   else
@@ -610,7 +610,7 @@ complete_line_internal (const char *text
 	 possible completions.  */
       list = NULL;
     }
-  else if (c == (struct cmd_list_element *) -1)
+  else if (c == CMD_LIST_AMBIGUOUS)
     {
       char *q;
 
Index: top.c
===================================================================
RCS file: /cvs/src/src/gdb/top.c,v
retrieving revision 1.194
diff -u -p -r1.194 top.c
--- top.c	11 Mar 2011 20:38:57 -0000	1.194
+++ top.c	11 Mar 2011 20:55:25 -0000
@@ -1425,8 +1425,7 @@ set_verbose (char *args, int from_tty, s
   struct cmd_list_element *showcmd;
 
   showcmd = lookup_cmd_1 (&cmdname, showlist, NULL, 1);
-  gdb_assert (showcmd != NULL
-	      && showcmd != (struct cmd_list_element *) -1);
+  gdb_assert (showcmd != NULL && showcmd != CMD_LIST_AMBIGUOUS);
 
   if (info_verbose)
     {
Index: cli/cli-decode.c
===================================================================
RCS file: /cvs/src/src/gdb/cli/cli-decode.c,v
retrieving revision 1.92
diff -u -p -r1.92 cli-decode.c
--- cli/cli-decode.c	6 Jan 2011 00:57:02 -0000	1.92
+++ cli/cli-decode.c	11 Mar 2011 20:55:25 -0000
@@ -1228,7 +1228,7 @@ lookup_cmd_1 (char **text, struct cmd_li
 	/* Will be modified in calling routine
 	   if we know what the prefix command is.  */
 	*result_list = 0;
-      return (struct cmd_list_element *) -1;	/* Ambiguous.  */
+      return CMD_LIST_AMBIGUOUS;	/* Ambiguous.  */
     }
 
   /* We've matched something on this list.  Move text pointer forward.  */
@@ -1261,7 +1261,7 @@ lookup_cmd_1 (char **text, struct cmd_li
 	    *result_list = clist;
 	  return found;
 	}
-      else if (c == (struct cmd_list_element *) -1)
+      else if (c == CMD_LIST_AMBIGUOUS)
 	{
 	  /* We've gotten this far properly, but the next step is
 	     ambiguous.  We need to set the result list to the best
@@ -1346,7 +1346,7 @@ lookup_cmd (char **line, struct cmd_list
       else
 	return 0;
     }
-  else if (c == (struct cmd_list_element *) -1)
+  else if (c == CMD_LIST_AMBIGUOUS)
     {
       /* Ambigous.  Local values should be off prefixlist or called
          values.  */
@@ -1569,7 +1569,7 @@ lookup_cmd_composition (char *text,
 	  *cmd = find_cmd (command, len, cur_list, 1, &nfound);
 	}
       
-      if (*cmd == (struct cmd_list_element *) -1)
+      if (*cmd == CMD_LIST_AMBIGUOUS)
 	{
 	  return 0;              /* ambiguous */
 	}
Index: cli/cli-decode.h
===================================================================
RCS file: /cvs/src/src/gdb/cli/cli-decode.h,v
retrieving revision 1.37
diff -u -p -r1.37 cli-decode.h
--- cli/cli-decode.h	1 Jan 2011 15:33:20 -0000	1.37
+++ cli/cli-decode.h	11 Mar 2011 20:55:25 -0000
@@ -210,6 +210,9 @@ struct cmd_list_element
     struct cmd_list_element *alias_chain;
   };
 
+/* Flag for an ambiguous cmd_list result.  */
+#define CMD_LIST_AMBIGUOUS ((struct cmd_list_element *) -1)
+
 /* API to the manipulation of command lists.  */
 
 extern struct cmd_list_element *add_cmd (char *, enum command_class,

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-03-12 22:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-11 21:29 [RFA] Add constant CMD_LIST_AMBIGUOUS to replace (struct cmd_list_element *) -1 Michael Snyder
2011-03-11 21:44 ` Tom Tromey
2011-03-13  8:21   ` Michael Snyder

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox