On Wed, Aug 02, 2017 at 06:49:55PM -0000, kot...@apache.org wrote:
> +svn_boolean_t
> +svn_ra_serf__is_local_network(svn_ra_serf__session_t *session)
> +{
> + return session->conn_latency >= 0 &&
> + session->conn_latency < apr_time_from_msec(5);
> +}
"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()?
