bug#46786: closed (28.0.50; pcase-let does not bind variables used inside macro)

[GNU bug Tracking System]

Your message dated Tue, 09 Mar 2021 18:57:11 -0500
with message-id <jwvim5zx4n5.fsf-monnier+emacs@gnu.org>
and subject line Re: bug#46786: 28.0.50; pcase-let does not bind variables used 
inside macro
has caused the debbugs.gnu.org bug report #46786,
regarding 28.0.50; pcase-let does not bind variables used inside macro
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
46786: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=46786
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 28.0.50; pcase-let does not bind variables used inside macro
>
>
>
>
> Date: 
>
> Thu, 25 Feb 2021 21:40:54 -0300
>
>
>
>
> This testcase is a reduced version of a problem that happens in
> the "matrix-client.el" package (available on alphapapa's GitHub):
>
> (require 'cl-lib)
> (require 'pcase)
>
> (cl-macrolet
>     ((message-self () `(message self)))
>   (pcase-let ((`(,self ,self2) '("1" "2")))
>     (message-self)))
>
> Behavior before commit a218c9861573b5ec4979ff2662f5c0343397e3ff
> ("lisp/emacs-lisp/pcase.el: Don't bind unused vars in branches"):
> outputs 1 to the minibuffer.
>
> Behavior after the commit: Symbol’s value as variable is void: self
>
> Best,
> Mauricio
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#46786: 28.0.50; pcase-let does not bind variables used inside macro
>
>
>
>
> Date: 
>
> Tue, 09 Mar 2021 18:57:11 -0500
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)
>
>
>
>
> > I've just received a report from a user that the fix worked great. Thank
> > you very much!
>
> Thanks for confirming, I'm glad you like it,
>
>
>         Stefan
>
>
>
> --- End Message ---