Message326236
I created a new PR based on Tim's t ^= t << 7 idea, except that I'm using << 1 instead, to have more mixing in the lower bits.
With the standard FNV multiplier on 64 bits, I did get collisions while testing. I haven't figured out exactly why these occurred, but it's probably due to the high number of 0 bits. Instead, I chose 3**41 as multiplier. But of course, there are still plenty of bikeshedding opportunities for the multiplier... |
|
Date |
User |
Action |
Args |
2018-09-24 13:48:28 | jdemeyer | set | recipients:
+ jdemeyer, tim.peters, rhettinger, mark.dickinson, eric.smith, sir-sigurd |
2018-09-24 13:48:28 | jdemeyer | set | messageid: <1537796908.12.0.956365154283.issue34751@psf.upfronthosting.co.za> |
2018-09-24 13:48:28 | jdemeyer | link | issue34751 messages |
2018-09-24 13:48:28 | jdemeyer | create | |
|