From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Stefan Kangas Newsgroups: gmane.emacs.bugs Subject: bug#67611: [PATCH] Add a Pcase pattern `cl-lambda` equivalent to `cl-destructuring-bind` Date: Thu, 11 Jan 2024 13:12:43 -0800 Message-ID: References: <277dd49a-e96d-4faf-a22e-aca952354a37@protonmail.com> <30f1bf76-1cf1-493e-be4f-38e405d0ecf6@protonmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="3631"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 67611@debbugs.gnu.org, Stefan Monnier , joaotavora@gmail.com To: Okamsn Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Thu Jan 11 22:13:28 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 1rO2Mg-0000lT-Qt for geb-bug-gnu-emacs@m.gmane-mx.org; Thu, 11 Jan 2024 22:13:26 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rO2MN-00042h-Tw; Thu, 11 Jan 2024 16:13:07 -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 1rO2ML-00041i-Cw for bug-gnu-emacs@gnu.org; Thu, 11 Jan 2024 16:13:05 -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 1rO2MK-0004Hm-UY for bug-gnu-emacs@gnu.org; Thu, 11 Jan 2024 16:13:05 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1rO2MI-0000Zy-01 for bug-gnu-emacs@gnu.org; Thu, 11 Jan 2024 16:13:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Stefan Kangas Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 11 Jan 2024 21:13:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 67611 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 67611-submit@debbugs.gnu.org id=B67611.17050075712195 (code B ref 67611); Thu, 11 Jan 2024 21:13:01 +0000 Original-Received: (at 67611) by debbugs.gnu.org; 11 Jan 2024 21:12:51 +0000 Original-Received: from localhost ([127.0.0.1]:34141 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rO2M7-0000ZL-Fu for submit@debbugs.gnu.org; Thu, 11 Jan 2024 16:12:51 -0500 Original-Received: from mail-lj1-x22d.google.com ([2a00:1450:4864:20::22d]:57734) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rO2M4-0000Z3-9C for 67611@debbugs.gnu.org; Thu, 11 Jan 2024 16:12:50 -0500 Original-Received: by mail-lj1-x22d.google.com with SMTP id 38308e7fff4ca-2cd1ca52f31so67330481fa.3 for <67611@debbugs.gnu.org>; Thu, 11 Jan 2024 13:12:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1705007564; x=1705612364; darn=debbugs.gnu.org; h=cc:to:subject:message-id:date:mime-version:references:in-reply-to :from:from:to:cc:subject:date:message-id:reply-to; bh=iLl7AGuRvjcaeVzmh3uGVp6vpj1E/SujS0TMWc5nZ9M=; b=A8wv2VDv62FxGc5Egs+6U4f7oy67npN5RGS8eY8WInIPOv4UNcvxKrH0IT6553CIpX fatZKmzi0jNbxEpGNl1Teowby6SJ3gS6YYZCzOIbR/kbajlD8pQ6yV6IkT8FSMJayI+g XAQbuwOj0mRFTupU6TPXCEbWy1R9Qprzt6bEehltQGJN3xdl+iClNG79Xs6JCaH6Az9c Apk1jCyLpFZs9Nn40b2DH7pUhVaxsctlfcEyoc3Y+U7oDL+u9BELcs5DT0Mm7vtIaZG3 XIHGBVI1p3UdAn/8DVOe5GLYOigpT6TJRUT4oMQh9M7eC6IXluypqMvV/Wj0a412G4s2 PXpQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1705007564; x=1705612364; h=cc:to:subject:message-id:date:mime-version:references:in-reply-to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=iLl7AGuRvjcaeVzmh3uGVp6vpj1E/SujS0TMWc5nZ9M=; b=XNKuphfODu1UjwBaBnjyYv4NO2rcH+ZY0UC6tlflFE1WwfSSqUR7h/5vJcrm9pCfhH aR5trBMxX2VlWFWqp7Z+ISwf73+BObpCuf8gs4Asqx77HClTuKm1fIlyVx3o/3OaLt2+ oH1Os/OVBu8Rtcf2tUyrN8r1z5UsMzxg78C8gfmOl1xV0qf4BEQqJeVM3nOBgIFbZPTZ eWaUmk6CCSu5WiaYYYP94wpP+oOEABsdJOrU0z0WVq9l8EYW+oOYHDHc8AnI3KTv3oa9 h2NAc4YDzuP6znRaDHzZgOGcrxjtFoLEaKHtkWdqHVtKMO2YKJQ1yjRQ+KDxw/xFZNyc ZmqA== X-Gm-Message-State: AOJu0YyUpTkeFSzGjBI3GoycqkdaAFWIwunFNgN3k3ehpBq1uwsvpkEj Pg80k3CVstfFhCkTlmz9FyzEIVY8J9K3XnaH63rvHD2Rmgw= X-Google-Smtp-Source: AGHT+IFNYfseqrm49xjmjCcwx/bdLp60Z6enes4sGqotSkiO5421JUP3DDXL5m0eTWEVXsQaHjFLPJb3GAmG0VMu56A= X-Received: by 2002:a2e:950e:0:b0:2cc:e919:eb8f with SMTP id f14-20020a2e950e000000b002cce919eb8fmr182351ljh.101.1705007564170; Thu, 11 Jan 2024 13:12:44 -0800 (PST) Original-Received: from 753933720722 named unknown by gmailapi.google.com with HTTPREST; Thu, 11 Jan 2024 13:12:43 -0800 In-Reply-To: <30f1bf76-1cf1-493e-be4f-38e405d0ecf6@protonmail.com> (okamsn@protonmail.com's message of "Mon, 25 Dec 2023 21:30:07 +0000") 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:277961 Archived-At: severity 67611 wishlist tags 67611 + notabug wontfix close 67611 thanks Okamsn writes: > Hello, > > Because I wrote this patch with the thought that others might want it, I > don't have any nontrivial examples to share right now. The best example > I have for the use of the optional arguments is for the implementing of > the optional arguments, which isn't very convincing. > > I've updated the patch to rename the pattern to `cl-arglist` and to > avoid creating intermediate variables using the `let` pattern, but I'm > fine with resting the discussion here until a stronger argument can be > made in favor of the patch. > > Thank you. Thanks for the patch. I can only echo the sentiment already given here, that making `pcase' more complex won't do much for its popularity. It sounds like we mostly agree that no change should be made here, at least not for the moment, so I'm inclined to close this one as wontfix. It's always awkward to have to say no to a well-reasoned patch, but sometimes a decision has to be made. I hope you can understand, and apologies for not accepting your patch. I'm closing the bug with this message. Thank you again for contributing.