Comments on: Diff for Javascript http://www.lshift.net/blog/2006/08/15/diff-for-javascript What happens at LShift Sun, 06 Jul 2008 01:49:25 +0000 http://wordpress.org/?v=2.0.12-alpha by: tonyg http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-85594 Wed, 20 Feb 2008 19:23:57 +0000 http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-85594 <p>You're right, Peter - computing a diff character-by-character seems to give spurious diffs! How odd. Have you dug into the code to find out why that might be?</p> <p>Splitting the string into words first gives a sensible answer:</p> <p>js> uneval(diff(str1.split(/\s+/), str2.split(/\s+/)));</p> <p>[{common:["The"]}, {file1:["red"], file2:[]}, {common:["brown"]}, {file1:[], file2:["spotted"]}, {common:["fox"]}, {file1:["jumped"], file2:["leaped"]}, {common:["over", "the", "rolling"]}, {file1:["log."], file2:["log"]}]</p> You’re right, Peter - computing a diff character-by-character seems to give spurious diffs! How odd. Have you dug into the code to find out why that might be?

Splitting the string into words first gives a sensible answer:

js> uneval(diff(str1.split(/\s+/), str2.split(/\s+/)));

[{common:[”The”]},
{file1:[”red”], file2:[]},
{common:[”brown”]},
{file1:[], file2:[”spotted”]},
{common:[”fox”]},
{file1:[”jumped”], file2:[”leaped”]},
{common:[”over”, “the”, “rolling”]},
{file1:[”log.”], file2:[”log”]}]

]]> by: Peter Pallos http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-85420 Tue, 19 Feb 2008 22:14:55 +0000 http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-85420 <p>The code fails with the two sentences that I found on a different site:</p> <p>str1 = "The red brown fox jumped over the rolling log."; str2 = "The brown spotted fox leaped over the rolling log"; alert(uneval(diff(str1, str2)));</p> <p>For example, a part of the result array is: {common:["o", "v"]}, {file1:["e"], file2:["e"]}, {common:["r"]}</p> The code fails with the two sentences that I found on a different site:

str1 = “The red brown fox jumped over the rolling log.”;
str2 = “The brown spotted fox leaped over the rolling log”;
alert(uneval(diff(str1, str2)));

For example, a part of the result array is:
{common:[”o”, “v”]}, {file1:[”e”], file2:[”e”]}, {common:[”r”]}

]]> by: frank http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-57695 Tue, 22 May 2007 16:01:39 +0000 http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-57695 <p>OK , perhaps i'll try to do a new one by myself . thanx</p> OK , perhaps i’ll try to do a new one by myself . thanx

]]> by: tonyg http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-57626 Tue, 22 May 2007 12:15:04 +0000 http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-57626 <p>Hi Frank, unfortunately I've not yet returned to this little project. I'll be sure to post when I do, though.</p> Hi Frank, unfortunately I’ve not yet returned to this little project. I’ll be sure to post when I do, though.

]]> by: frank http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-57485 Tue, 22 May 2007 03:48:06 +0000 http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-57485 <p>How is your diff3 going on ? If it finish , may I see it's source code by JS .</p> <p>Thanks alot</p> How is your diff3 going on ?
If it finish , may I see it’s source code by JS .

Thanks alot

]]> by: mikeb http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-8830 Sun, 27 Aug 2006 07:48:16 +0000 http://www.lshift.net/blog/2006/08/15/diff-for-javascript#comment-8830 <p>Also see http://neil.fraser.name/software/diff_match_patch/</p> Also see http://neil.fraser.name/software/diff_match_patch/

]]>