Skip to content

Adjustment for the removal of IncrementalEdit(offset:length:replacement:) 🤝 swift-syntax#2588 #1158

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Apr 4, 2024

This initializer was only introduced very recently since IncrementalEdit stores the replacement bytes instead of just the replacement length. We are now changing IncrementalEdit store the range it’s replacing as Range<AbsolutePosition> instead of ByteSourceRange.

Companion of swiftlang/swift-syntax#2588

…ent:)`

This initializer was only introduced very recently since `IncrementalEdit` stores the replacement bytes instead of just the replacement length. We are now changing `IncrementalEdit` store the range it’s replacing as `Range<AbsolutePosition>` instead of `ByteSourceRange`.

Companion of swiftlang/swift-syntax#2588
@ahoppen ahoppen requested a review from benlangmuir as a code owner April 4, 2024 17:08
@ahoppen ahoppen requested a review from bnbarham April 4, 2024 17:08
@ahoppen
Copy link
Member Author

ahoppen commented Apr 4, 2024

swiftlang/swift-syntax#2588

@swift-ci Please test

@ahoppen ahoppen changed the title Adjustment for the removal of IncrementalEdit(offset:length:replacement:) Adjustment for the removal of IncrementalEdit(offset:length:replacement:) 🤝 swift-syntax#2588 Apr 11, 2024
@ahoppen ahoppen merged commit b9af0cf into swiftlang:main Apr 19, 2024
3 checks passed
@ahoppen ahoppen deleted the incremental-edit-takes-range branch April 19, 2024 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants