[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
- Re: [O] org-review-schedule, (continued)
- Re: [O] org-review-schedule, Bastien, 2014/04/19
- Re: [O] org-review-schedule, Alan Schmitt, 2014/04/19
- Re: [O] org-review-schedule, Alan Schmitt, 2014/04/25
- Re: [O] org-review-schedule, Nicolas Goaziou, 2014/04/25
- Re: [O] org-review-schedule, Alan Schmitt, 2014/04/25
- Re: [O] org-review-schedule, Nicolas Goaziou, 2014/04/25
- Re: [O] org-review-schedule, Alan Schmitt, 2014/04/27
- Re: [O] org-review-schedule, AW, 2014/04/28
- Re: [O] org-review-schedule, Alan Schmitt, 2014/04/28
- Re: [O] org-review-schedule,
Alan Schmitt <=
- Re: [O] org-review-schedule, Thorsten Jolitz, 2014/04/26
- Re: [O] org-review-schedule, Nicolas Goaziou, 2014/04/26
- Re: [O] org-review-schedule, Alan Schmitt, 2014/04/27