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

Reply via email to