[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1803899 - in /subversion/trunk/subversion: libsvn_ra_serf/ libsvn_subr/ mod_dav_svn/

From: Evgeny Kotkov <evgeny.kotkov_at_visualsvn.com>
Date: Thu, 3 Aug 2017 14:53:57 +0300

Stefan Sperling <stsp_at_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.

Thanks,
Evgeny Kotkov
Received on 2017-08-03 13:54:22 CEST

This is an archived mail posted to the Subversion Dev mailing list.