Re: [RFA] More linespec quoting "fixes"

[Keith Seitz <keiths at redhat dot com>]

On 02/24/2010 02:10 PM, Keith Seitz wrote:

> * linespec.c (decode_line_1): Update comments for is_quote_enclosed.
> If the filename portion of the linespec was quoted, recheck the
> remainder for additional quoting.
> (locate_first_half): Skip over completer chars, too.

And, of course, there is a small typo in the patch... locate_first_half 
should be:

@@ -1056,7 +1066,14 @@ locate_first_half (char **argptr, int *i
       p++;
     }
   else
-    *is_quote_enclosed = 0;
+    {
+      *is_quote_enclosed = 0;
+      if (strchr (get_gdb_completer_quote_characters (), *p))
+	{
+	  ++(*argptr);
+	  ++p;
+	}
+    }
   for (; *p; p++)
     {
       if (p[0] == '<')

Sorry about that. FWIW, I've attached the corrected patch as well.

Keith

Attachment: more-linespec-quoting.patch
Description: Text document