RE: svn commit: r1580914 - /subversion/trunk/subversion/libsvn_subr/io.c
From: Bert Huijben <bert_at_qqmail.nl>
Date: Mon, 24 Mar 2014 18:07:30 +0100
> -----Original Message-----
As noted on IRC, this patch can make us wait for the next... next second... by determining to what second we wait later.
I don't see why this patch improves the current behavior of this function.
The if case you removed checked if the time to wait for is in the past, which may happen for various reasons... E.g.g when the stat operation is somehow slow... or because the OS-scheduler paged us out and only returned later.
In these cases we now just wait one second extra...
Originally we always waited (<= 1.5), but then we made use of the delay time to determine if we really had to wait. Doing this with this code made sure the new code was not *more expensive* than the old check.
But now the code is updated to still wait the old long time... after we spend time determining whether we should wait at all.
This is an archived mail posted to the Subversion Dev mailing list.