This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author jdemeyer
Recipients eric.smith, jdemeyer, mark.dickinson, rhettinger, sir-sigurd, tim.peters
Date 2018-09-24.13:48:28
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1537796908.12.0.956365154283.issue34751@psf.upfronthosting.co.za>
In-reply-to
Content
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...
History
Date User Action Args
2018-09-24 13:48:28jdemeyersetrecipients: + jdemeyer, tim.peters, rhettinger, mark.dickinson, eric.smith, sir-sigurd
2018-09-24 13:48:28jdemeyersetmessageid: <1537796908.12.0.956365154283.issue34751@psf.upfronthosting.co.za>
2018-09-24 13:48:28jdemeyerlinkissue34751 messages
2018-09-24 13:48:28jdemeyercreate