Incorrect Comment?

[ccorcos]

Reading through the code here and I think this comment is incorrect:

jsondiffpatch/src/filters/arrays.js

Line 338 in ef38c5d

// insert items, in reverse order to avoid moving our own floor

We're performing inserts in order. And the logic is correct because that's the only way we can build up a really long array.

Just thought I'd let you know. And nice job -- your code is pretty clean!

[benjamine]

hi @ccorcos , sorry for the delay, and thanks for the comment!
I read it again and I think you're right!, probably my mistake copying the comment from the removals a bit above, but inserts make sense to make them in incremental index order, as those indices in the delta are already expressed as in the final (right) array.