unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Default lexical-binding to t
@ 2024-11-01 12:55 Stefan Monnier
  2024-11-01 14:42 ` Gerd Möllmann
  2024-11-01 17:03 ` Karl Fogel
  0 siblings, 2 replies; 3+ messages in thread
From: Stefan Monnier @ 2024-11-01 12:55 UTC (permalink / raw)
  To: emacs-devel

In bug#74145 I suggest that it is time to change the default of
`lexical-binding` to t.  Clearly this is a breaking change, but only for
those files which:

- Don't have a `lexical-binding` cookie.
- Have code which happens to behave differently under the new dialect
  (such code is not rare, but a lot of code works identically in the
  two dialects).

I believe by the time Emacs-31 will be released, such files will be
uncommon, and it is easy to fix them (either by adjusting he code, or
by slapping a `lexical-binding` cookie).


        Stefan




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Default lexical-binding to t
  2024-11-01 12:55 Default lexical-binding to t Stefan Monnier
@ 2024-11-01 14:42 ` Gerd Möllmann
  2024-11-01 17:03 ` Karl Fogel
  1 sibling, 0 replies; 3+ messages in thread
From: Gerd Möllmann @ 2024-11-01 14:42 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> In bug#74145 I suggest that it is time to change the default of
> `lexical-binding` to t.  Clearly this is a breaking change, but only for
> those files which:
>
> - Don't have a `lexical-binding` cookie.
> - Have code which happens to behave differently under the new dialect
>   (such code is not rare, but a lot of code works identically in the
>   two dialects).
>
> I believe by the time Emacs-31 will be released, such files will be
> uncommon, and it is easy to fix them (either by adjusting he code, or
> by slapping a `lexical-binding` cookie).

+1



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Default lexical-binding to t
  2024-11-01 12:55 Default lexical-binding to t Stefan Monnier
  2024-11-01 14:42 ` Gerd Möllmann
@ 2024-11-01 17:03 ` Karl Fogel
  1 sibling, 0 replies; 3+ messages in thread
From: Karl Fogel @ 2024-11-01 17:03 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

On 01 Nov 2024, Stefan Monnier wrote:
>In bug#74145 I suggest that it is time to change the default of
>`lexical-binding` to t.  Clearly this is a breaking change, but 
>only for
>those files which:
>
>- Don't have a `lexical-binding` cookie.
>- Have code which happens to behave differently under the new 
>dialect
>  (such code is not rare, but a lot of code works identically in 
>  the
>  two dialects).
>
>I believe by the time Emacs-31 will be released, such files will 
>be
>uncommon, and it is easy to fix them (either by adjusting he 
>code, or
>by slapping a `lexical-binding` cookie).

+1  It's time.

Anecdata, but: a lot of the random Emacs packages I encounter in 
the wild lately have `lexical-binding` cookies already (and 
usually to t anyway).  Most recently 
https://github.com/kmonad/kbd-mode/blob/master/kbd-mode.el, for 
example.  No doubt there are also many that don't, but I think 
knowledge about the new dialect is diffused widely enough now that 
when Emacs' default switches over, some user of a given package 
will quickly recognize what's going on and submit the appropriate 
change to that package; in a few cases I have done so myself.

Best regards,
-Karl



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-11-01 17:03 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-01 12:55 Default lexical-binding to t Stefan Monnier
2024-11-01 14:42 ` Gerd Möllmann
2024-11-01 17:03 ` Karl Fogel

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).