From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27420 invoked by alias); 13 Jul 2018 07:23:19 -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 27410 invoked by uid 89); 13 Jul 2018 07:23:19 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-8.7 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_1,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy=Zaretskii, zaretskii, HContent-Transfer-Encoding:8bit X-HELO: mailsec114.isp.belgacom.be Received: from mailsec114.isp.belgacom.be (HELO mailsec114.isp.belgacom.be) (195.238.20.110) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 13 Jul 2018 07:23:15 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=skynet.be; i=@skynet.be; q=dns/txt; s=securemail; t=1531466595; x=1563002595; h=message-id:subject:from:to:cc:date:in-reply-to: references:mime-version:content-transfer-encoding; bh=Ma6fheK3BHqxBrJlZZCM60r65Ngs+5R+AxD5qE3AfIo=; b=LwYsrPZo/DmSrSBmBMh22VIUQVPETyUWxSulzxbQcFZuYIFT10lk4vPs qKhLh+rgGsEKoaE0OsN3EtLI9MW5RQ==; Received: from 160.177-177-91.adsl-dyn.isp.belgacom.be (HELO md) ([91.177.177.160]) by relay.skynet.be with ESMTP/TLS/AES256-GCM-SHA384; 13 Jul 2018 09:23:13 +0200 Message-ID: <1531466593.1737.1.camel@skynet.be> Subject: Re: [RFA] (try to) consistently use 'frame level' concept instead of 'frame number'. From: Philippe Waroquiers To: Eli Zaretskii Cc: gdb-patches@sourceware.org Date: Fri, 13 Jul 2018 07:23:00 -0000 In-Reply-To: <83fu0neifl.fsf@gnu.org> References: <20180712221536.26845-1-philippe.waroquiers@skynet.be> <83fu0neifl.fsf@gnu.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2018-07/txt/msg00377.txt.bz2 On Fri, 2018-07-13 at 09:52 +0300, Eli Zaretskii wrote: > > From: Philippe Waroquiers > > Cc: Philippe Waroquiers > > Date: Fri, 13 Jul 2018 00:15:36 +0200 > > > > Following the discussion in the 'frame apply' patch and the patch proposed > > in https://sourceware.org/ml/gdb-patches/2018-06/msg00170.html > > the idea is rather to speak about 'frame level' to identify a frame, > > rather than 'frame number'. > > I question the wisdom of changing such veteran terminology. Yes, for sure, frame number was used since quite some time. But the concept of frame level was also already used, and IMO there is quite some confusions between the 2. For example, in @node GDB/MI Frame Information. We also have some places that are defining frame number as being the level e.g.-stack-list-frames. And IMO, at other places, level is used where it should rather be number or count. E.g. 'set backtrace limit N' 'set backtrace limit 0' 'set backtrace limit unlimited'      Limit the backtrace to N levels.  A value of 'unlimited' or zero      means unlimited levels. looks somewhat confusing, and should rather be limit the backtrace to N entries, or N frames, or whatever but probably not N levels. To be further discussed it looks like ... Thanks Philippe > > > -@cindex frame number > > -@value{GDBN} assigns numbers to all existing stack frames, starting with > > +@cindex frame level > > +In @value{GDBN}, each existing stack frame has a level, starting with > > zero for the innermost frame, one for the frame that called it, > > -and so on upward. These numbers do not really exist in your program; > > -they are assigned by @value{GDBN} to give you a way of designating stack > > +and so on upward. These levels give you a way of designating stack > > frames in @value{GDBN} commands. > > If we are going to make this change, then I would suggest to keep the > index entry, _add_ to it an entry about "frame level", and explain > here what that level is, something like this: > > @value{GDBN} labels each existing stack frame with a @dfn{level}, a > number that is zero for the innermost frame, one for the frame that > called it, and so on upward. These level numbers give you a way of > designating stack frames in @value{GDBN} commands. > > > -it had a separate frame, which is numbered zero as usual, allowing > > +it had a separate frame, which has a level zero as usual, allowing > > ^^^^^^^^^^ > "level of zero". > > > -Select frame number @var{n}. Recall that frame zero is the innermost > > +Select frame level @var{n}. Recall that frame zero is the innermost > > ^^^^^^^^^^^^^^^^^^^^^^^^^^ > "Select frame whose level is @var{n}." > > Thanks.