* News from the lexical world
@ 2021-02-12 16:23 Stefan Monnier
2021-02-12 16:49 ` Stefan Kangas
` (3 more replies)
0 siblings, 4 replies; 25+ messages in thread
From: Stefan Monnier @ 2021-02-12 16:23 UTC (permalink / raw)
To: emacs-devel
Sorry to interrupt your great keybindings discussion, but I'll just
point out that we have now converted to lexical-binding a bit over 80%
of Emacs's ELisp files.
The breakdown of the remaining work is:
2 test/manual/cedet/
3 lisp/org/
4 lisp/cedet/semantic/analyze/
4 test/manual/
8 lisp/vc/
9 lisp/cedet/
9 lisp/cedet/semantic/bovine/
9 lisp/cedet/semantic/wisent/
9 lisp/emulation/
15 lisp/net/
18 lisp/mail/
21 lisp/textmodes/
22 lisp/cedet/srecode/
25 lisp/erc/
25 lisp/mh-e/
26 lisp/cedet/ede/
37 lisp/progmodes/
45 lisp/cedet/semantic/
49 lisp/obsolete/
59 lisp/
399 / 2003 = 19.920119820269594%
I don't expect we'll reach 100% soon but 90% seems quite feasible for
the actual Emacs-28 release.
Stefan
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-02-12 16:23 News from the lexical world Stefan Monnier
@ 2021-02-12 16:49 ` Stefan Kangas
2021-02-13 11:06 ` Lars Ingebrigtsen
2021-02-12 21:37 ` Andrea Corallo via Emacs development discussions.
` (2 subsequent siblings)
3 siblings, 1 reply; 25+ messages in thread
From: Stefan Kangas @ 2021-02-12 16:49 UTC (permalink / raw)
To: Stefan Monnier, emacs-devel
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> Sorry to interrupt your great keybindings discussion, but I'll just
> point out that we have now converted to lexical-binding a bit over 80%
> of Emacs's ELisp files.
[...]
> 399 / 2003 = 19.920119820269594%
For comparison, the same figure on the emacs-27 branch is:
1055 / 1871 = 56.38695884553715 %
IOW, we have already made some good progress for Emacs 28. But there is
no reason to slow down now -- let's get it done.
If anyone is willing to help convert a couple of files, for example ones
that they are already familiar with, it would help us progress even
faster here.
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-02-12 16:23 News from the lexical world Stefan Monnier
2021-02-12 16:49 ` Stefan Kangas
@ 2021-02-12 21:37 ` Andrea Corallo via Emacs development discussions.
2021-02-15 19:18 ` Phillip Lord
2021-04-10 21:32 ` Stefan Monnier
3 siblings, 0 replies; 25+ messages in thread
From: Andrea Corallo via Emacs development discussions. @ 2021-02-12 21:37 UTC (permalink / raw)
To: Stefan Monnier; +Cc: emacs-devel
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> Sorry to interrupt your great keybindings discussion, but I'll just
> point out that we have now converted to lexical-binding a bit over 80%
> of Emacs's ELisp files.
This is great news!
Thanks to everyone involved into this.
Andrea
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-02-12 16:49 ` Stefan Kangas
@ 2021-02-13 11:06 ` Lars Ingebrigtsen
0 siblings, 0 replies; 25+ messages in thread
From: Lars Ingebrigtsen @ 2021-02-13 11:06 UTC (permalink / raw)
To: Stefan Kangas; +Cc: Stefan Monnier, emacs-devel
Stefan Kangas <stefankangas@gmail.com> writes:
>> Sorry to interrupt your great keybindings discussion, but I'll just
>> point out that we have now converted to lexical-binding a bit over 80%
>> of Emacs's ELisp files.
> [...]
>> 399 / 2003 = 19.920119820269594%
>
> For comparison, the same figure on the emacs-27 branch is:
>
> 1055 / 1871 = 56.38695884553715 %
That's awesome -- then it sounds like Emacs will be at 100% in Emacs 29
sometime, I guess? (Or in Emacs 28 if that drags on way too long.)
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-02-12 16:23 News from the lexical world Stefan Monnier
2021-02-12 16:49 ` Stefan Kangas
2021-02-12 21:37 ` Andrea Corallo via Emacs development discussions.
@ 2021-02-15 19:18 ` Phillip Lord
2021-04-10 21:32 ` Stefan Monnier
3 siblings, 0 replies; 25+ messages in thread
From: Phillip Lord @ 2021-02-15 19:18 UTC (permalink / raw)
To: Stefan Monnier; +Cc: emacs-devel
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> Sorry to interrupt your great keybindings discussion, but I'll just
> point out that we have now converted to lexical-binding a bit over 80%
> of Emacs's ELisp files.
That's an amazing job!
Phil
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-02-12 16:23 News from the lexical world Stefan Monnier
` (2 preceding siblings ...)
2021-02-15 19:18 ` Phillip Lord
@ 2021-04-10 21:32 ` Stefan Monnier
2021-04-11 6:37 ` Lars Ingebrigtsen
` (2 more replies)
3 siblings, 3 replies; 25+ messages in thread
From: Stefan Monnier @ 2021-04-10 21:32 UTC (permalink / raw)
To: emacs-devel
Stefan Monnier [2021-02-12 11:23:13] wrote:
> Sorry to interrupt your great keybindings discussion, but I'll just
> point out that we have now converted to lexical-binding a bit over 80%
> of Emacs's ELisp files.
Things have progressed a bit since. We're now down to:
1 lisp/org/
1 lisp/progmodes/
2 / 2019 = 0.099058940069341253%
I.e. there are only 2 files left in Emacs's ELisp codebase which come
without the `lexical-binding:t` cookie. These are `org-agenda.el`
(which has already been converted upstream, so we're just waiting for
that code to make its way to an Org release and then to Emacs's
`master`) and `verilog-mode.el` (where the situation is a good bit more
delicate: verilog-mode can be used with Verilog files written
specifically for verilog-mode, with embedded ELisp snippets run via
`eval`, and those snippets tend to use "arbitrary" variables which
happened to be bound at that time, so we're still working on
a satisfactory solution, which will likely involve documenting which of
those variables can be used that way, along with what they mean).
This doesn't quite mean that all of Emacs's ELisp code is now using the
new dialect. The old non-lexbind dialect is still used in a few cases:
- There are still a fair number of calls to `eval` with a nil (or, more
often, absent) second argument.
- There is also still some code which builds lambda expressions "by
hand" (typically via backquote or otherwise embedded inside a larger
quoted piece of data) and without passing the resulting "source code
expression" to `eval` (or to the byte-compiler) to turn it into
a "function value". In the dynbind world those two were one and the
same, but not so in the lexbind world, and a value that starts with
a `lambda` will see its body evaluated using the old ELisp dialect.
I welcome help updating those cases.
Stefan
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-10 21:32 ` Stefan Monnier
@ 2021-04-11 6:37 ` Lars Ingebrigtsen
2021-04-11 7:20 ` Andrea Corallo via Emacs development discussions.
2021-04-26 6:47 ` Andrew Cohen
2 siblings, 0 replies; 25+ messages in thread
From: Lars Ingebrigtsen @ 2021-04-11 6:37 UTC (permalink / raw)
To: Stefan Monnier; +Cc: emacs-devel
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> Things have progressed a bit since. We're now down to:
>
> 1 lisp/org/
> 1 lisp/progmodes/
>
> 2 / 2019 = 0.099058940069341253%
Wow, that's amazing. Thank you very much, Stefan & Stefan.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-10 21:32 ` Stefan Monnier
2021-04-11 6:37 ` Lars Ingebrigtsen
@ 2021-04-11 7:20 ` Andrea Corallo via Emacs development discussions.
2021-04-11 13:44 ` T.V Raman
2021-04-26 6:47 ` Andrew Cohen
2 siblings, 1 reply; 25+ messages in thread
From: Andrea Corallo via Emacs development discussions. @ 2021-04-11 7:20 UTC (permalink / raw)
To: Stefan Monnier; +Cc: emacs-devel
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> Stefan Monnier [2021-02-12 11:23:13] wrote:
>> Sorry to interrupt your great keybindings discussion, but I'll just
>> point out that we have now converted to lexical-binding a bit over 80%
>> of Emacs's ELisp files.
>
> Things have progressed a bit since. We're now down to:
>
> 1 lisp/org/
> 1 lisp/progmodes/
>
> 2 / 2019 = 0.099058940069341253%
Super! Thanks Stefan(s) for the effort!
Andrea
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-11 7:20 ` Andrea Corallo via Emacs development discussions.
@ 2021-04-11 13:44 ` T.V Raman
2021-04-11 14:39 ` Stefan Monnier
0 siblings, 1 reply; 25+ messages in thread
From: T.V Raman @ 2021-04-11 13:44 UTC (permalink / raw)
To: Andrea Corallo via Emacs development discussions.
Cc: Stefan Monnier, Andrea Corallo
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 772 bytes --]
Andrea Corallo via "Emacs development discussions."
<emacs-devel@gnu.org> writes:
Between native compilation and lexical scoping updates, emacs 28 is
looking to be a truly momentous released!
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>> Stefan Monnier [2021-02-12 11:23:13] wrote:
>>> Sorry to interrupt your great keybindings discussion, but I'll just
>>> point out that we have now converted to lexical-binding a bit over 80%
>>> of Emacs's ELisp files.
>>
>> Things have progressed a bit since. We're now down to:
>>
>> 1 lisp/org/
>> 1 lisp/progmodes/
>>
>> 2 / 2019 = 0.099058940069341253%
>
> Super! Thanks Stefan(s) for the effort!
>
> Andrea
>
--
Thanks,
--Raman
7©4 Id: kg:/m/0285kf1 •0Ü8
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-11 13:44 ` T.V Raman
@ 2021-04-11 14:39 ` Stefan Monnier
2021-04-11 15:10 ` T.V Raman
2021-04-11 15:42 ` Eli Zaretskii
0 siblings, 2 replies; 25+ messages in thread
From: Stefan Monnier @ 2021-04-11 14:39 UTC (permalink / raw)
To: T.V Raman
Cc: Andrea Corallo, Andrea Corallo via Emacs development discussions.
> Between native compilation and lexical scoping updates, emacs 28 is
> looking to be a truly momentous released!
Although the fact that bundled ELisp files use lexical-binding should
have no visible impact on Emacs users (not even tho who develop
packages).
Stefan
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-11 14:39 ` Stefan Monnier
@ 2021-04-11 15:10 ` T.V Raman
2021-04-11 15:42 ` Eli Zaretskii
1 sibling, 0 replies; 25+ messages in thread
From: T.V Raman @ 2021-04-11 15:10 UTC (permalink / raw)
To: Stefan Monnier
Cc: Andrea Corallo via Emacs development discussions., Andrea Corallo
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=gb18030, Size: 498 bytes --]
Stefan Monnier <monnier@iro.umontreal.ca> writes:
Agreed re end-user visibility, but it's still a watershed moment for
emacs.
>> Between native compilation and lexical scoping updates, emacs 28 is
>> looking to be a truly momentous released!
>
> Although the fact that bundled ELisp files use lexical-binding should
> have no visible impact on Emacs users (not even tho who develop
> packages).
>
>
> Stefan
>
--
Thanks,
--Raman
7©4 Id: kg:/m/0285kf1 •0Ü8
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-11 14:39 ` Stefan Monnier
2021-04-11 15:10 ` T.V Raman
@ 2021-04-11 15:42 ` Eli Zaretskii
2021-04-11 16:34 ` Stefan Monnier
1 sibling, 1 reply; 25+ messages in thread
From: Eli Zaretskii @ 2021-04-11 15:42 UTC (permalink / raw)
To: Stefan Monnier; +Cc: akrl, emacs-devel, raman
> From: Stefan Monnier <monnier@iro.umontreal.ca>
> Date: Sun, 11 Apr 2021 10:39:03 -0400
> Cc: Andrea Corallo <akrl@sdf.org>,
> "Andrea Corallo via Emacs development discussions." <emacs-devel@gnu.org>
>
> > Between native compilation and lexical scoping updates, emacs 28 is
> > looking to be a truly momentous released!
>
> Although the fact that bundled ELisp files use lexical-binding should
> have no visible impact on Emacs users
Nor native compilation, for that matter: it doesn't add functionality,
"only" speeds Emacs up.
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-11 15:42 ` Eli Zaretskii
@ 2021-04-11 16:34 ` Stefan Monnier
2021-04-12 8:34 ` Andrea Corallo via Emacs development discussions.
0 siblings, 1 reply; 25+ messages in thread
From: Stefan Monnier @ 2021-04-11 16:34 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: akrl, emacs-devel, raman
>> > Between native compilation and lexical scoping updates, emacs 28 is
>> > looking to be a truly momentous released!
>> Although the fact that bundled ELisp files use lexical-binding should
>> have no visible impact on Emacs users
> Nor native compilation, for that matter: it doesn't add functionality,
> "only" speeds Emacs up.
Indeed (tho it should be somewhat visible because of the extra files
and the occasional difference in performance: use of lexical-binding
can't even claim such minor impacts).
Stefan
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-11 16:34 ` Stefan Monnier
@ 2021-04-12 8:34 ` Andrea Corallo via Emacs development discussions.
2021-04-12 11:58 ` Eli Zaretskii
0 siblings, 1 reply; 25+ messages in thread
From: Andrea Corallo via Emacs development discussions. @ 2021-04-12 8:34 UTC (permalink / raw)
To: Stefan Monnier; +Cc: Eli Zaretskii, raman, emacs-devel
Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>> > Between native compilation and lexical scoping updates, emacs 28 is
>>> > looking to be a truly momentous released!
>>> Although the fact that bundled ELisp files use lexical-binding should
>>> have no visible impact on Emacs users
>> Nor native compilation, for that matter: it doesn't add functionality,
>> "only" speeds Emacs up.
>
> Indeed (tho it should be somewhat visible because of the extra files
> and the occasional difference in performance: use of lexical-binding
> can't even claim such minor impacts).
But we can say that the native compiler works better on lexical code as
on dynamic one just gives-up on any Lisp optimization :)
Andrea
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-12 8:34 ` Andrea Corallo via Emacs development discussions.
@ 2021-04-12 11:58 ` Eli Zaretskii
2021-04-12 12:58 ` Andrea Corallo via Emacs development discussions.
0 siblings, 1 reply; 25+ messages in thread
From: Eli Zaretskii @ 2021-04-12 11:58 UTC (permalink / raw)
To: Andrea Corallo; +Cc: emacs-devel, monnier, raman
> From: Andrea Corallo <akrl@sdf.org>
> Cc: Eli Zaretskii <eliz@gnu.org>, raman@google.com, emacs-devel@gnu.org
> Date: Mon, 12 Apr 2021 08:34:37 +0000
>
> But we can say that the native compiler works better on lexical code as
> on dynamic one just gives-up on any Lisp optimization :)
Yes, this should definitely be documented.
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-12 11:58 ` Eli Zaretskii
@ 2021-04-12 12:58 ` Andrea Corallo via Emacs development discussions.
2021-04-12 13:23 ` Stefan Kangas
2021-04-12 13:24 ` Eli Zaretskii
0 siblings, 2 replies; 25+ messages in thread
From: Andrea Corallo via Emacs development discussions. @ 2021-04-12 12:58 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: monnier, raman, emacs-devel
Eli Zaretskii <eliz@gnu.org> writes:
>> From: Andrea Corallo <akrl@sdf.org>
>> Cc: Eli Zaretskii <eliz@gnu.org>, raman@google.com, emacs-devel@gnu.org
>> Date: Mon, 12 Apr 2021 08:34:37 +0000
>>
>> But we can say that the native compiler works better on lexical code as
>> on dynamic one just gives-up on any Lisp optimization :)
>
> Yes, this should definitely be documented.
Where do you think would be the best place to document it?
Thanks
Andrea
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-12 12:58 ` Andrea Corallo via Emacs development discussions.
@ 2021-04-12 13:23 ` Stefan Kangas
2021-04-12 13:24 ` Eli Zaretskii
1 sibling, 0 replies; 25+ messages in thread
From: Stefan Kangas @ 2021-04-12 13:23 UTC (permalink / raw)
To: Andrea Corallo, Eli Zaretskii; +Cc: emacs-devel, monnier, raman
Andrea Corallo via "Emacs development discussions."
<emacs-devel@gnu.org> writes:
>>> But we can say that the native compiler works better on lexical code as
>>> on dynamic one just gives-up on any Lisp optimization :)
>>
>> Yes, this should definitely be documented.
>
> Where do you think would be the best place to document it?
This provides a real incentive to convert code to lexical-binding.
My vote would be for the ELisp manual, to give it as high visibility as
possible. Even better if we could also mention it in NEWS.
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-12 12:58 ` Andrea Corallo via Emacs development discussions.
2021-04-12 13:23 ` Stefan Kangas
@ 2021-04-12 13:24 ` Eli Zaretskii
1 sibling, 0 replies; 25+ messages in thread
From: Eli Zaretskii @ 2021-04-12 13:24 UTC (permalink / raw)
To: Andrea Corallo; +Cc: emacs-devel, monnier, raman
> From: Andrea Corallo <akrl@sdf.org>
> Cc: monnier@iro.umontreal.ca, raman@google.com, emacs-devel@gnu.org
> Date: Mon, 12 Apr 2021 12:58:04 +0000
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
> >> From: Andrea Corallo <akrl@sdf.org>
> >> Cc: Eli Zaretskii <eliz@gnu.org>, raman@google.com, emacs-devel@gnu.org
> >> Date: Mon, 12 Apr 2021 08:34:37 +0000
> >>
> >> But we can say that the native compiler works better on lexical code as
> >> on dynamic one just gives-up on any Lisp optimization :)
> >
> > Yes, this should definitely be documented.
>
> Where do you think would be the best place to document it?
Where we describe lexical-binding in the ELisp manual, I think.
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-10 21:32 ` Stefan Monnier
2021-04-11 6:37 ` Lars Ingebrigtsen
2021-04-11 7:20 ` Andrea Corallo via Emacs development discussions.
@ 2021-04-26 6:47 ` Andrew Cohen
2021-04-26 10:08 ` Stefan Kangas
2 siblings, 1 reply; 25+ messages in thread
From: Andrew Cohen @ 2021-04-26 6:47 UTC (permalink / raw)
To: emacs-devel
Haven't had time to update emacs in quite some time. Updated today and
ran into a problem with supercite.el:
When trying to reply to an article and including the original I get an error
run-hooks: Symbol’s value as variable is void: attribution
which arises in 'sc-select-attribution. I haven't had a chance to find
the actual bug, but removing lexical binding fixes the problem.
--
Andrew Cohen
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-26 6:47 ` Andrew Cohen
@ 2021-04-26 10:08 ` Stefan Kangas
2021-04-26 12:06 ` Andrew Cohen
0 siblings, 1 reply; 25+ messages in thread
From: Stefan Kangas @ 2021-04-26 10:08 UTC (permalink / raw)
To: Andrew Cohen, emacs-devel
Andrew Cohen <acohen@ust.hk> writes:
> Haven't had time to update emacs in quite some time. Updated today and
> ran into a problem with supercite.el:
>
> When trying to reply to an article and including the original I get an error
>
> run-hooks: Symbol’s value as variable is void: attribution
>
> which arises in 'sc-select-attribution. I haven't had a chance to find
> the actual bug, but removing lexical binding fixes the problem.
Could you run `M-x toggle-debug-on-error', redo the steps that led to
this error and post the resulting backtrace?
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-26 10:08 ` Stefan Kangas
@ 2021-04-26 12:06 ` Andrew Cohen
2021-04-26 16:06 ` Stefan Monnier
0 siblings, 1 reply; 25+ messages in thread
From: Andrew Cohen @ 2021-04-26 12:06 UTC (permalink / raw)
To: emacs-devel
I have found the problem (but haven't had the time to fix it :()
In supercite 'sc-select-attribution the variables 'citation and
'attribution are bound and then the hook 'sc-attribs-postselect-hook is
run. I use bbdb which adds to that hook an anonymous defun:
(add-hook 'sc-attribs-postselect-hook
(lambda ()
(setq bbdb-sc-last-attrib
(if sc-downcase-p
(downcase attribution)
attribution))))
and with lexical binding in supercite this of course fails.
--
Andrew Cohen
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-26 12:06 ` Andrew Cohen
@ 2021-04-26 16:06 ` Stefan Monnier
2021-04-26 23:31 ` Andrew Cohen
0 siblings, 1 reply; 25+ messages in thread
From: Stefan Monnier @ 2021-04-26 16:06 UTC (permalink / raw)
To: Andrew Cohen; +Cc: emacs-devel
> I have found the problem (but haven't had the time to fix it :()
>
> In supercite 'sc-select-attribution the variables 'citation and
> 'attribution are bound and then the hook 'sc-attribs-postselect-hook is
> run. I use bbdb which adds to that hook an anonymous defun:
>
> (add-hook 'sc-attribs-postselect-hook
> (lambda ()
> (setq bbdb-sc-last-attrib
> (if sc-downcase-p
> (downcase attribution)
> attribution))))
Hmm... I can't find this code in the BBDB copy I have (from elpa.git).
Which version of BBDB do you have?
Stefan
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-26 16:06 ` Stefan Monnier
@ 2021-04-26 23:31 ` Andrew Cohen
2021-04-29 22:06 ` Stefan Monnier
0 siblings, 1 reply; 25+ messages in thread
From: Andrew Cohen @ 2021-04-26 23:31 UTC (permalink / raw)
To: emacs-devel
>>>>> "SM" == Stefan Monnier <monnier@iro.umontreal.ca> writes:
[...]
>> (add-hook 'sc-attribs-postselect-hook (lambda () (setq
>> bbdb-sc-last-attrib (if sc-downcase-p (downcase attribution)
>> attribution))))
SM> Hmm... I can't find this code in the BBDB copy I have (from
SM> elpa.git). Which version of BBDB do you have?
I have the git version from
https://git.savannah.nongnu.org/cgit/bbdb.git/
The code is in bbdb-sc.el
;; Insert our hooks
;; Dammit, supercite! It runs `sc-attribs-postselect-hook' in an
;; environment with the local variable `attribution' that we rely on.
(with-no-warnings (defvar attribution))
;;;###autoload
(defun bbdb-insinuate-sc ()
"Hook BBDB into Supercite.
Do not call this in your init file. Use `bbdb-initialize'.
However, this is not the full story. See bbdb-sc.el for how to fully hook
BBDB into Supercite."
(add-hook 'sc-post-hook 'bbdb-sc-set-attrib)
(add-hook 'sc-attribs-postselect-hook
(lambda ()
(setq bbdb-sc-last-attrib
(if sc-downcase-p
(downcase attribution)
attribution)))))
But I guess this is a more generic problem? Any function added to
sc-attribs-postselect-hook will need access to 'attribution and/or
'citation which are not in scope.
Best,
Andy
--
Andrew Cohen
^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-26 23:31 ` Andrew Cohen
@ 2021-04-29 22:06 ` Stefan Monnier
2021-04-30 0:17 ` Andrew Cohen
0 siblings, 1 reply; 25+ messages in thread
From: Stefan Monnier @ 2021-04-29 22:06 UTC (permalink / raw)
To: Andrew Cohen; +Cc: emacs-devel
> But I guess this is a more generic problem? Any function added to
> sc-attribs-postselect-hook will need access to 'attribution and/or
> 'citation which are not in scope.
Indeed, I see now that it's documented in the docstring of
`sc-select-attribution`.
I installed the patch below which should fix it. Please confirm.
Stefan
diff --git a/lisp/mail/supercite.el b/lisp/mail/supercite.el
index dc1c641052b..d545b0c3f15 100644
--- a/lisp/mail/supercite.el
+++ b/lisp/mail/supercite.el
@@ -1128,6 +1128,8 @@ sc-select-attribution
auto-selected citation string and the variable `attribution' is bound
to the auto-selected attribution string."
(run-hooks 'sc-attribs-preselect-hook)
+ (with-suppressed-warnings ((lexical citation attribution))
+ (defvar citation) (defvar attribution))
(let ((query-p sc-confirm-always-p)
attribution citation
(attriblist sc-preferred-attribution-list))
^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: News from the lexical world
2021-04-29 22:06 ` Stefan Monnier
@ 2021-04-30 0:17 ` Andrew Cohen
0 siblings, 0 replies; 25+ messages in thread
From: Andrew Cohen @ 2021-04-30 0:17 UTC (permalink / raw)
To: Stefan Monnier; +Cc: emacs-devel
>>>>> "SM" == Stefan Monnier <monnier@iro.umontreal.ca> writes:
[...]
SM> I installed the patch below which should fix it. Please
SM> confirm.
Yes, that patch fixes it.
Best,
Andy
--
Andrew Cohen
^ permalink raw reply [flat|nested] 25+ messages in thread
end of thread, other threads:[~2021-04-30 0:17 UTC | newest]
Thread overview: 25+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-12 16:23 News from the lexical world Stefan Monnier
2021-02-12 16:49 ` Stefan Kangas
2021-02-13 11:06 ` Lars Ingebrigtsen
2021-02-12 21:37 ` Andrea Corallo via Emacs development discussions.
2021-02-15 19:18 ` Phillip Lord
2021-04-10 21:32 ` Stefan Monnier
2021-04-11 6:37 ` Lars Ingebrigtsen
2021-04-11 7:20 ` Andrea Corallo via Emacs development discussions.
2021-04-11 13:44 ` T.V Raman
2021-04-11 14:39 ` Stefan Monnier
2021-04-11 15:10 ` T.V Raman
2021-04-11 15:42 ` Eli Zaretskii
2021-04-11 16:34 ` Stefan Monnier
2021-04-12 8:34 ` Andrea Corallo via Emacs development discussions.
2021-04-12 11:58 ` Eli Zaretskii
2021-04-12 12:58 ` Andrea Corallo via Emacs development discussions.
2021-04-12 13:23 ` Stefan Kangas
2021-04-12 13:24 ` Eli Zaretskii
2021-04-26 6:47 ` Andrew Cohen
2021-04-26 10:08 ` Stefan Kangas
2021-04-26 12:06 ` Andrew Cohen
2021-04-26 16:06 ` Stefan Monnier
2021-04-26 23:31 ` Andrew Cohen
2021-04-29 22:06 ` Stefan Monnier
2021-04-30 0:17 ` Andrew Cohen
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).