Results 508 comments of hhugo

@vouillon, can you rebase this PR using the latest version of dune 3.17 ? I think they've drop the change that caused most dune file to be reformatted.

I plan to merge this PR before merging #1724

> Just to make sure we are on the same page, my intention is to wait for #1640 to be merged, rebase on it, and then address the rest of...

#1640 has been merged, I'll let you rebase this one

I'm not very happy with the callback based approach used on `Link_js`. Without thinking much about it, I would assume that one could reconstruct the edit instruction inside `let copy...

Regarding the representations of edits. We currently have per line instructions. Have you considered using a different approach (closer to what's done on master with `reloc`) using "copy n lines...

> > My understanding is that the Line_edits change is what gives use the most part of the improvement. Could we achieve similar perf using sections (Index sourcemap) ? >...

I've opened #1715 instead. It removes the need to change encoded mappings.

Closing in favor of #1715

(The CI is running the testsuite on windows, with the mingw OCaml port and gmp)