[issue2986] difflib.SequenceMatcher not matching long sequences

Terry J. Reedy report at bugs.python.org
Fri Jul 9 03:12:05 CEST 2010


Terry J. Reedy <tjreedy at udel.edu> added the comment:

My proposal F, to expose the common frequency threshold as a fourth positional parameter with default 1, would do that: repeat current behavior. We should, and Eli and I would, add some of the anomalous cases to the test suite and verily that the default is to reproduce the current anomalies, and that passing None changes the result.

Any opinions, anyone, on 'common', 'thresh', 'threshold', or anything else as the new parameter name?

We will have to explain in the doc patch that the parameter is new in 2.7.1 to fix a partial bug and that giving any explicit value will make code not run with 2.7 (.0).

Exposing the set of common values as an instance attribute, as I proposed on pydev, would be a new feature not needed to fix the bug. So it should be limited to 3.2.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue2986>
_______________________________________


More information about the Python-bugs-list mailing list