From: Stefan Monnier <monnier@IRO.UMontreal.CA>
To: Glenn Morris <rgm@gnu.org>
Cc: 30051-done@debbugs.gnu.org, Richard.Kim1@synopsys.com
Subject: bug#30051: 27.0.50; diary-lib.el uses lexical scoping when dynamic scope is needed
Date: Wed, 10 Jan 2018 14:27:09 -0500 [thread overview]
Message-ID: <jwv1sixg26m.fsf-monnier+emacsbugs@gnu.org> (raw)
In-Reply-To: <u37esqilym.fsf@fencepost.gnu.org> (Glenn Morris's message of "Tue, 09 Jan 2018 19:41:53 -0500")
>> Debugger entered--Lisp error: (void-variable number)
>> appt-make-list()
>> run-hooks(diary-hook)
>> diary-list-entries((1 8 2018) 1)
[...]
>> This appears to be due to using lexical binding for diary-lib.el.
Indeed.
>> However appt-make-list depends on dynamically scoped `number' variable
>> to be defined by diary-list-entries as documented in the doc-string:
>>
>> We assume that the variables DATE and NUMBER hold the arguments that
>> `diary-list-entries' received.
Indeed, thanks. I pushed the patch below which should fix this problem.
Thank you,
Stefan
diff --git a/lisp/calendar/diary-lib.el b/lisp/calendar/diary-lib.el
index 159dd9ba3d..c327717c8a 100644
--- a/lisp/calendar/diary-lib.el
+++ b/lisp/calendar/diary-lib.el
@@ -151,8 +151,10 @@ diary-comment-end
:group 'diary)
(defcustom diary-hook nil
- "List of functions called after the display of the diary.
-Used for example by the appointment package - see `appt-activate'."
+ "Hook run after displaying the diary.
+Used for example by the appointment package - see `appt-activate'.
+The variables `number' and `original-date' are dynamically bound around
+the call."
:type 'hook
:group 'diary)
@@ -779,10 +781,10 @@ diary-list-entries
`diary-hook' runs last, after the diary is displayed.
This is used e.g. by `appt-check'.
-Functions called by these hooks may use the variables ORIGINAL-DATE
-and NUMBER, which are the arguments with which this function was called.
-Note that hook functions should _not_ use DATE, but ORIGINAL-DATE.
-\(Sexp diary entries may use DATE - see `diary-list-sexp-entries'.)
+Functions called by these hooks may use the variables `original-date'
+and `number', which are the arguments with which this function was called.
+Note that hook functions should _not_ use `date', but `original-date'.
+\(Sexp diary entries may use `date' - see `diary-list-sexp-entries'.)
This function displays the list using `diary-display-function', unless
LIST-ONLY is non-nil, in which case it just returns the list."
@@ -872,7 +874,9 @@ diary-list-entries
(copy-sequence
(car display-buffer-fallback-action))))))
(funcall diary-display-function)))
- (run-hooks 'diary-hook)))))
+ (calendar-dlet* ((number number)
+ (original-date original-date))
+ (run-hooks 'diary-hook))))))
(and temp-buff (buffer-name temp-buff) (kill-buffer temp-buff)))
(or d-incp (message "Preparing diary...done"))
diary-entries-list)))
prev parent reply other threads:[~2018-01-10 19:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-09 21:07 bug#30051: 27.0.50; diary-lib.el uses lexical scoping when dynamic scope is needed Richard Kim
2018-01-09 21:15 ` bug#30051: Acknowledgement (27.0.50; diary-lib.el uses lexical scoping when dynamic scope is needed) Richard Kim
2018-01-10 0:41 ` bug#30051: 27.0.50; diary-lib.el uses lexical scoping when dynamic scope is needed Glenn Morris
2018-01-10 19:27 ` Stefan Monnier [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=jwv1sixg26m.fsf-monnier+emacsbugs@gnu.org \
--to=monnier@iro.umontreal.ca \
--cc=30051-done@debbugs.gnu.org \
--cc=Richard.Kim1@synopsys.com \
--cc=rgm@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).