spaolacci / murmur3

Native MurmurHash3 Go implementation
BSD 3-Clause "New" or "Revised" License
947 stars 127 forks source link

Output not compatible with other implementations #3

Closed dgryski closed 10 years ago

dgryski commented 10 years ago

Because the seed is reset to 1 instead of 0, the output doesn't match other implementations of murmur3 hash :(

pedronis commented 10 years ago

standalone Sum64() wasn't changed so now it desagrees with New64() ... Sum64(). I can make a pull request but not immediately

pedronis commented 10 years ago

ok, made a PR for this with testing