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.

Sign up for GitHub

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

Jump to bottom

Fix two issues with fuzzy matching #970

Merged
merged 2 commits into from
Feb 20, 2023
Merged

Fix two issues with fuzzy matching #970

merged 2 commits into from
Feb 20, 2023

Commits on Feb 12, 2023

  1. In fuzzy matching, also .lower().strip() fuzzy candidates 

    This seems intended at easing fuzzy matching with trivial edits in the
msgstr (changing case and adding whitespace), but it was only done on
the new msgstr, not on the old mgstr candidates, so it was possible for
merging catalogs to miss messages.
    @jeanas
    jeanas committed Feb 12, 2023
    Configuration menu
    Copy the full SHA
    63bb71a View commit details
    Browse the repository at this point in the history

  2. Turn off difflib "autojunk" heuristic in fuzzy matching 

    difflib has a heuristic that used to make fuzzy matching unreliable for
>200char strings.  See

python/cpython#90825

Fixes #969
    @jeanas
    jeanas committed Feb 12, 2023
    Configuration menu
    Copy the full SHA
    c8b7ac5 View commit details
    Browse the repository at this point in the history