GNU bug report logs - #72328
[PATCH] Nested backquote in pcase

Previous Next

Package: emacs;

Reported by: Thuna <thuna.cing <at> gmail.com>

Date: Sun, 28 Jul 2024 01:06:02 UTC

Severity: normal

Tags: patch

Done: Stefan Monnier <monnier <at> iro.umontreal.ca>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: michael_heerdegen <at> web.de, thuna.cing <at> gmail.com, 72328 <at> debbugs.gnu.org
Subject: bug#72328: [PATCH] Nested backquote in pcase
Date: Sun, 22 Sep 2024 17:16:46 +0300
> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Cc: Eli Zaretskii <eliz <at> gnu.org>,  thuna.cing <at> gmail.com,  72328 <at> debbugs.gnu.org
> Date: Sun, 22 Sep 2024 09:30:25 -0400
> 
> >> Ping!  Stefan, should I install this in your name?
> > In the meantime I installed the patch locally and got two warnings when
> > rebuilding Emacs completely:
> > | In testcover-analyze-coverage:
> > | emacs-lisp/testcover.el:472:8: Warning: Nested ` are not supported
> > |   ELC      emacs-lisp/timer-list.elc
> > |
> > | In testcover-analyze-coverage-wrapped-form:
> > | emacs-lisp/testcover.el:551:8: Warning: Nested ` are not supported
> >
> > But they should be trivial to fix (they are only warnings anyway).
> 
> Indeed, these are cases which would change meaning under the semantics
> that was proposed in this bug report.  The patch below would get rid of
> the nested backquotes, thus making that code "agnostic" to their semantics.

I'm not sure I understand: this is instead of the patch you posted
previously, or in addition to it?




This bug report was last modified 236 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.