From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-15254-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 28097 invoked by alias); 9 May 2002 22:53:56 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 28090 invoked from network); 9 May 2002 22:53:55 -0000
Received: from unknown (HELO cygnus.com) (205.180.83.203)
  by sources.redhat.com with SMTP; 9 May 2002 22:53:55 -0000
Received: from redhat.com (reddwarf.sfbay.redhat.com [172.16.24.50])
	by runyon.cygnus.com (8.8.7-cygnus/8.8.7) with ESMTP id PAA12443;
	Thu, 9 May 2002 15:53:52 -0700 (PDT)
Message-ID: <3CDAFAEA.7C4E5207@redhat.com>
Date: Thu, 09 May 2002 15:53:00 -0000
From: Michael Snyder <msnyder@redhat.com>
Organization: Red Hat, Inc.
X-Accept-Language: en
MIME-Version: 1.0
To: Daniel Jacobowitz <drow@mvista.com>
CC: gdb-patches@sources.redhat.com
Subject: Re: [RFA] Fix mi-break.exp: 'b "basics.c":16'
References: <20020402192959.A17349@nevyn.them.org>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
X-SW-Source: 2002-05/txt/msg00302.txt.bz2

Daniel Jacobowitz wrote:
> 
> [I dislike decode_line_1.  But that seems to be the general consensus...]
> 
> There was a logic error in the code to handle "file":line.  Several,
> actually.  I suspect that it once worked and had bitrotten as the behavior
> of the function changed.  For instance, at the beginning of the function:
> 
>   if (p[0] == '"')
>     {
>       is_quote_enclosed = 1;
>       (*argptr)++;
>       p++;
>     }
> 
> Then below it checked 'is_quote_enclosed && (**argptr) == '"'). 

[line break inserted]

> That'll only be true given a literal '"":<line>', which was not 
> the intent of the test.

I don't understand this statement.  Between the two bits of text
you've named, both p and *argptr are changed.  Seems to me, there 
could be any number of characters between the two quotes.


> This patch updates the behavior, should change nothing else, and causes no
> regressions.  OK to check in?
> 
> --
> Daniel Jacobowitz                           Carnegie Mellon University
> MontaVista Software                         Debian GNU/Linux Developer
> 
> 2002-04-02  Daniel Jacobowitz  <drow@mvista.com>
> 
>         * linespec.c (decode_line_1): Check for a double quote after
>         a filename correctly.
> 
> Index: linespec.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/linespec.c,v
> retrieving revision 1.17
> diff -u -p -r1.17 linespec.c
> --- linespec.c  2002/03/22 18:57:07     1.17
> +++ linespec.c  2002/04/03 00:19:30
> @@ -929,20 +929,12 @@ decode_line_1 (char **argptr, int funfir
>        if ((*p == '"') && is_quote_enclosed)
>         --p;
>        copy = (char *) alloca (p - *argptr + 1);
> -      if ((**argptr == '"') && is_quote_enclosed)
> -       {
> -         memcpy (copy, *argptr + 1, p - *argptr - 1);
> -         /* It may have the ending quote right after the file name */
> -         if (copy[p - *argptr - 2] == '"')
> -           copy[p - *argptr - 2] = 0;
> -         else
> -           copy[p - *argptr - 1] = 0;
> -       }
> +      memcpy (copy, *argptr, p - *argptr);
> +      /* It may have the ending quote right after the file name */
> +      if (is_quote_enclosed && copy[p - *argptr - 1] == '"')
> +       copy[p - *argptr - 1] = 0;
>        else
> -       {
> -         memcpy (copy, *argptr, p - *argptr);
> -         copy[p - *argptr] = 0;
> -       }
> +       copy[p - *argptr] = 0;
> 
>        /* Find that file's data.  */
>        s = lookup_symtab (copy);