search

dgryski / go-farm

go-farm: a pure-Go farmhash implementation
MIT License
238 stars 22 forks source link

Fix a bug in _uo #1

Closed leventov closed 9 years ago

leventov commented 9 years ago

https://github.com/google/farmhash/blob/master/dev/farmhashuo.cc#L23

dgryski commented 9 years ago

Thanks!

dgryski commented 9 years ago

I'll write some more tests and make sure no other dumb bugs exist. I think this one got through because all my tests are short strings.

leventov commented 9 years ago

I'm pretty sure there is no more bugs. I was writing a Java implementation based on your work, and when fixed this typo it started to pass tests based directly on smoke testing from this file: https://github.com/google/farmhash/blob/master/src/farmhash.cc