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

Disable debug mode in hash functions #52

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

d3m3vilurr
Copy link
Contributor

a last step of the hash functions runs one more swap if activated debug mode.

this debug flags makes slow down the processing speed and it also always makes different result as fingerprint.

close #51

@lovell
Copy link
Owner

lovell commented Sep 2, 2024

Thanks for the PR, I need to update the CI environment as CentOS 7 is EOL, leave it with me.

@lovell
Copy link
Owner

lovell commented Sep 3, 2024

Please can you rebase against upstream/main to pick up the CI fixes.

a last step of the hash functions runs one more swap if activated debug
mode.

this debug flags  makes slow down the processing speed and it also always
makes different result as fingerprint.

close lovell#51
@d3m3vilurr
Copy link
Contributor Author

@lovell can you recheck this PR? I rebased it yesterday.

@lovell
Copy link
Owner

lovell commented Sep 5, 2024

Thank you, looks good, this will be semver major as this affects the public API.

As an aside, given the major version bump will also be dropping support for Node.js 10, I can now switch to BigInt for 64-bit results.

@lovell lovell merged commit c8ab3c4 into lovell:main Sep 5, 2024
23 checks passed
@lovell
Copy link
Owner

lovell commented Sep 5, 2024

v4.0.0

@d3m3vilurr d3m3vilurr deleted the disable-debug-mode branch September 5, 2024 13:15
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.

hash64 generate different value as other language implements
2 participants