Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fnv-plus #10

Closed
tjwebb opened this issue Mar 1, 2014 · 2 comments
Closed

fnv-plus #10

tjwebb opened this issue Mar 1, 2014 · 2 comments

Comments

@tjwebb
Copy link

tjwebb commented Mar 1, 2014

Hey I ran across your project as I was researching bloom filters, and I noticed this on your website:
"Unfortunately I can't use the 64-bit trick in the linked post as JavaScript only supports bitwise operations on 32 bits."

I don't know if you'd be interested in this, but a little while ago I wrote a version of fnv with an expanded keyspace (up to 1024-bit): https://github.com/tjwebb/fnv-plus.

@jasondavies
Copy link
Owner

Thanks, but I already perform a simple optimisation to generate the second 32-bit hash: simply run one additional iteration of FNV on the first hash. Admittedly, I haven’t done any in-depth analysis on this step. It might be preferable to seed the second step using some input bits, for example.

@tjwebb
Copy link
Author

tjwebb commented Mar 2, 2014

Yea; the main advantage would just be if you wanted to support >32-bit keyspace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants