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

Re: svn commit: r1502401 - in /subversion/branches/tristate-chunked-request/subversion: include/svn_config.h libsvn_ra_serf/serf.c libsvn_ra_serf/util.c libsvn_subr/config_file.c

From: Greg Stein <gstein_at_gmail.com>
Date: Thu, 11 Jul 2013 21:42:08 -0400

On Thu, Jul 11, 2013 at 8:11 PM, <breser_at_apache.org> wrote:
> Author: breser
> Date: Fri Jul 12 00:11:48 2013
> New Revision: 1502401
> URL: http://svn.apache.org/r1502401
> Log:
> Reimplement the tristate http-chunked-requests config option on top of the
> latest trunk implementation of boolean http-detect-chunking.

Ugh. Why the rename? I spent a bunch of effort bringing the name in
line with dev@ thinking. This option primarily controls if/when we
detect for chunked request capability on the connection.

> * subversion/libsvn_ra_serf/serf.c
> (load_config): Switch to using the tristate option.

If keeping a tristate, then yeah. I think this is the right
implementation. Very clean, and we have the ability to spread the
detection across ra_serf rather than a spurious/additional OPTIONS


Much of this revision deals with changing the option name. I'm tending
to disagree with that change.

On the tri-state itself... I'll start a new thread.

Received on 2013-07-12 03:42:41 CEST

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