* linespec.c (is_objc_method_format): New function

(decode_line_1, locate_first_half): Use it.
Fixes PR objc/1298

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 360026e12404b62a3826374dc8ef96d5e2302e33..6bc9468ba800c8e1bdcb0c14dece980c1bf68a3f 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2003-08-02  Adam Fedor  <[email protected]>
+
+       * linespec.c (is_objc_method_format): New function
+       (decode_line_1, locate_first_half): Use it.
+       Fixes PR objc/1298
+

 2003-08-01  Andrew Cagney  <[email protected]>
 
        * NEWS: Mention that m32r is multi-arch.
 2003-08-01  Andrew Cagney  <[email protected]>
 
        * NEWS: Mention that m32r is multi-arch.

diff --git a/gdb/linespec.c b/gdb/linespec.c
index 4b863ef8f745dcd656d43b1892eec25cc9a22937..7e8c6a7c9297e6dc458c97bc83981ab2c5d44b7e 100644 (file)
--- a/gdb/linespec.c
+++ b/gdb/linespec.c
@@ -94,6 +94,8 @@ static void build_canonical_line_spec (struct symtab_and_line *,
 
 static char *find_toplevel_char (char *s, char c);
 
 
 static char *find_toplevel_char (char *s, char c);
 

+static int is_objc_method_format (const char *s);
+

 static struct symtabs_and_lines decode_line_2 (struct symbol *[],
                                               int, int, char ***);
 
 static struct symtabs_and_lines decode_line_2 (struct symbol *[],
                                               int, int, char ***);
 


@@ -443,6 +445,25 @@ find_toplevel_char (char *s, char c)
   return 0;
 }
 
   return 0;
 }
 

+/* Determines if the gives string corresponds to an Objective-C method
+   representation, such as -[Foo bar:] or +[Foo bar]. Objective-C symbols
+   are allowed to have spaces and parentheses in them.  */
+
+static int 
+is_objc_method_format (const char *s)
+{
+  if (s == NULL || *s == '\0')
+    return 0;
+  /* Handle arguments with the format FILENAME:SYMBOL.  */
+  if ((s[0] == ':') && (strchr ("+-", s[1]) != NULL) 
+      && (s[2] == '[') && strchr(s, ']'))
+    return 1;
+  /* Handle arguments that are just SYMBOL.  */
+  else if ((strchr ("+-", s[0]) != NULL) && (s[1] == '[') && strchr(s, ']'))
+    return 1;
+  return 0;
+}
+

 /* Given a list of NELTS symbols in SYM_ARR, return a list of lines to
    operate on (ask user if necessary).
    If CANONICAL is non-NULL return a corresponding array of mangled names
 /* Given a list of NELTS symbols in SYM_ARR, return a list of lines to
    operate on (ask user if necessary).
    If CANONICAL is non-NULL return a corresponding array of mangled names


@@ -669,8 +690,7 @@ decode_line_1 (char **argptr, int funfirstline, struct symtab *default_symtab,
 
   /* Check if this is an Objective-C method (anything that starts with
      a '+' or '-' and a '[').  */
 
   /* Check if this is an Objective-C method (anything that starts with
      a '+' or '-' and a '[').  */

-  if (*p && (p[0] == ':') && (strchr ("+-", p[1]) != NULL) 
-      && (p[2] == '['))
+  if (is_objc_method_format (p))

     {
       is_objc_method = 1;
       paren_pointer  = NULL; /* Just a category name.  Ignore it.  */
     {
       is_objc_method = 1;
       paren_pointer  = NULL; /* Just a category name.  Ignore it.  */


@@ -972,8 +992,7 @@ locate_first_half (char **argptr, int *is_quote_enclosed)
        }
       /* Check for a colon and a plus or minus and a [ (which
          indicates an Objective-C method) */
        }
       /* Check for a colon and a plus or minus and a [ (which
          indicates an Objective-C method) */

-      if (*p && (p[0] == ':') && (strchr ("+-", p[1]) != NULL) 
-         && (p[2] == '['))
+      if (is_objc_method_format (p))

        {
          break;
        }
        {
          break;
        }