Skip to content

merge updated code #1

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

Merged
merged 12 commits into from
Apr 18, 2021
Merged

merge updated code #1

merged 12 commits into from
Apr 18, 2021

Conversation

foxdd
Copy link
Owner

@foxdd foxdd commented Apr 18, 2021

No description provided.

dbaggerman and others added 12 commits March 22, 2021 17:14
Turns out it only takes a handful of ASCII characters before the old
bloom filter became 01111111 and matched everything. This implements a
better bloom filter which drastically reduces collisions and makes the
filter far more effective.

Since the new hashing takes a little more computation, using a lookup
table is a modest improvement over recalculating every character
processed.
Clean up new bloom filter implementation
Fix vimscript comments and files
Copy link
Owner Author

@foxdd foxdd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@foxdd foxdd merged commit a663333 into foxdd:master Apr 18, 2021
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

Successfully merging this pull request may close these issues.

4 participants