Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

2024-05-07 Thread Bastien Guerry
Ihor Radchenko writes: > And they reverted the warning at the end. > https://git.savannah.gnu.org/cgit/emacs.git/commit?id=d51de0c5d90117bc1dc4bc5bc700253d71cd4579 Yes, it makes sense. -- Bastien Guerry

Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

2024-05-07 Thread Ihor Radchenko
Ihor Radchenko writes: >> ⛔ Warning (lexical-binding): File >> /home/bzg/.emacs.d/org-clock-save.el lacks `lexical-binding' >> directive on its first line >> >> The attached patch should fix this. Is it okay to add it to bugfix? > > I think that it is premature. > This is a warning recentl

Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

2024-05-06 Thread Bastien Guerry
Ihor Radchenko writes: > I think that it is premature. > This is a warning recently introduced on Emacs master, and it is still > being debated upon. See > https://yhetil.org/emacs-devel/50e29988-d354-4d10-990f-31828dbe6...@gmail.com/t/#u Okay, fair enough. -- Bastien Guerry

Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

2024-05-05 Thread Ihor Radchenko
Bastien writes: > Recent versions of Emacs complain about org-clock-save.el: > > ⛔ Warning (lexical-binding): File > /home/bzg/.emacs.d/org-clock-save.el lacks `lexical-binding' > directive on its first line > > The attached patch should fix this. Is it okay to add it to bugfix? I think t

[PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

2024-05-05 Thread Bastien
Recent versions of Emacs complain about org-clock-save.el: ⛔ Warning (lexical-binding): File /home/bzg/.emacs.d/org-clock-save.el lacks `lexical-binding' directive on its first line The attached patch should fix this. Is it okay to add it to bugfix? Thanks, -- Bastien diff --git a/lisp