API messed up 1.0.0 -> 1.0.3 about diff-match-patch HOT 3 OPEN

Yea, the change happened here https://github.com/JackuB/diff-match-patch/pull/13/files#diff-168726dbe96b3ce427e7fedce31bb0bcR76

It's emulating array for backward compatibility. But the destructing use case is not taken care of.

1. We could add Symbol.iterator to the Diff object (this would require some kind of transpilation)
2. Publish this as breaking and go for v2 (and release 1.0.4 without this change)
3. Patch it to use array (current tests are passing)

diff_match_patch.Diff = function(op, text) {
  return [op, text];
};

Wondering if there is any reason they did that - most likely to keep the API 1:1 with rest of the implementations. Will try to look into it a bit more.

from diff-match-patch.

I was going to submit an issue in google but saw yours , thanks.
If google is happy with status quo, I'd propose option 2 as the one that can break in least possible ways, although it's a bit a strange api. Lets see what they say..

from diff-match-patch.

Released 1.0.4 with a rollback https://github.com/JackuB/diff-match-patch/releases/tag/1.0.4
Will hold on with releasing v2 as there is a chance they will rollback the API change

from diff-match-patch.