From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lndn.lancelotsix.com (vps-42846194.vps.ovh.net [IPv6:2001:41d0:801:2000::2400]) by sourceware.org (Postfix) with ESMTPS id D0E4A3858C2C for ; Thu, 30 Sep 2021 23:16:04 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org D0E4A3858C2C Received: from ubuntu.lan (unknown [IPv6:2a02:390:9086::635]) by lndn.lancelotsix.com (Postfix) with ESMTPSA id AAC9B80D76; Thu, 30 Sep 2021 23:16:03 +0000 (UTC) Date: Thu, 30 Sep 2021 23:15:58 +0000 From: Lancelot SIX To: Simon Marchi Cc: gdb-patches@sourceware.org Subject: Re: [PATCH v4 1/4] gdb: Introduce setting construct within cmd_list_element Message-ID: <20210930231558.3ka4vglabsdq4ern@ubuntu.lan> References: <20210929215011.1489639-1-lsix@lancelotsix.com> <20210929215011.1489639-2-lsix@lancelotsix.com> <90e82ecb-d1ab-d866-ccdb-b6ab64e52f5e@polymtl.ca> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.11 (lndn.lancelotsix.com [0.0.0.0]); Thu, 30 Sep 2021 23:16:03 +0000 (UTC) X-Spam-Status: No, score=-11.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) 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: , X-List-Received-Date: Thu, 30 Sep 2021 23:16:08 -0000 > > I think the change below should fix it. Hi, Sorry for this. I did rebase just few days ago and let this one slip through. I’ll fix this and address your other comments shortly. > > Since it's one of those > settings that reverts the value if the new value is invalid, it could > eventually be changed to use a getter/setter, so that the setter simply > rejects the new value if it's not valid. I’ll do that in separate commit once this series is pushed if this ok with you. For the moment I’ll just fix the current compile error with something similar to what you propose bellow. Thanks a lot for the review. Best, Lancelot. > > > From 96df63a390451e0bbdfc74784ea09a3aa360656d Mon Sep 17 00:00:00 2001 > From: Simon Marchi > Date: Thu, 30 Sep 2021 08:26:41 -0400 > Subject: [PATCH] fix > > Change-Id: Id78157cacf2de9cfacba1f19dea17b0f7b65aa0f > --- > gdb/bt-utils.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/gdb/bt-utils.c b/gdb/bt-utils.c > index 79e6e090d428..2054e68c6e5a 100644 > --- a/gdb/bt-utils.c > +++ b/gdb/bt-utils.c > @@ -29,15 +29,15 @@ gdb_internal_backtrace_set_cmd (const char *args, int from_tty, > cmd_list_element *c) > { > gdb_assert (c->type == set_cmd); > - gdb_assert (c->var_type == var_boolean); > - gdb_assert (c->var != nullptr); > + gdb_assert (c->var->type () == var_boolean); > + gdb_assert (c->var.has_value ()); > > #ifndef GDB_PRINT_INTERNAL_BACKTRACE > - bool *var_ptr = (bool *) c->var; > + bool val = c->var->get (); > > - if (*var_ptr) > + if (val) > { > - *var_ptr = false; > + c->var->set (false); > error (_("support for this feature is not compiled into GDB")); > } > #endif > -- > 2.33.0 >