CADbloke / google-diff-match-patch

Automatically exported from code.google.com/p/google-diff-match-patch

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

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: ****