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

Re: svn commit: r1333326 - in /subversion/trunk/subversion: include/private/svn_hash_private.h libsvn_fs_fs/temp_serializer.c libsvn_subr/hash.c

From: Stefan Fuhrmann <eqfox_at_web.de>
Date: Fri, 04 May 2012 16:24:45 +0200

Greg Stein wrote:
> On Thu, May 3, 2012 at 3:16 AM,<stefan2_at_apache.org> wrote:
>> Author: stefan2
>> Date: Thu May 3 07:16:11 2012
>> New Revision: 1333326
>>
>> URL: http://svn.apache.org/viewvc?rev=1333326&view=rev
>> Log:
>> Introduce private API functions that wrap apr_hash_make_custom
>> and return hash tables that are 2 to 4 times faster than the APR default.
>> Both yield repeatable results (each instance will store items in the same
>> order if the keys are the same). The first, svn_hask__make will return
>> a hash table that behaves like pre APR 1.4.6 default hashes.
>>
>> * subversion/include/private/svn_hash_private.h
>> (svn_hash__make, svn_hash__make_fast): new private API
> It would be great to just use svn_subr_private.h.

I've been mislead by svn_string_private.h & friends.
But having just one private header per lib definitely
makes more sense.

Moved in r1333340.
>> ...
>> +++ subversion/trunk/subversion/libsvn_subr/hash.c Thu May 3 07:16:11 2012
>> ...
>> +#if SVN_UNALIGNED_ACCESS_IS_OK
>> +# define READ_CHUNK(p)\
>> + *(const apr_uint32_t *)(p);
> erroneous trailing semicolon. this can stay on a single line, too.

Oops. Fixed in r1333354.

Thanks for the review!

-- Stefan^2.
Received on 2012-05-06 02:20:09 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.