emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] org-review-schedule


From: Alan Schmitt
Subject: Re: [O] org-review-schedule
Date: Sat, 26 Apr 2014 10:57:40 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (darwin)

Hi Nicolas,

I've changed all of these, and I will keep testing it over the next few
days. I have one question remaining, though.

On 2014-04-25 08:51, Nicolas Goaziou <address@hidden> writes:

>>         (if (time-less-p nt (current-time)) nt)))))
>
> This is a matter of taste, but I find one-armed `if' a bit confusing.
> Since return value matters, I suggest to use
>
>   (and (time-less-p nt (current-time)) nt)

Why not use "(when (time-less-p nt (current-time)) nt)" instead of "and"
here?

Thanks,

Alan



reply via email to

[Prev in Thread] Current Thread [Next in Thread]