-
Notifications
You must be signed in to change notification settings - Fork 447
Cherry-pick some post-4.2 changes into swift-4.2-branch #6
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
ahoppen
merged 25 commits into
swiftlang:swift-4.2-branch
from
allevato:swift-4.2-cherrypicks
Sep 12, 2018
Merged
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
d259fcc
Simplify AbsolutePosition offset calculation and support columns
5018466
Clarify comment
8044478
Un-rename property
b664867
Monomorphize AbsolutePosition.copy()
23502d3
Rename byteOffset to utf8Offset and remove utf16
b970c42
Re-add AbsolutePosition.swift
cc54246
Actually add offsets in add(columns:) and add(lines:size:)
60e37d0
Add accessors for source locations and test diagnostic emission (#16141)
harlanhaskins 3dc6e42
Manually cherry-pick tests from apple/swift:
allevato a2ed7fb
Add descriptions for SwiftSyntax errors (#16339)
harlanhaskins 54ea839
SwiftSyntax: Allow absolute position access for dangling nodes.
nkcsgexi 08fb8a0
Add incremental syntax tree deserialization to SwiftSyntax
ahoppen 7deb6b4
Make RawSyntax a struct
ahoppen f104c5f
Add type annotations to speed up compile time
ahoppen 156cc9f
Don't throw just because compilation fails
ahoppen 9bf9210
Record the nodes that have been reused during an incremental transfer
ahoppen 6294ff3
Refactor AbsolutePosition
ahoppen 84c6d0d
Make AbsolutePosition a value type
ahoppen b7deaa6
Remove validate methods
ahoppen 9cef71e
Make SourceLength a struct
ahoppen 2703c00
Make RawSyntaxData a direct enum
ahoppen f511ccc
Don't set the process terminationHandler in SwiftcInvocation
ahoppen e58d288
Update tests after cherrypicks.
allevato 5d9650b
Allow *ListSyntax nodes to be visited.
allevato bd3484b
Apply review fixes
allevato File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add accessors for source locations and test diagnostic emission (#16141)
* [SwiftSyntax] Add accessors for source locations and test diagnostic emission * Add tests for endLocation * Pre-emptively copy AbsolutePosition to avoid mutating it twice
- Loading branch information
commit 60e37d0cc5442aa9b8644b7cdcf484d9075b0e04
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has since been removed on
master
in favour oftrailingTriviaLength
so I don't think we should add it back here. Should be part of fc313fcThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.