From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30468 invoked by alias); 20 Feb 2012 17:11:49 -0000 Received: (qmail 30458 invoked by uid 22791); 20 Feb 2012 17:11:47 -0000 X-SWARE-Spam-Status: No, hits=-1.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout22.012.net.il (HELO mtaout22.012.net.il) (80.179.55.172) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 20 Feb 2012 17:11:08 +0000 Received: from conversion-daemon.a-mtaout22.012.net.il by a-mtaout22.012.net.il (HyperSendmail v2007.08) id <0LZP00000BP6RO00@a-mtaout22.012.net.il> for gdb-patches@sourceware.org; Mon, 20 Feb 2012 19:11:06 +0200 (IST) Received: from HOME-C4E4A596F7 ([77.124.150.51]) by a-mtaout22.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0LZP00MZ4BQG83Q0@a-mtaout22.012.net.il>; Mon, 20 Feb 2012 19:11:05 +0200 (IST) Date: Mon, 20 Feb 2012 19:24:00 -0000 From: Eli Zaretskii Subject: Re: [RFA take 5] Allow setting breakpoints on inline functions (PR 10738) In-reply-to: <20120220155848.GA5813@redhat.com> To: Gary Benson Cc: gdb-patches@sourceware.org, mark@klomp.org, jan.kratochvil@redhat.com, dje@google.com Reply-to: Eli Zaretskii Message-id: <83wr7htosv.fsf@gnu.org> References: <20120220155848.GA5813@redhat.com> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-02/txt/msg00409.txt.bz2 > Date: Mon, 20 Feb 2012 15:58:49 +0000 > From: Gary Benson > Cc: Eli Zaretskii , Mark Wielaard , > Jan Kratochvil , > Doug Evans > > This patch makes GDB able to set breakpoints on inlined functions. Thanks. > I have updated the documentation part of the patch to retain all > information about older versions, and also to document the new > option. > > How does it look? Looks good, but I have a couple of minor comments. > --- a/gdb/NEWS > +++ b/gdb/NEWS > @@ -44,6 +44,8 @@ > * The command "info catch" has been removed. It has been disabled > since December 2007. > > +* GDB can now set breakpoints on inlined functions. > + I think we should also advertise the new command-line option in NEWS. > +@item -allow-incomplete-gdb-indexes > +@cindex @code{--allow-incomplete-gdb-indexes} > +This option causes @value{GDBN} to read and use possibly incomplete > +@samp{.gdb_index} sections in symbol files. This can speed up > +startup, but may result in some functionality being lost. Please add here a cross-reference to the node "Index Section Format", because the effect of this option is not fully understood until the reader reads there. The documentation parts are OK, with those comments taken care of.