Subject: Re: bug#50195: [PATCH] Adding diary-offset, a diary-sexp offsetting another diary-sexp.

From: Lars Ingebrigtsen <[email-masked]>
Date: Wed, 25 Aug 2021 13:31:49 +0200

Yuchen Pei <[email-masked]> writes:

> Let me know what you think. Would you like me to add some tests?

Looks useful -- some tests would be nice.

> +;; To be called from diary-sexp-entry, where DATE, ENTRY are bound.
> +(defun diary-offset (sexp days)
> +  "Offsetted diary entry.
> +Entry applies if the date is DAYS days after another diary-sexp SEXP."
> +  (with-no-warnings (defvar date) (defvar entry))
> +  (integerp days)
> +  (let ((date

The (integerp days) here doesn't do anything -- is it supposed to be
(when (integerp days)

?

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