Pattern too long for this browser
GoogleCodeExporter opened this issue · comments
What steps will reproduce the problem?
var dmp = new diff_match_patch();
var last = 'abcdefghij , h : 1 , t : 1 abcdefghij , h : 1 , t :
1 abcdefghij , h : 0 , t : 1';
var current = 'abcdefghij , h : 0 , t : 1 abcdefghij , h : 0
, t : 1 abcdefghij , h : 0 , t : 1';
var patches = dmp.patch_make(current, last);
var mod_current = 'abcdefghij , h : 0 , t : 1 abcdefghij , h : 1
, t : 1 abcdefghij , h : 0 , t : 1';
var res = dmp.patch_apply(patches, mod_current);
What is the expected output? What do you see instead?
Expect patch to succeed or fail. Actually it throws an error "Pattern too
long for this browser". Only affects JavaScript version (the bug is also
in the Python version but is never expressed).
Fathei Ali reported this error and I tracked it down to an indexing bug in
patch_splitMax. Most of the time this would have no effect, but very
occasionally it fails to split a long patch.
A new version has been pushed which corrects this bug and unit tests have
been added in all languages for verification.
Original issue reported on code.google.com by neil.fra...@gmail.com
on 4 Dec 2008 at 5:50
Original comment by neil.fra...@gmail.com
on 4 Dec 2008 at 5:51
- Changed state: Fixed
- Added labels: ****
- Removed labels: ****
Issue 24 has been merged into this issue.
Original comment by neil.fra...@gmail.com
on 7 Sep 2009 at 12:45
- Added labels: ****
- Removed labels: ****