GNU bug report logs - #50195
[PATCH] Adding diary-offset, a diary-sexp offsetting another diary-sexp.

Previous Next

Package: emacs;

Reported by: Yuchen Pei <hi <at> ypei.me>

Date: Wed, 25 Aug 2021 05:57:01 UTC

Severity: wishlist

Tags: patch

Fixed in version 28.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


Message #20 received at 50195 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: hi <at> ypei.me, 50195 <at> debbugs.gnu.org
Subject: Re: bug#50195: [PATCH] Adding diary-offset, a diary-sexp offsetting
 another diary-sexp.
Date: Wed, 25 Aug 2021 14:38:51 +0200
Eli Zaretskii <eliz <at> gnu.org> writes:

>> It's how all these functions in diary-lib are defined after the lexical
>> rewrite -- the `eval' depends on those dynamic variables.
>
> Ugh!  Is it worth it?  If we cannot convert these functions to use
> lexical-binding, maybe we should say lexical-binding: nil in the first
> line of the file, and remove all those gross kludges?

I think having all files lexically bound is a win in itself -- it means
that you know the Lisp dialect without looking at the mode line (and can
write code that depends on closures etc freely).

It does lead to these hacks here, yes, but it's already done, so...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




This bug report was last modified 3 years and 319 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.