[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: Julian Foad <julianfoad_at_btopenworld.com>
Date: Wed, 23 May 2012 09:05:01 +0100 (BST)

Greg Stein wrote:
> Branko ─îibej wrote:
>> I'd really like to see you explain why this change of yours (33 ->
>> 33^4) is relevant in practice. It's not at all clear that this
>> multiplier gives a better key distribution than the time-honoured 33.
>
> Actually, there are some reasoned/studied arguments for 33 ("it works
> well, but nobody knows why"). And 33^4 is likely a poor replacement
> :-P

Stefan's version is not using a different multiplier, it's just unrolling the loop to do four of the multiplications at once, AFAICT.

- Julian
Received on 2012-05-23 10:05:41 CEST

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