emacs-orgmode
[Top][All Lists]
Advanced

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

Re: [O] [PATCH] org-timer.el: Use hh:mm:ss format instead of minutes


From: Nicolas Goaziou
Subject: Re: [O] [PATCH] org-timer.el: Use hh:mm:ss format instead of minutes
Date: Thu, 07 May 2015 22:09:06 +0200

Brice Waegenire <address@hidden> writes:

> Thanks for help on this!
> Here is the last version of the patch taking into account all of your
> comments.

Thank you.

>                    (read-from-minibuffer
> -                   "How many minutes left? "
> -                   (if (not (eq org-timer-default-timer 0))
> -                       (number-to-string org-timer-default-timer))))))
> +                   "How much time left? (minutes or h:mm:ss) "
> +                   (when (not (string-equal org-timer-default-timer "0"))

Nitpick: `unless'

> +                       (eval org-timer-default-timer))))))

Since `org-timer-default-timer' is a string, there's no need to eval it.

BTW, did you sign FSF papers already? If not, you need to add TINYCHANGE
to the end of the commit message.


Regards,



reply via email to

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