Re: missing lexical-binding cookie warning when loading .el files

["Arsen Arsenović" <arsen@aarsen.me>]

[-- Attachment #1: Type: text/plain, Size: 801 bytes --]

Hi Mattias,

Mattias Engdegård <mattias.engdegard@gmail.com> writes:

> I'm back-pedalling on this warning: in particular I underestimated the
> amount of little glue files (autoloads, subdirs.el, etc) that users
> have. Almost all of these, and most of the other files, will likely
> work without change when the default is changed to lexical-binding, so
> let users worry about that then.

May I propose emitting it only when there are variables whose scoping
would be changed by it?  IMO, this warning is a reasonable one, but
there's also lots of glue files where it doesn't matter.

IIUC, the heuristic for that shouldn't be very hard to get right (but I
am also not fully aware of the lexical-binding semantics so I'm not
certain).

Have a lovely day.
-- 
Arsen Arsenović

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 251 bytes --]