Skip to content

Conversation

mvasilkov
Copy link
Contributor

Description

The FNV-1a hash function existed for the sole purpose of running its tests.

It should be safe to remove it.

@nbbeeken
Copy link
Contributor

Hi @mvasilkov thanks for your help! I've enabled the CI to run on the changes and if everything looks good I'll go ahead and merge this.
Just for posterity's sake this work is also tracked by NODE-1738

@nbbeeken
Copy link
Contributor

Oh dear, I'd love to merge this but something has been merged that caused a conflict, sorry about that, when you get the chance if you could resolve the conflicts I can merge this right away!

@mvasilkov
Copy link
Contributor Author

Hi, sure, updated the PR.

@nbbeeken nbbeeken merged commit b95f059 into mongodb:master Sep 28, 2020
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.

2 participants