Bug 72636 – fuzzy_match doesn't recognize "Dan B" or "hyatt"

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Ryosuke Niwa]

fuzzy_match should be able to recognize "Dan B" and "hyatt".

[Ryosuke Niwa]

Created attachment 115653 [details]
fixes the bug

[Eric Seidel (no email)]

Comment on attachment 115653 [details]
fixes the bug

Curious if you looked at bug 26533 at all.  It had to solve these very same problems.  (You're doing a much nicer job, of course.) :)

[Ryosuke Niwa]

(In reply to comment #2)
> (From update of attachment 115653 [details])
> Curious if you looked at bug 26533 at all.  It had to solve these very same problems.  (You're doing a much nicer job, of course.) :)

Yeah, I've looked at your script. What I'm trying to do is to minimize the hard-coded list of things so that it can adopt to new misspells and new contributors better. Eventually, fuzz_match should be able to recognize any name listed in your original script :)

[Ryosuke Niwa]

Do you have any suggestion to the patch?

[Eric Seidel (no email)]

Comment on attachment 115653 [details]
fixes the bug

View in context: https://bugs.webkit.org/attachment.cgi?id=115653&action=review

Seems OK.  You should add a comment about the commented out code, since we don't normally commit commented out code.

> Tools/Scripts/webkitpy/common/config/committers.py:539
> +                if email[:len(string)] == string:

Why not just .startswith?

> Tools/Scripts/webkitpy/common/config/committers.py:545
> +        if not ' ' in contributor.full_name:

' ' not in foo
works too.

> Tools/Scripts/webkitpy/common/config/committers.py:550
> +        first_name = split_fullname[0]
> +        last_name = split_fullname[-1]
> +        return first_name, last_name, first_name + last_name[0], first_name + ' ' + last_name[0]

This is kinda a big hack, but OK.  You might want to put this on Contributor itself in a later patch. Not sure.

[Ryosuke Niwa]

Thanks for the review. Will address all comments and land.

[Ryosuke Niwa]

Committed r100700: <http://trac.webkit.org/changeset/100700>