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

Minor refactoring of ERXStringUtilities.distance(). #312 #314

Merged
merged 3 commits into from
Nov 8, 2012
Merged

Minor refactoring of ERXStringUtilities.distance(). #312 #314

merged 3 commits into from
Nov 8, 2012

Commits on Nov 8, 2012

  1. Factors distance() implementation into new method. wocommunity#312 

    This is really just a cut and paste of the implementation of 'double
distance(String, String)' into 'int levenshteinDistance(String,
String)', and a replacement of the body of the former method with a
call to the latter. (We don't even need a cast to go from int to
double.) We also take the opportunity to deprecate distance(), and
mark it for potential future removal. Note that distance() still
passes all the tests in ERXStringUtilitiesTest.testDistance().
    @paulhoadley
    paulhoadley committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    41e6573 View commit details
    Browse the repository at this point in the history

  2. Adds test for ERXStringUtilities.levenshteinDistance(). wocommunity#312

    @paulhoadley
    paulhoadley committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    888ef95 View commit details
    Browse the repository at this point in the history

  3. Improves Javadoc for levenshteinDistance(). wocommunity#312

    @paulhoadley
    paulhoadley committed Nov 8, 2012
    Configuration menu
    Copy the full SHA
    b081b45 View commit details
    Browse the repository at this point in the history