From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 103478 invoked by alias); 22 Jul 2015 14:26:06 -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 103468 invoked by uid 89); 22 Jul 2015 14:26:06 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=AWL,BAYES_00,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-yk0-f178.google.com Received: from mail-yk0-f178.google.com (HELO mail-yk0-f178.google.com) (209.85.160.178) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Wed, 22 Jul 2015 14:26:04 +0000 Received: by ykdu72 with SMTP id u72so193672824ykd.2 for ; Wed, 22 Jul 2015 07:26:02 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.13.237.1 with SMTP id w1mr2738391ywe.132.1437575162812; Wed, 22 Jul 2015 07:26:02 -0700 (PDT) Received: by 10.13.233.198 with HTTP; Wed, 22 Jul 2015 07:26:02 -0700 (PDT) In-Reply-To: <55AF5F7E.5000600@adacore.com> References: <54F47563.4050103@adacore.com> <54FF0D05.70907@redhat.com> <550C1170.9070208@adacore.com> <55685B60.3000004@redhat.com> <55775EB0.4080701@adacore.com> <55AF5F7E.5000600@adacore.com> Date: Wed, 22 Jul 2015 14:26:00 -0000 Message-ID: Subject: Re: [PATCH] Add proper handling for non-local references in nested functions From: Doug Evans To: Pierre-Marie de Rodat Cc: Pedro Alves , GDB Patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-07/txt/msg00612.txt.bz2 On Wed, Jul 22, 2015 at 2:16 AM, Pierre-Marie de Rodat wrote: > On 06/09/2015 11:46 PM, Pierre-Marie de Rodat wrote: >> >> On 05/29/2015 02:28 PM, Pedro Alves wrote: >>>> >>>> This would look cleaner indeed. It's a big change itself though so if >>>> most consider this as a good idea I don't mind doing it... although it >>>> would be for another commit! >>> >>> >>> I would think it great if someone did that. :-) >> >> >> Okay... I may give it a try, then. ;-) > > > Here it is! I > just rebased my work on non-local references on top of this cleanup and > performed the changes you asked me to do. Just a question: > >>> It'd be great if you could skim over the patch add any missing >>> function intro comments. You've already done a good job at that, >>> I think only here and there missed it. > > > What I usually do is to put comments in front of function definitions and > leave function declarations without them in the header. It's generally what > I observe in the sources, but since sometimes the documentation is in the > header file, it happens that I do the same: I try to stay consistent with > nearby code. ;-) Please tell me if you want me to do something different. > > Regtested again on x86_64-linux: no regression. Ok to push? Thank you! Hi. Ditto. Others may approve this, but give me a chance to review it too. One thought that comes to mind when reading the patch is that you introduce the term "static link", and it doesn't mean what the casual reader will think it means. E.g., + This method is designed to work with static links (nested functions + handling). Static links are function properties whose evaluation return + the frame base address for the enclosing frame. I think we need something less ambiguous / more clear.