Re: Implements a pseudo-patience-diff SequenceMatcher for Rietveld. (issue 6613055)

[mar...@chromium.org]

I didn't review the algorithm but the rest of the change lgtm.

cc'ing the mailing list for more comments.


https://codereview.appspot.com/6613055/diff/2002/codereview/patiencediff.py
File codereview/patiencediff.py (right):

https://codereview.appspot.com/6613055/diff/2002/codereview/patiencediff.py#newcode18
codereview/patiencediff.py:18: """Provides s SequenceMatcher that
prefers longer "first" matches to longer
remove extra 's'

https://codereview.appspot.com/6613055/diff/2002/codereview/patiencediff.py#newcode22
codereview/patiencediff.py:22: def __init__(self, isjunk=None, a='',
b=''):
Remove lines 21-24, they are not needed.

https://codereview.appspot.com/6613055/diff/2002/codereview/patiencediff.py#newcode26
codereview/patiencediff.py:26: """Return list of triples describing
matching subsequences.
Returns

https://codereview.appspot.com/6613055/