|
From: | Paul Eggert |
Subject: | bug#36940: tests slowness and failure after recent Tramp changes |
Date: | Fri, 9 Aug 2019 18:39:13 -0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
Michael Albinus wrote:
With that patch, your test passes. (Damn time arithmetic!)
No, time arithmetic is fun! :-)Your patch has several instances of (floor (float-time X)) that would be slightly better if written as (time-convert X 'integer), as the former has (unlikely) rounding errors that the latter lacks.
I assume that the test now uses integer timestamps because file systems routinely drop low-order parts of timestamps. But won't the test continue to have problems with filesystems whose timestamp resolution is 2 seconds? I have heard of such things.
[Prev in Thread] | Current Thread | [Next in Thread] |