Re: svn commit: r1333326 - in /subversion/trunk/subversion: include/private/svn_hash_private.h libsvn_fs_fs/temp_serializer.c libsvn_subr/hash.c
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
>> 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!
Received on 2012-05-06 02:20:09 CEST
This is an archived mail posted to the Subversion Dev