From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1032 invoked by alias); 17 Feb 2014 15:04:51 -0000 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 Received: (qmail 1022 invoked by uid 89); 17 Feb 2014 15:04:50 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL autolearn=no version=3.3.2 X-HELO: mtaout20.012.net.il Received: from mtaout20.012.net.il (HELO mtaout20.012.net.il) (80.179.55.166) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 17 Feb 2014 15:04:49 +0000 Received: from conversion-daemon.a-mtaout20.012.net.il by a-mtaout20.012.net.il (HyperSendmail v2007.08) id <0N1500M00B61M600@a-mtaout20.012.net.il> for gdb-patches@sourceware.org; Mon, 17 Feb 2014 17:04:42 +0200 (IST) Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout20.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0N1500MQNB7U6L80@a-mtaout20.012.net.il>; Mon, 17 Feb 2014 17:04:42 +0200 (IST) Date: Mon, 17 Feb 2014 15:04:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH 12/12] NEWS and Doc on --available-children-only In-reply-to: <5301D9F4.5010306@codesourcery.com> To: Yao Qi Cc: gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83zjlp93vz.fsf@gnu.org> References: <1392367471-13527-1-git-send-email-yao@codesourcery.com> <1392367471-13527-13-git-send-email-yao@codesourcery.com> <83ha82c9rf.fsf@gnu.org> <5301D9F4.5010306@codesourcery.com> X-IsSubscribed: yes X-SW-Source: 2014-02/txt/msg00532.txt.bz2 > Date: Mon, 17 Feb 2014 17:44:20 +0800 > From: Yao Qi > CC: > > On 02/14/2014 05:40 PM, Eli Zaretskii wrote: > > This doesn't really say much about the new option, because what > > "available children" means is not explained at all. Can you add a > > sentence about that, or, better, reword this sentence > > > > When used, only the available children are displayed. > > > > so that the second part explains which children will be displayed and > > which won't? > > IMO, "available children" is not a new term in GDB. We've already had > some in doc: > > Note that this number is not completely reliable for a dynamic varobj. > It will return the current number of children, but more children may > ^^^^^^^^^^^^ > be available. > ^^^^^^^^^^^^^ > > The @samp{numchild} field in other varobj responses is generally not > valid for a dynamic varobj -- it will show the number of children that > @value{GDBN} knows about, but because dynamic varobjs lazily > instantiate their children, this will not reflect the number of > children which may be available. > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ No, that's definitely not the same. Here, the text just says that not all children are shown, and "available" is used in its usual meaning. > "available children" means children, whose values are available, or > value available children. How about replacing "available children" > with "value available children"? Is it clear? No, sorry. "Children whose values are available" is not clear at all. Can you explain to me what makes the value "available", or what prevents it from becoming available? Then I will suggest a suitable wording. > The number of children of the varobj. This number is not necessarily > reliable for a dynamic varobj. Instead, you must examine the > -@samp{has_more} attribute. > +@samp{has_more} attribute. If the @samp{--available-children-only} > +option was specified, this number is not reliable either. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ "this number is not reliable for any varobj". Please also make similar corrections in other places. The rest of the patch is OK, so we are only left with describing what are the "value available children". Thanks.