bug#11260: 24.1.50; `holiday-sexp' may create entry twice

[Andreas Schwab]

Michael Heerdegen <michael_heerdegen@web.de> writes:

> the following happens also in Emacs 23.  Recipe:
>
> 1. emacs -Q
> 2. M-x calendar RET
> 3. evaluate: (require 'holidays)
> 4. evaluate:
>    (add-to-list
>     'calendar-holidays
>     '(holiday-sexp
>       '(list 4 17 2012)
>       "April 17th 2012"))
>
> 5. In the calendar, move point to April 17th 2012 and hit h.
>
> You'll see that "April 17th 2012" appears twice.

Your expression does not use year.

> | (defun holiday-sexp (sexp string)
> |   "Sexp holiday for dates in the calendar window.
> | SEXP is an expression in variable `year' that is evaluated to
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."