From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17371 invoked by alias); 30 Aug 2002 18:53:47 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 17326 invoked from network); 30 Aug 2002 18:53:46 -0000 Received: from unknown (HELO takamaka.act-europe.fr) (205.232.38.110) by sources.redhat.com with SMTP; 30 Aug 2002 18:53:46 -0000 Received: by takamaka.act-europe.fr (Postfix, from userid 507) id 74316D2CC1; Fri, 30 Aug 2002 11:53:46 -0700 (PDT) Date: Fri, 30 Aug 2002 12:06:00 -0000 From: Joel Brobecker To: Kevin Buettner Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA/controversial] move bp by line number past the prologue Message-ID: <20020830185346.GE1707@gnat.com> References: <20020829181524.GC971@gnat.com> <1020829201635.ZM24274@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1020829201635.ZM24274@localhost.localdomain> User-Agent: Mutt/1.4i X-SW-Source: 2002-08/txt/msg01040.txt.bz2 On Thu, Aug 29, 2002 at 01:16:35PM -0700, Kevin Buettner wrote: > My opinion is that this is a reasonable change providing that we > still have a mechanism for setting a breakpoint on the prologue. > When I want to do this I usually do ``b *foo'' instead of relying > on line numbers. (Usually I don't know the line number anyway.) I completely agree. > I've reread the thread in which this matter was discussed earlier > (i.e, the "[RFC] breakpoints and function prologues" messages). Given > that there was such disagreement before, I doubt that everyone will > agree with my point of view. If this turns out to be the case, might > I suggest a user settable option for controlling whether setting a > breakpoint by line number will always cause the prologue to be > skipped? That way both sides get the behavior they want. (GUIs can > set the desired behavior at initialization time.) How about this change? Any advice on the option name and description would be appreciated. With this patch, the 5 regressions I was mentioning in my previous e-mail can be cleared by turning this new behavior off (there is also one other expected regression in help.exp, since the output of "help breakpoint" is modified by this patch). Before sending a patch for these regressions, I prefer to wait for comments regarding this patch. 2002-08-30 Jim Ingham * linespec.c (decode_line_1): Skip the function prologue if funfirstline is set, unless adjust_break_after_prologue is unset by the user. Changes the behavior of the break command to skip the function prologue when breaking by line number. 2002-08-30 Joel Brobecker * linespec.c (adjust_break_after_prologue): New static variable. (_initialize_linespec): New function. * Makefile.in: Update the dependencies for linespec.c -- Joel