News for hackers. Stuff that matters.

A comment on MurmurHash 2.0. Parent comment

Re:MurmurHash 2.0 by jmhodges on Saturday June 20, 2009 @12:52AM (#666146)
Yes, and those "k = m" lines are the problem! Those assume unsigned* types! And 64-bit integers! Those multiplications and other operations are all wrong!

You didn't port this to Java, you ported a broken algorithm to Java. Way to go, though.

[ Parent | on HN ]
Re:MurmurHash 2.0 by jmhodges on Saturday June 20, 2009 @03:44AM (#666241)
Sorry, not 64-bit integers but you get my drift.
[ Parent | on HN ]