Comments on: A diff/patch solution for Flash: a proof of concept Monte Carlo algorithm for comparing text versions in as3

By: hristo

hristo — Tue, 27 Jul 2010 09:31:06 +0000

Hey Liudas,

Sorry for the late reply. I’ll look into why it started doing this, could be one of the recent fixes. So funny that we haven’t noticed this.

Thanks!

By: Liudas

Liudas — Tue, 29 Jun 2010 14:33:11 +0000

There are small bug( maybe not bug) in your code – when two texts are equal, diff returns both text joined. Texts check before going to diff, solves this.

By: Liudas

Liudas — Wed, 23 Jun 2010 13:27:03 +0000

I don’t understand counting of index in “@@”. It would be nice to see your “human-readable version of the solution” code.

Thanks,
Liudas

By: Rich

Rich — Thu, 01 Apr 2010 21:12:18 +0000

Hey hristo!

Great work on the class. Everything’s running quite smoothly nowadays. I’ll be in touch as I do more experimentation, especially with using third party diff/patch suites with the diffs produced by Patch.as.

Have a wonderful day! :)
-Rich

By: hristo

hristo — Wed, 31 Mar 2010 09:52:18 +0000

Hey Jack, it’s all fixed and all, I was just pondering on some features.
I’ll update the files here in a minute and I’ll also mail you the right sources.

Neil’s code, which is the Bitap class, is released under Apache 2. The other stuff just comes without warranty :)

By: Jack

Jack — Mon, 29 Mar 2010 06:08:14 +0000

any word on the new release? This is perfect for my app, and I’m anxious to try it out :]

what license will you release it as?

By: hristo

hristo — Thu, 18 Mar 2010 11:03:12 +0000

We’re working with Rich to clear the classes from bugs and to make them more useful for memo management. Rich is doing the unit testing, something I’ve always underestimated and stupidly avoided.

It all looks very good right now, but I’ll postpone updates for a few more days so that we are certain that everything is bug-free.

By: Rich

Rich — Mon, 15 Mar 2010 21:38:12 +0000

Update: Swapping the parameters seems to help:

p.diff(‘I said, hi there! How are you doing?’, ‘hi there. How you doing?’);

yields:

@@ -1,29 +1,17 @@
-I said,
hi there
-!
+.
How
- are
you
@@ -33,4 +21,4 @@
ing?

BUT, shouldn’t the diff work both ways? (e.g. in one case, we’re inserting the string ‘ are’, while in the other case, we’re removing the string ‘ are’.)

By: Rich

Rich — Mon, 15 Mar 2010 21:36:08 +0000

Hi hristo!

Rich again. I’m just now getting around to experimenting with your Patch class. I’m confused, though. When I run this:

var p:Patch = new Patch();
p.diff(‘hi there. How you doing?’, ‘I said, hi there! How are you doing?’);
trace(p.string);

the only diff I get is for ‘I said, ‘. In other words, this is all that trace(p.string) outputs:

@@ -1,17 +1,29 @@
+I said,
hi the

There are two more differences – the ‘.’ becoming a ‘!’ and the insertion of ‘ are’. Am I doing something wrong?

Thanks! :)
-Rich

By: hristo

hristo — Thu, 21 Jan 2010 12:08:57 +0000

Thanks!
Let me know how it works, I’ve been wondering whether it’ll be good for structured text