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

Fix Jaro and Jaro-Winkler panic #6

Merged
merged 1 commit into from
Aug 24, 2016
Merged

Fix Jaro and Jaro-Winkler panic #6

merged 1 commit into from
Aug 24, 2016

Conversation

dguo
Copy link
Member

@dguo dguo commented Aug 24, 2016

When given two strings both with length one, Jaro and Jaro-Winkler would panic
with an arithmetic operation overflow. Fixes #5

When given two strings both with length one, Jaro and Jaro-Winkler would panic
with an arithmetic operation overflow.
@dguo dguo added the bug label Aug 24, 2016
@dguo dguo merged commit 3dfff3d into master Aug 24, 2016
@dguo dguo deleted the jaro-overflow branch August 24, 2016 02:53
jnqnfe added a commit to jnqnfe/strsim-rs that referenced this pull request Nov 4, 2018
use vec! for `curr_distances` construction, as with jaro optimisation rapidfuzz#3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant