> -----Original Message-----
> From: julianf...@apache.org [mailto:julianf...@apache.org]
> Sent: maandag 24 maart 2014 17:45
> To: comm...@subversion.apache.org
> Subject: svn commit: r1580914 -
> /subversion/trunk/subversion/libsvn_subr/io.c
>
> Author: julianfoad
> Date: Mon Mar 24 16:45:26 2014
> New Revision: 1580914
>
> URL: http://svn.apache.org/r1580914
> Log:
> * subversion/svn/notify.c
> (svn_io_sleep_for_timestamps): Simplify, eliminating a return path that
> looked
> like a potential source of bugs but was probably in fact safe.
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.
Bert
