From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Michael Heerdegen via "Bug reports for GNU Emacs, the Swiss army knife of text editors" Newsgroups: gmane.emacs.bugs Subject: bug#69108: false-positive warning "variable =?UTF-8?Q?=E2=80=98=5F=E2=80=99?= not left unused" in if-let* and if-let Date: Wed, 14 Feb 2024 02:01:39 +0100 Message-ID: <87il2ryi30.fsf@web.de> References: Reply-To: Michael Heerdegen Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="23069"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: 69108@debbugs.gnu.org To: Konstantin Kharlamov Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Wed Feb 14 02:02:06 2024 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1ra3f4-0005qK-4e for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 14 Feb 2024 02:02:06 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ra3em-0002hS-1u; Tue, 13 Feb 2024 20:01:48 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ra3ek-0002hE-43 for bug-gnu-emacs@gnu.org; Tue, 13 Feb 2024 20:01:46 -0500 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1ra3ei-0004Fv-4F for bug-gnu-emacs@gnu.org; Tue, 13 Feb 2024 20:01:45 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ra3ez-00081G-QA for bug-gnu-emacs@gnu.org; Tue, 13 Feb 2024 20:02:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Michael Heerdegen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 14 Feb 2024 01:02:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 69108 X-GNU-PR-Package: emacs Original-Received: via spool by 69108-submit@debbugs.gnu.org id=B69108.170787250430797 (code B ref 69108); Wed, 14 Feb 2024 01:02:01 +0000 Original-Received: (at 69108) by debbugs.gnu.org; 14 Feb 2024 01:01:44 +0000 Original-Received: from localhost ([127.0.0.1]:48875 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ra3ei-00080f-BV for submit@debbugs.gnu.org; Tue, 13 Feb 2024 20:01:44 -0500 Original-Received: from mout.web.de ([212.227.17.12]:59647) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1ra3ec-00080D-5c for 69108@debbugs.gnu.org; Tue, 13 Feb 2024 20:01:43 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=s29768273; t=1707872474; x=1708477274; i=michael_heerdegen@web.de; bh=3Xq1xjCostnC7iY4gPW6jy9cm9BF0iaFQEPW2+kWfiE=; h=X-UI-Sender-Class:From:To:Cc:Subject:In-Reply-To:References: Date; b=OcpHDgMK/Y5X7g6XTvLxGnbAI6T+f5w9PZ88X/epiHmnJJHqNxT1cg2do1r9nbC9 KpH6cLKYLFVAxPZmf7gcgz9vrmkK3I6R2sXroFLMQCuEF3luaD/M87PjUxWP6hF47 JazjyydzCi2HWIS45NS5rj73TPiTx9DX55Wx1O+FP/lR6hfmRrHMpIoHhztDKOrGG ZOKg2vETEW2FwlA5HaJNW4KMtIWGF3XlcEj3qEu91/d16dSBolXXhKiY3+10wZ9Yr EVcocFTN2tJ4OfRL2EBoQg6eQketVjzTNRBPSJOLQI4WRrKmDKhrQUkzLneWOOWrf a/MKIIKQx+5sllSNRw== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Original-Received: from drachen.dragon ([88.66.201.224]) by smtp.web.de (mrweb106 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MPrLN-1rNjp43lJL-00MuI1; Wed, 14 Feb 2024 02:01:13 +0100 In-Reply-To: (Konstantin Kharlamov's message of "Wed, 14 Feb 2024 00:21:01 +0300") X-Provags-ID: V03:K1:0K77YDCXqhcgl0XH9wwC8V5l0aEv2s7xlXAu7g/H7CGkYvqpZ8R T1w43cwSG2UPzZZTzzKOlnSn9cnl4jPTarkvA62k6LFyXsL/EdjU3CMcEb0+ZYJwaAnpWPl hBr+FjyDT2crfY4ZmLwqIgHaR93Zd4hzOoXKb4krtEx4vF6TWlKskhfw5sf+I1oCYoCZi+A dSeTNnLsskag44ngFr5ag== UI-OutboundReport: notjunk:1;M01:P0:7vbzFHxkzg8=;S5uaqZUFRrb8tMN0rsc1yosNaIh TdM5dZckr4BC9I3aChkIn4am/xy/u3XzIFF4IN41+Y4BLfJZrcSTlftu1kiA31SbN3cSUyBpy /Uxx8AXMsYyBG55Z/3jfmzRG2Z91Rnh9DaB60DeJkIC5JDZdvEhfISO/+FEEeBLmp9ah1MXvs s0Aekl5Jm4O2IJmH5DDth3ckSVVOHiU6mPs1YljiC4ofKDLYXKOyl1bJ2o8YaJULjdIq4u+3Y RH7cTsQh1e11bxYxFuS59p6255cGRBWhNfnFRiCiUhTFi8F7jMETe+MZuCx8TNxCPXjj5uXZ7 oupbKntiIMiAO4D+MqkKPvZjzTKGDoX8OTrKpsmEP4McOlhEG0zXk0sqBuhWeYs8LrPpNpzO3 0pblgimhFgchAmwrYjV9FO8qWnWzZI4bWCToywjBn87mRiBjBJLhTKIlbbNHJzJAQihnpx3KK gAIQmW3zWUJikuUuzKPwQ+9vFZ4bxioZ3tqKv715XE9sFU/oBTZeDLcgrq+Hmm796N+g2YAmX 9YG/GhHVfLJu/VqXNI4ftvkUIVv46YMfbIlbfids5HxG59fHueZuh8nBcVzK7U0mzm/Y8e3We DeFIVKg+D2l2qOGVINVwoAHeY+9W8sIjErDKEn9m8woCYAci2D2poM8Wx9zQfp2Fk8qaedIYL gdXm1rrjn7Z0TA8LtpRsl9tQJZv2mkklLfAxNEaZHhuK1ZRKMls568W+K9Kb6WeG7K5Ak5QKE D7RrRBnn0qihiCQMUj/Iv9J0QyNPOyX2Z8Teay8Vq1Bpiy9H61T46mXbSxLCZfSBOhHMM0FC X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:279987 Archived-At: Konstantin Kharlamov writes: > I've been writing an answer for a question on emacs.stackexchange=C2=B9 a= nd > to avoid nested `if` and `let` clauses I used a `if-let*`, and result > of one of the checks I assigned to a `_` variable, because the > variable would be left unused, it's only the check being non-nil that > mattered. > > But when byte-compiled that triggered a: > > test.el:6:9: Warning: variable =E2=80=98_=E2=80=99 not left unused > > =E2=80=A6which is untrue, because it is unused. I also find this annoying. Currently the variable is actually used (implicitly, in the expansion), so it's not an error in the compiler. But the warning is not really helpful (code works as intended), and (_ TEST-EXPR) is maybe even easier to read or more intuitive than the official (TEST-EXPR) syntax. Would be a one liner to make both cases generate the same expansion. Are there any votes against this? Michael.