Simple -Wshadow=local fixes
[deliverable/binutils-gdb.git] / gdb / cli / cli-utils.c
index 8a7e5d40d316c38f17ac18a8c7d72ad784d0ff08..30ee4450f93fcef967bdcb6f107c844dde4f6f29 100644 (file)
@@ -1,6 +1,6 @@
 /* CLI utilities.
 
-   Copyright (c) 2011 Free Software Foundation, Inc.
+   Copyright (C) 2011-2018 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
 
 #include "defs.h"
 #include "cli/cli-utils.h"
-#include "gdb_string.h"
 #include "value.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.
-
-   TRAILER is a character which can be found after the number; most
-   commonly this is `-'.  If you don't want a trailer, use \0.  */
+/* See documentation in cli-utils.h.  */
 
-static int
-get_number_trailer (char **pp, int trailer)
+int
+get_number_trailer (const char **pp, int trailer)
 {
   int retval = 0;      /* default */
-  char *p = *pp;
+  const char *p = *pp;
+  bool negative = false;
+
+  if (*p == '-')
+    {
+      ++p;
+      negative = true;
+    }
 
   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.\n"));
+             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;
+         const char *start = ++p;
+         LONGEST 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),
+                                      &longest_val))
+           retval = (int) longest_val;
+         else
+           {
+             printf_filtered (_("Convenience variable must "
+                                "have integer value.\n"));
+             retval = 0;
+           }
        }
     }
   else
     {
-      if (*p == '-')
-       ++p;
+      const char *p1 = p;
       while (*p >= '0' && *p <= '9')
        ++p;
-      if (p == *pp)
+      if (p == p1)
        /* There is no number here.  (e.g. "cond a == b").  */
        {
          /* Skip non-numeric token.  */
@@ -77,7 +91,7 @@ get_number_trailer (char **pp, int trailer)
          retval = 0;
        }
       else
-       retval = atoi (*pp);
+       retval = atoi (p1);
     }
   if (!(isspace (*p) || *p == '\0' || *p == trailer))
     {
@@ -88,13 +102,13 @@ get_number_trailer (char **pp, int trailer)
     }
   p = skip_spaces (p);
   *pp = p;
-  return retval;
+  return negative ? -retval : retval;
 }
 
 /* See documentation in cli-utils.h.  */
 
 int
-get_number (char **pp)
+get_number (const char **pp)
 {
   return get_number_trailer (pp, '\0');
 }
@@ -102,63 +116,130 @@ get_number (char **pp)
 /* See documentation in cli-utils.h.  */
 
 int
-get_number_or_range (char **pp)
+get_number (char **pp)
 {
-  static int last_retval, end_value;
-  static char *end_ptr;
-  static int in_range = 0;
+  int result;
+  const char *p = *pp;
+
+  result = get_number_trailer (&p, '\0');
+  *pp = (char *) p;
+  return result;
+}
 
-  if (**pp != '-')
+/* See documentation in cli-utils.h.  */
+
+number_or_range_parser::number_or_range_parser (const char *string)
+{
+  init (string);
+}
+
+/* See documentation in cli-utils.h.  */
+
+void
+number_or_range_parser::init (const char *string)
+{
+  m_cur_tok = string;
+  m_last_retval = 0;
+  m_end_value = 0;
+  m_end_ptr = NULL;
+  m_in_range = false;
+}
+
+/* See documentation in cli-utils.h.  */
+
+int
+number_or_range_parser::get_number ()
+{
+  if (m_in_range)
     {
-      /* 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 == '-')
+      /* 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 until the end of range is
+        reached.  */
+
+      if (++m_last_retval == m_end_value)
        {
-         char **temp;
+         /* End of range reached; advance token pointer.  */
+         m_cur_tok = m_end_ptr;
+         m_in_range = false;
+       }
+    }
+  else if (*m_cur_tok != '-')
+    {
+      /* Default case: state->m_cur_tok is pointing either to a solo
+        number, or to the first number of a range.  */
+      m_last_retval = get_number_trailer (&m_cur_tok, '-');
+      /* If get_number_trailer has found a -, it might be the start
+        of a command option.  So, do not parse a range if the - is
+        followed by an alpha.  */
+      if (*m_cur_tok == '-' && !isalpha (*(m_cur_tok + 1)))
+       {
+         const char **temp;
 
          /* This is the start of a range (<number1> - <number2>).
             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 = &m_end_ptr;
+         m_end_ptr = skip_spaces (m_cur_tok + 1);
+         m_end_value = ::get_number (temp);
+         if (m_end_value < m_last_retval)
            {
              error (_("inverted range"));
            }
-         else if (end_value == last_retval)
+         else if (m_end_value == m_last_retval)
            {
              /* Degenerate range (number1 == number2).  Advance the
                 token pointer so that the range will be treated as a
-                single number.  */ 
-             *pp = end_ptr;
+                single number.  */
+             m_cur_tok = m_end_ptr;
            }
          else
-           in_range = 1;
+           m_in_range = true;
        }
     }
-  else if (! in_range)
-    error (_("negative value"));
   else
     {
-      /* pp 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
-        is reached.  */
-
-      if (++last_retval == end_value)
+      if (isdigit (*(m_cur_tok + 1)))
+       error (_("negative value"));
+      if (*(m_cur_tok + 1) == '$')
        {
-         /* End of range reached; advance token pointer.  */
-         *pp = end_ptr;
-         in_range = 0;
+         /* Convenience variable.  */
+         m_last_retval = ::get_number (&m_cur_tok);
+         if (m_last_retval < 0)
+           error (_("negative value"));
        }
     }
-  return last_retval;
+  return m_last_retval;
+}
+
+/* See documentation in cli-utils.h.  */
+
+void
+number_or_range_parser::setup_range (int start_value, int end_value,
+                                    const char *end_ptr)
+{
+  gdb_assert (start_value > 0);
+
+  m_in_range = true;
+  m_end_ptr = end_ptr;
+  m_last_retval = start_value - 1;
+  m_end_value = end_value;
+}
+
+/* See documentation in cli-utils.h.  */
+
+bool
+number_or_range_parser::finished () const
+{
+  /* Parsing is finished when at end of string or null string,
+     or we are not in a range and not in front of an integer, negative
+     integer, convenience var or negative convenience var.  */
+  return (m_cur_tok == NULL || *m_cur_tok == '\0'
+         || (!m_in_range
+             && !(isdigit (*m_cur_tok) || *m_cur_tok == '$')
+             && !(*m_cur_tok == '-'
+                  && (isdigit (m_cur_tok[1]) || m_cur_tok[1] == '$'))));
 }
 
 /* Accept a number and a string-form list of numbers such as is 
@@ -170,38 +251,143 @@ get_number_or_range (char **pp)
    no arguments.  */
 
 int
-number_is_in_list (char *list, int number)
+number_is_in_list (const char *list, int number)
 {
   if (list == NULL || *list == '\0')
     return 1;
 
-  while (list != NULL && *list != '\0')
-    if (get_number_or_range (&list) == number)
-      return 1;
+  number_or_range_parser parser (list);
+
+  if (parser.finished ())
+    error (_("Arguments must be numbers or '$' variables."));
+  while (!parser.finished ())
+    {
+      int gotnum = parser.get_number ();
 
+      if (gotnum == 0)
+       error (_("Arguments must be numbers or '$' variables."));
+      if (gotnum == number)
+       return 1;
+    }
   return 0;
 }
 
 /* See documentation in cli-utils.h.  */
 
-char *
-skip_spaces (char *chp)
+const char *
+remove_trailing_whitespace (const char *start, const char *s)
+{
+  while (s > start && isspace (*(s - 1)))
+    --s;
+
+  return s;
+}
+
+/* See documentation in cli-utils.h.  */
+
+std::string
+extract_arg (const char **arg)
 {
-  if (chp == NULL)
-    return NULL;
-  while (*chp && isspace (*chp))
-    chp++;
-  return chp;
+  const char *result;
+
+  if (!*arg)
+    return std::string ();
+
+  /* Find the start of the argument.  */
+  *arg = skip_spaces (*arg);
+  if (!**arg)
+    return std::string ();
+  result = *arg;
+
+  /* Find the end of the argument.  */
+  *arg = skip_to_space (*arg + 1);
+
+  if (result == *arg)
+    return std::string ();
+
+  return std::string (result, *arg - result);
+}
+
+/* See documentation in cli-utils.h.  */
+
+std::string
+extract_arg (char **arg)
+{
+  const char *arg_const = *arg;
+  std::string result;
+
+  result = extract_arg (&arg_const);
+  *arg += arg_const - *arg;
+  return result;
 }
 
 /* See documentation in cli-utils.h.  */
 
-char *
-skip_to_space (char *chp)
+int
+check_for_argument (const char **str, const char *arg, int arg_len)
 {
-  if (chp == NULL)
-    return NULL;
-  while (*chp && !isspace (*chp))
-    chp++;
-  return chp;
+  if (strncmp (*str, arg, arg_len) == 0
+      && ((*str)[arg_len] == '\0' || isspace ((*str)[arg_len])))
+    {
+      *str += arg_len;
+      return 1;
+    }
+  return 0;
+}
+
+/* See documentation in cli-utils.h.  */
+
+int
+parse_flags (const char **str, const char *flags)
+{
+  const char *p = skip_spaces (*str);
+
+  if (p[0] == '-'
+      && isalpha (p[1])
+      && (p[2] == '\0' || isspace (p[2])))
+    {
+      const char pf = p[1];
+      const char *f = flags;
+
+      while (*f != '\0')
+       {
+         if (*f == pf)
+           {
+             *str = skip_spaces (p + 2);
+             return f - flags + 1;
+           }
+         f++;
+       }
+    }
+
+  return 0;
+}
+
+/* See documentation in cli-utils.h.  */
+
+bool
+parse_flags_qcs (const char *which_command, const char **str,
+                qcs_flags *flags)
+{
+  switch (parse_flags (str, "qcs"))
+    {
+    case 0:
+      return false;
+    case 1:
+      flags->quiet = true;
+      break;
+    case 2:
+      flags->cont = true;
+      break;
+    case 3:
+      flags->silent = true;
+      break;
+    default:
+      gdb_assert_not_reached ("int qcs flag out of bound");
+    }
+
+  if (flags->cont && flags->silent)
+    error (_("%s: -c and -s are mutually exclusive"), which_command);
+
+  return true;
 }
This page took 0.035742 seconds and 4 git commands to generate.