From: Thierry Volpiatto <thierry.volpiatto@gmail.com>
To: 15661@debbugs.gnu.org
Subject: bug#15661: 24.3.50; calendar.el broken
Date: Mon, 21 Oct 2013 14:15:17 +0200 [thread overview]
Message-ID: <87r4beu8fe.fsf@gmail.com> (raw)
In-Reply-To: <526409D4.20903@easy-emacs.de>
Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
> Am 21.10.2013 09:47, schrieb Thierry Volpiatto:
>> Andreas Röhler <andreas.roehler@easy-emacs.de> writes:
>>
>>> Am 20.10.2013 20:13, schrieb Glenn Morris:
>>>> Andreas Röhler wrote:
>>>>
>>>>> M-x diary-view-entries RET
>>>>>
>>>>> fails saying:
>>>>>
>>>>> Symbol's value as variable is void: displayed-month
>>>>
>>>> Works fine here. Recipe from emacs -Q?
>>>>
>>>
>>> Yes, started from emacs -Q
>>> Also got it from openSuse 12.3
>>
>> displayed-month among others is make local-var in calendar buffer,
>
> By the mode itself, which looks wrong,
Sure it is.
> as libraries outside like diary seem to make use of it also.
>
> so if
>> you call the function from there (d), it will work as expected,
>> otherwise the var(s) is/are unknown(s).
>>
>
> Get this error also from
>
> M-x diary-view-other-diary-entries RET
>
> while
>
> M-x diary-show-all-entries RET
>
> works
>
> ;;;;;;;;
>
> Seeing inside calendar.el
>
> (unless (boundp 'displayed-month) (setq displayed-month 1))
>
> which doesn't look recommended.
>
> IIUC
>
> (defvar displayed-month 1)
>
> should DTRT.
The right thing should be to remove all these vars and reimplement the code
properly.
--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
next prev parent reply other threads:[~2013-10-21 12:15 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <mailman.4352.1382287759.10748.bug-gnu-emacs@gnu.org>
2013-10-20 16:50 ` bug#15661: 24.3.50; calendar.el broken Andreas Röhler
2013-10-20 17:52 ` Alan Mackenzie
2013-10-21 13:14 ` Stefan Monnier
2013-10-22 9:58 ` Andreas Röhler
2013-10-22 14:32 ` Stefan Monnier
2013-10-20 18:13 ` Glenn Morris
2013-10-21 5:57 ` Andreas Röhler
2013-10-21 7:47 ` Thierry Volpiatto
2013-10-21 8:56 ` Andreas Röhler
2013-10-21 12:15 ` Thierry Volpiatto [this message]
2013-10-21 16:27 ` Glenn Morris
2013-10-22 19:38 ` Glenn Morris
2013-10-23 5:51 ` Thierry Volpiatto
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87r4beu8fe.fsf@gmail.com \
--to=thierry.volpiatto@gmail.com \
--cc=15661@debbugs.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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.