all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Glenn Morris <rgm@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: does Calendar understand DST rules for past years?
Date: Sun, 24 Dec 2006 19:07:12 -0500	[thread overview]
Message-ID: <b2k60get3j.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <E1GxSJb-0006He-TM@fencepost.gnu.org> (Richard Stallman's message of "Thu, 21 Dec 2006 13:06:47 -0500")

Richard Stallman wrote:

> I asked Glenn Morris to look at this, but since he is away for a couple
> of weeks, I ask if anyone else can figure out what is right here.

I can at least reply to mail, even if I can't access CVS.

> man/calendar.el says
>
>       The dates used by Emacs for holidays are based on @emph{current
>     practice}, not historical fact.  Historically, for instance, the start
>     of daylight savings time and even its existence have varied from year to
>     year, but present United States law mandates that daylight savings time
>     begins on the first Sunday in April.  When the daylight savings rules
>     are set up for the United States, Emacs always uses the present
>     definition, even though it is wrong for some prior years.
>
> but the code seems to test calendar-dst-check-each-year-flag, which is
> normally true. 

The above manual text is out of date following my recent changes to
cal-dst (one month ago?). Above is how it used to work. Now if the
flag is true (as you say), it checks each year as needed whenever the
dst start/end dates for the year are first required. Then it caches
the results for that year and reuses them if needed.

If the flag is false, it works as the manual says.

> I do not understand the code which that calls; it needs a lot more
> comments. Can you please add comments?

Comments not critical, can add those on my return.

> But I can't see any sense in checking each year unless there is a
> facility for recording the changes in rules between years. 

Results for each year stored in the cache variable I added a month or
so ago.

> In that case, the drawback which is documented is not reallky true.

Not sure what drawback is. Above manual text is now incorrect when
flag is true. Should have updated the manual when I made my changes,
sorry.

  reply	other threads:[~2006-12-25  0:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-21 18:06 does Calendar understand DST rules for past years? Richard Stallman
2006-12-25  0:07 ` Glenn Morris [this message]
2006-12-25 16:53   ` Richard Stallman
2006-12-25 23:29     ` Kevin Ryde
2006-12-26 17:22       ` Richard Stallman
2006-12-26 22:37     ` Glenn Morris

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=b2k60get3j.fsf@fencepost.gnu.org \
    --to=rgm@gnu.org \
    --cc=emacs-devel@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.