From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id 7ZMsC+7gx2Qn1DMAWB0awg (envelope-from ) for ; Mon, 31 Jul 2023 12:27:26 -0400 Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=r+cvcqig; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id 246271E0C0; Mon, 31 Jul 2023 12:27:26 -0400 (EDT) Received: from server2.sourceware.org (ip-8-43-85-97.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 046861E092 for ; Mon, 31 Jul 2023 12:27:24 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id BC1663858414 for ; Mon, 31 Jul 2023 16:27:22 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org BC1663858414 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1690820842; bh=eJ5uTtwpLLOixjGcfmdkjH3tKp+2RNJbK9uEW6ugl8Q=; h=To:Cc:Subject:References:Date:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=r+cvcqigyXtTHUC+mMXQU9zHW0KcBvIRqxtKayyvlxCxQuY5SajVuaWQn6AIQ9u0H CvWMvd9pSvAtOEmAkdoA+CTAmOeDxfRdl28PzPqE3PrYmbkLQDUap6C5Nqc094pkYi KXxbYYhp3hYyQYKr4p89SMVBsnpwbQLGzKpt8Nbk= Received: from mail-il1-x130.google.com (mail-il1-x130.google.com [IPv6:2607:f8b0:4864:20::130]) by sourceware.org (Postfix) with ESMTPS id 242463858CD1 for ; Mon, 31 Jul 2023 16:27:00 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 242463858CD1 Received: by mail-il1-x130.google.com with SMTP id e9e14a558f8ab-34928e9f24bso2271115ab.0 for ; Mon, 31 Jul 2023 09:27:00 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690820819; x=1691425619; h=mime-version:user-agent:message-id:in-reply-to:date:references :subject:cc:to:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=eJ5uTtwpLLOixjGcfmdkjH3tKp+2RNJbK9uEW6ugl8Q=; b=T5QkbcCEfaff4UH9l5yxpq19e/42Yw1dwkHzw6kCVICu1qZ4qvUEmLTMtVYJNVK2x7 4eVIB/Cf4ugpD7tfRBmLsMfq50GIdY1QvhK3HkeG6FAZawPCSn7dS10SIw1vnDwdvo05 V+Z9GLGifh54CuR/MNkf8w9P0CjVqtQQRpkWdz8Ny3NBIelJ46RzoxBE/kXt1sacIyuJ 6fwZry06CCkc0F2IA26wOy/0o73WoxcIrW7HPD8vUn50Zhp0trUo+oBHWE2rlMV1wY1p FrLUENHdI893wKvGceGVpL792Nt3YUi4OtpydgevxGFjNAKr84FstFvs/rFGGqRbmMB+ 0HcQ== X-Gm-Message-State: ABy/qLZTZeyYidf1IvSdAiGL7igXSngCd+5vKI5aimZUfnPiSeMIEwMj Ukyu9y9RJTcShl1AEZWk+WUIgQ== X-Google-Smtp-Source: APBJJlFlGXvfPye/Cv3H14rx+/usXbQwKq2twxG7TH8zy+mqyd6UWf1ArseUyVdXYgGLCpd/wROMQA== X-Received: by 2002:a92:c26d:0:b0:345:8373:bf68 with SMTP id h13-20020a92c26d000000b003458373bf68mr9161114ild.25.1690820818870; Mon, 31 Jul 2023 09:26:58 -0700 (PDT) Received: from murgatroyd (75-166-135-140.hlrn.qwest.net. [75.166.135.140]) by smtp.gmail.com with ESMTPSA id q17-20020a92c011000000b00345b34fcf34sm3272964ild.38.2023.07.31.09.26.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 31 Jul 2023 09:26:58 -0700 (PDT) To: Lancelot SIX Cc: Tom Tromey , gdb-patches@sourceware.org Subject: Re: [PATCH 4/6] Full paths in DAP stackTrace responses References: <20230725-dap-bt-path-v1-0-bb015b0d8e54@adacore.com> <20230725-dap-bt-path-v1-4-bb015b0d8e54@adacore.com> <20230728151539.qnrgzb37fub34tke@octopus> X-Attribution: Tom Date: Mon, 31 Jul 2023 10:26:57 -0600 In-Reply-To: <20230728151539.qnrgzb37fub34tke@octopus> (Lancelot SIX's message of "Fri, 28 Jul 2023 16:29:08 +0100") Message-ID: <878rawgi7i.fsf@tromey.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-5.5 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Tom Tromey via Gdb-patches Reply-To: Tom Tromey Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" >> +class FrameDecorator(_FrameDecoratorBase): >> + """Basic implementation of a Frame Decorator""" >> + Lancelot> I can see that this pre-existed your patch, but is it expected to have Lancelot> two docstrings instead of one with empty lines to separate paragraphs? Nope, I've fixed it. >> @@ -172,7 +172,11 @@ def _frame_iterator(frame, frame_low, frame_high, always): >> >> # Apply a basic frame decorator to all gdb.Frames. This unifies >> # the interface. >> - frame_iterator = map(FrameDecorator, frame_iterator) >> + if always: >> + decorator = DAPFrameDecorator >> + else: >> + decorator = FrameDecorator >> + frame_iterator = map(decorator, frame_iterator) Lancelot> The purpose of always (always return an iterator) seems orthogonal to Lancelot> the way `filename()` behaves. I find this confusing. Should the Lancelot> parameter be renamed / a new parameter added? I renamed it to "dap_semantics". Tom