2011-03-11 Michael Snyder <msnyder@vmware.com>
[deliverable/binutils-gdb.git] / gdb / cli / cli-utils.c
index 0c98ca9eb0f06f853609e17951d42deef379678d..2cce06897a10f6a1a3bcc5579501fdd4e221c8e6 100644 (file)
 #include "cli/cli-utils.h"
 #include "gdb_string.h"
 #include "value.h"
+#include "gdb_assert.h"
 
 #include <ctype.h>
 
 /* *PP is a string denoting a number.  Get the number of the.  Advance
    *PP after the string and any trailing whitespace.
 
-   Currently the string can either be a number or "$" followed by the
-   name of a convenience variable.
+   Currently the string can either be a number, or "$" followed by the
+   name of a convenience variable, or ("$" or "$$") followed by digits.
 
    TRAILER is a character which can be found after the number; most
    commonly this is `-'.  If you don't want a trailer, use \0.  */
@@ -41,24 +42,39 @@ get_number_trailer (char **pp, int trailer)
 
   if (*p == '$')
     {
-      /* Make a copy of the name, so we can null-terminate it
-         to pass to lookup_internalvar().  */
-      char *varname;
-      char *start = ++p;
-      LONGEST val;
-
-      while (isalnum (*p) || *p == '_')
-       p++;
-      varname = (char *) alloca (p - start + 1);
-      strncpy (varname, start, p - start);
-      varname[p - start] = '\0';
-      if (get_internalvar_integer (lookup_internalvar (varname), &val))
-       retval = (int) val;
-      else
+      struct value *val = value_from_history_ref (p, &p);
+
+      if (val) /* Value history reference */
        {
-         printf_filtered (_("Convenience variable must "
-                            "have integer value.\n"));
-         retval = 0;
+         if (TYPE_CODE (value_type (val)) == TYPE_CODE_INT)
+           retval = value_as_long (val);
+         else
+           {
+             printf_filtered (_("History value must have integer type."));
+             retval = 0;
+           }
+       }
+      else     /* Convenience variable */
+       {
+         /* Internal variable.  Make a copy of the name, so we can
+            null-terminate it to pass to lookup_internalvar().  */
+         char *varname;
+         char *start = ++p;
+         LONGEST val;
+
+         while (isalnum (*p) || *p == '_')
+           p++;
+         varname = (char *) alloca (p - start + 1);
+         strncpy (varname, start, p - start);
+         varname[p - start] = '\0';
+         if (get_internalvar_integer (lookup_internalvar (varname), &val))
+           retval = (int) val;
+         else
+           {
+             printf_filtered (_("Convenience variable must "
+                                "have integer value.\n"));
+             retval = 0;
+           }
        }
     }
   else
@@ -101,19 +117,25 @@ get_number (char **pp)
 
 /* See documentation in cli-utils.h.  */
 
-int
-get_number_or_range (char **pp)
+void
+init_number_or_range (struct get_number_or_range_state *state,
+                     char *string)
 {
-  static int last_retval, end_value;
-  static char *end_ptr;
-  static int in_range = 0;
+  memset (state, 0, sizeof (*state));
+  state->string = string;
+}
+
+/* See documentation in cli-utils.h.  */
 
-  if (**pp != '-')
+int
+get_number_or_range (struct get_number_or_range_state *state)
+{
+  if (*state->string != '-')
     {
-      /* Default case: pp is pointing either to a solo number, 
-        or to the first number of a range.  */
-      last_retval = get_number_trailer (pp, '-');
-      if (**pp == '-')
+      /* Default case: state->string is pointing either to a solo
+        number, or to the first number of a range.  */
+      state->last_retval = get_number_trailer (&state->string, '-');
+      if (*state->string == '-')
        {
          char **temp;
 
@@ -121,44 +143,72 @@ get_number_or_range (char **pp)
             Skip the '-', parse and remember the second number,
             and also remember the end of the final token.  */
 
-         temp = &end_ptr; 
-         end_ptr = *pp + 1; 
-         while (isspace ((int) *end_ptr))
-           end_ptr++;  /* skip white space */
-         end_value = get_number (temp);
-         if (end_value < last_retval) 
+         temp = &state->end_ptr; 
+         state->end_ptr = skip_spaces (state->string + 1);
+         state->end_value = get_number (temp);
+         if (state->end_value < state->last_retval) 
            {
              error (_("inverted range"));
            }
-         else if (end_value == last_retval)
+         else if (state->end_value == state->last_retval)
            {
              /* Degenerate range (number1 == number2).  Advance the
                 token pointer so that the range will be treated as a
                 single number.  */ 
-             *pp = end_ptr;
+             state->string = state->end_ptr;
            }
          else
-           in_range = 1;
+           state->in_range = 1;
        }
     }
-  else if (! in_range)
+  else if (! state->in_range)
     error (_("negative value"));
   else
     {
-      /* pp points to the '-' that betokens a range.  All
+      /* state->string points to the '-' that betokens a range.  All
         number-parsing has already been done.  Return the next
         integer value (one greater than the saved previous value).
-        Do not advance the token pointer 'pp' until the end of range
+        Do not advance the token pointer until the end of range
         is reached.  */
 
-      if (++last_retval == end_value)
+      if (++state->last_retval == state->end_value)
        {
          /* End of range reached; advance token pointer.  */
-         *pp = end_ptr;
-         in_range = 0;
+         state->string = state->end_ptr;
+         state->in_range = 0;
        }
     }
-  return last_retval;
+  state->finished = *state->string == '\0';
+  return state->last_retval;
+}
+
+/* Accept a number and a string-form list of numbers such as is 
+   accepted by get_number_or_range.  Return TRUE if the number is
+   in the list.
+
+   By definition, an empty list includes all numbers.  This is to 
+   be interpreted as typing a command such as "delete break" with 
+   no arguments.  */
+
+int
+number_is_in_list (char *list, int number)
+{
+  struct get_number_or_range_state state;
+
+  if (list == NULL || *list == '\0')
+    return 1;
+
+  init_number_or_range (&state, list);
+  while (!state.finished)
+    {
+      int gotnum = get_number_or_range (&state);
+
+      if (gotnum == 0)
+       error (_("Args must be numbers or '$' variables."));
+      if (gotnum == number)
+       return 1;
+    }
+  return 0;
 }
 
 /* See documentation in cli-utils.h.  */
This page took 0.02712 seconds and 4 git commands to generate.