[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] org-review-schedule
From: |
Thorsten Jolitz |
Subject: |
Re: [O] org-review-schedule |
Date: |
Sat, 26 Apr 2014 12:38:11 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Alan Schmitt <address@hidden> writes:
> 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?
the answer is in the quote already:
,---------------------------------------------
| Since return value matters, I suggest to use ...
`---------------------------------------------
--
cheers,
Thorsten