derek-zhang / smhasher

Automatically exported from code.google.com/p/smhasher
0 stars 0 forks source link

Tail mixing logic is different from body in MurmurHash3_x86_32 #7

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I wanted to make sure this is intentional and not a mistake. The mixing logic 
for the tail (last 1 to 3 bytes) is different from the body mixing, and I was 
not sure why. Here is the tail mix:

k1 *= c1;
k1 = ROTL32(k1,16);
k1 *= c2; 
h1 ^= k1;

Original issue reported on code.google.com by frank...@gmail.com on 13 May 2011 at 10:18

GoogleCodeExporter commented 9 years ago
Yep, that's a typo. Fixed, and verification value for Murmur3 updated.

Original comment by tanj...@gmail.com on 20 May 2011 at 11:01