subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Evgeny Kotkov <evgeny.kot...@visualsvn.com>
Subject Re: svn commit: r1803899 - in /subversion/trunk/subversion: libsvn_ra_serf/ libsvn_subr/ mod_dav_svn/
Date Thu, 03 Aug 2017 12:31:36 GMT
Evgeny Kotkov <evgeny.kotkov@visualsvn.com> writes:

> Stefan Sperling <stsp@elego.de> writes:
>
>> "local network" is a rather blurry concept.
>> This will often return FALSE for local wireless connections, for
>> example, because of shared medium contention.
>>
>> I agree that 5msec is a well chosen latency threshold.
>> But why not name this function after the question it answers,
>> e.g. svn_ra_serf__is_low_latency_connection()?
>
> My original intention here was to have a function that can distinguish
> between LAN and WAN, using the information that we have — that's
> latency now and possibly something else in the future.  But indeed,
> considering local wireless connections, the name of the function could
> be misleading.
>
> Something like svn_ra_serf__is_low_latency_connection() would probably
> be more appropriate right now.  And, we could deal with the additional
> information about the connection to base the detection on, if and when
> we have that.
>
> I'll try to update the function name and the related comments.

Committed in https://svn.apache.org/r1803989


Regards,
Evgeny Kotkov

Mime
View raw message