Allow to specify multiple components #2
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This doesn't work properly yet.
roughAlgorithm
works, it's the easiest due tosplitAtPositionLine
niceAlgorithm
doesn't work, it seems my offset logic doesn't play well withsplitAtPosition
... I wasn't able to figure out why yetfancyAlgorithm
: haven't tried yetMy idea was:
An alternative I tried was: using the original cabal file bytestring for each component, then returning position + bytestring-to-insert and combine that in one pass. But here we also need to compute offsets, which is the main difficulty.