Achim Gratz <strom...@nexgo.de> writes:
> Bastien writes:
>> Fixed, thanks!
>
> Uh… no, you switched the wrong function in the second line (which
> incidentally indicates that this branch of the conditional never sees
> any testing). Fixed in master by using org-float-time.
Indeed, thanks for fixing this.
> I checked in
> XEmacs 21.5 and it has float-time, so maybe we wouldn't need a
> compatibility function after all, but I don't know when XEmacs
> introduced float-time.
Mhh... the less compatibility functions the better.
I found this ChangeLog entry in XEmacs 21.5-b33:
2010-05-30 Aidan Kehoe <keh...@parhasard.net>
* subr.el (float-time): Add this function, available in
editfns.c in GNU.
And time-to-seconds seems absent -- but it seems absent from
*any* version of XEmacs I could check (21.4.1, 21.4.9, 21.5.9).
So yes, I think it's safe to remove this compat function.
Let me know if you find time-to-seconds in <2010 versions
of XEmacs.
Thanks,
--
Bastien
