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

Re: svn commit: r1615412 - /subversion/branches/authzperf/subversion/libsvn_subr/config.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Wed, 06 Aug 2014 07:21:53 +0200

On 06.08.2014 07:19, Branko Čibej wrote:
> On 03.08.2014 17:41, stefan2_at_apache.org wrote:
>> Author: stefan2
>> Date: Sun Aug 3 15:41:02 2014
>> New Revision: 1615412
>>
>> URL: http://svn.apache.org/r1615412
>> Log:
>> * subversion/libsvn_subr/config.c
>> (svn_config_create2,
>> svn_config_get_config,
>> svn_config__get_default_config,
>> svn_config_addsection,
>> svn_config_copy_config): Use our custom hash function for faster
>> insertion and lookup.
>
> Stefan, there's no reason to do this on the authzperf branch; it has
> very little to do with authz performance, and it just clutters up the
> diff wrt. trunk for no really good reason.
>
> If you really think this change is worthwhile, then it's OK to do it
> on trunk –- *after* we branch 1.9.

Same for r1615419, IMO.

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco | Realising the impossibilities of Big Data
e. brane_at_wandisco.com
Received on 2014-08-06 07:21:55 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.