From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-92474-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 24354 invoked by alias); 26 Jun 2012 16:26:51 -0000
Received: (qmail 24051 invoked by uid 22791); 26 Jun 2012 16:26:49 -0000
X-SWARE-Spam-Status: No, hits=-3.9 required=5.0	tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_NO,RCVD_IN_NIX_SPAM,SPF_SOFTFAIL
X-Spam-Check-By: sourceware.org
Received: from mtaout20.012.net.il (HELO mtaout20.012.net.il) (80.179.55.166)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 26 Jun 2012 16:26:31 +0000
Received: from conversion-daemon.a-mtaout20.012.net.il by a-mtaout20.012.net.il (HyperSendmail v2007.08) id <0M6800400G7Y4500@a-mtaout20.012.net.il> for gdb-patches@sourceware.org; Tue, 26 Jun 2012 19:26:05 +0300 (IDT)
Received: from HOME-C4E4A596F7 ([87.69.210.75]) by a-mtaout20.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0M68003V6GBGLTB0@a-mtaout20.012.net.il>; Tue, 26 Jun 2012 19:26:05 +0300 (IDT)
Date: Tue, 26 Jun 2012 16:26:00 -0000
From: Eli Zaretskii <eliz@gnu.org>
Subject: Re: [doc RFA] "set debug dwarf2-read" and "set debug symtab-create"
In-reply-to: <CADPb22SzBXCRCufriTYyj=ODfmtH-d=TzA-k8dogFizgFNhK0w@mail.gmail.com>
To: Doug Evans <dje@google.com>
Cc: gdb-patches@sourceware.org
Reply-to: Eli Zaretskii <eliz@gnu.org>
Message-id: <83zk7q82fr.fsf@gnu.org>
MIME-version: 1.0
Content-type: text/plain; charset=iso-8859-1
Content-transfer-encoding: 8BIT
References: <20120619222645.36C911E26C4@ruffy2.mtv.corp.google.com> <CADPb22SzBXCRCufriTYyj=ODfmtH-d=TzA-k8dogFizgFNhK0w@mail.gmail.com>
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2012-06/txt/msg00807.txt.bz2

> Date: Tue, 26 Jun 2012 08:45:04 -0700
> From: Doug Evans <dje@google.com>
> Cc: gdb-patches <gdb-patches@sourceware.org>
> 
> Hi.  "Ping." :-)

Sorry, but your wording was ambiguous:

> > I'm committing this after a doc RFA.

I interpreted that as meaning "doc review was already done, I'm
committing this".

> --- a/gdb/NEWS
> +++ b/gdb/NEWS
> @@ -462,6 +462,16 @@ tstop [NOTES]
>
>  * New options
>
> +set debug dwarf2-read
> +show debug dwarf2-read
> +  Turns on or off display of GDB info related to reading DWARF debugging
> +  information.  The default is off.

I'd prefer to say "display of debugging messages related to reading
DWARF debug info".

> +@item set debug dwarf2-read
> +@cindex DWARF2 Reading
> +Turns on or off display of @value{GDBN} info related to reading
> +DWARF debugging information.  The default is off.
> +@item show debug dwarf2-read
> +Show the current state of DWARF2 reader debugging.

Same here.

> +@item set debug symtab-create
> +@cindex Symbol Table Creation

Index entries should start with a lower-case letter.  I would simply
lower-case all the words in this entry.

OK with those changes.

Thanks, and sorry for the delay.