-
Notifications
You must be signed in to change notification settings - Fork 213
Remove SyntaxTrivia from the compiler #11702
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
DustinCampbell
merged 30 commits into
dotnet:main
from
DustinCampbell:remove-dead-syntax
Apr 9, 2025
Merged
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
c2be582
SyntaxRewriter: Remove VisitToken implementation
DustinCampbell 6237c2b
SyntaxToken: Remove WithLeadingTrivia and WithTrailingTrivia
DustinCampbell 7ff4145
SyntaxWalker: Remove VisitLeadingTrivia and VisitTrailingTrivia
DustinCampbell 39a09d0
SyntaxNode: Remove HasLeadingTrivia and HasTrailingTrivia
DustinCampbell 00f4ae5
SyntaxRewriter: Remove VisitList(SyntaxTriviaList)
DustinCampbell 3060e2d
SyntaxTriviaList: Remove Enumerator.InitializeFrom helpers
DustinCampbell ac8fdb9
SyntaxNode: Remove GetLeadingTrivia and GetTrailingTrivia
DustinCampbell ab2325f
Remove SyntaxTriviaList and SyntaxTriviaListBuilder
DustinCampbell 4371ff3
Internal.SyntaxToken: Remove uncalled constructors
DustinCampbell dffabf3
Internal.SyntaxRewriter: Remove VisitToken implementation
DustinCampbell 8bc91a1
GreenNode: Remove WithLeadingTrivia and WithTrailingTrivia
DustinCampbell 758cc7d
Internal.SyntaxToken: Remove LeadingTrivia and TrailingTrivia
DustinCampbell 34f2a80
GeenNode: Remove HasLeadingTrivia and HasTrailingTrivia
DustinCampbell 6197c1e
GreenNode: Remove GetLeadingTriviaWidth() and GetTrailingTriviaWidth()
DustinCampbell bdcc24f
GreenNode: Remove logic to write leading/trailing trivia
DustinCampbell 099c81f
Internal.SyntaxToken: Don't compare trivia in IsEquivalentTo
DustinCampbell 339803e
GreenNode: Remove GetLeadingTrivia() and GetTrailingTrivia()
DustinCampbell d17f252
Internal.SyntaxToken: Remove leading and trailing trivia fields
DustinCampbell 5432996
Remove SyntaxTrivia and InternalSyntax.SyntaxTrivia
DustinCampbell 1977b4a
GreenNode: Consolidate FullWidth and Width
DustinCampbell 74259d7
SyntaxNode: Consolidate FullWidth and Width
DustinCampbell 2186f51
SyntaxNode: Consolidate FullSpan and Span
DustinCampbell 194bae9
Remove outdated comment and simplify GreenNode.WriteTo
DustinCampbell 111ae93
Remove unused NodeFlags
DustinCampbell 48b244a
GreenNode: Remove GetValue and GetValueText
DustinCampbell 559723d
GreenNode: Remove GetFirstToken/Terminal and GetLastToken/Terminal
DustinCampbell 8faf0e6
Remove SyntaxList<T>.FullSpan
DustinCampbell 3d73b3b
GreenNode: Make ToString() to return the same text as ToFullString()
DustinCampbell 947d638
Remove GreenNode.ToFullString() and SyntaxNode.ToFullString()
DustinCampbell 684cae0
Merge branch 'main' into remove-dead-syntax
DustinCampbell 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
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
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.
Uh oh!
There was an error while loading. Please reload this page.