--- Begin Message ---
|
Subject: |
27.0.50; thunk-delay: running expansion also requires lexical-binding |
|
Date: |
Wed, 25 Oct 2017 15:47:45 +0200 |
Hello,
having only lexical-binding: t in "thunk.el"'s header is not sufficient:
to make thunks work ok, `thunk-delay' needs lexical-binding when the
expansion is run.
Try e.g. something trivial as
#+begin_src emacs-lisp
(defun my-thunk-test ()
(thunk-force (thunk-delay (+ 1 2))))
#+end_src
This compiles ok, but executing (my-thunk-test) without lexbind enabled
gives you a meaningless error like "void-variable: forced".
I think we should document (in the file header) that lexical-binding
mode is required when thunks are created with `thunk-delay'. I think we
could additionally raise an error in the expansion of `thunk-delay' when
lexical-binding -> nil.
TIA,
Michael.
In GNU Emacs 27.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.22.24)
of 2017-10-25 built on drachen
Repository revision: cb73c70180f57f3fb99fae3aaefbacf0a61cea3f
Windowing system distributor 'The X.Org Foundation', version 11.0.11905000
System Description: Debian GNU/Linux testing (buster)
--- End Message ---
--- Begin Message ---
|
Subject: |
Re: bug#28990: 27.0.50; thunk-delay: running expansion also requires lexical-binding |
|
Date: |
Wed, 01 Nov 2017 15:08:47 +0100 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
Nicolas Petton <address@hidden> writes:
> > Nicolas, Stefan - ok to fix it like that?
>
> Yes, that's fine with me.
>
> > I would additionally add a sentence to "thunk.el"'s header saying that
> > creating thunks requires lexical binding.
>
> That's a good idea, thanks.
Done in bbb85eff49.
Michael.
--- End Message ---