Skip to content

[swiftSyntax] Incremental syntax classification #18424

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 2 commits into from
Aug 3, 2018

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Jul 31, 2018

Currently, we are able to incrementally transfer the syntax tree but to receive information for syntax highlighting the SyntaxClassifier always needed to walk the entire tree. With this PR we are storing the IDs of the nodes that got reused as part of the last incremental transfer and are able to just reclassify the parts of the syntax tree that have changed since the last transfer.

ahoppen added 2 commits July 31, 2018 15:41
…ental transfer

This way we will be able to avoid reclassifying these nodes for syntax
highlighting since we know they haven't changed.
@ahoppen ahoppen requested review from rintaro and nkcsgexi July 31, 2018 23:10
@ahoppen
Copy link
Member Author

ahoppen commented Jul 31, 2018

@swift-ci Please smoke test

@ahoppen ahoppen merged commit 285b337 into swiftlang:master Aug 3, 2018
@ahoppen ahoppen deleted the incremental-classification branch August 22, 2018 15:48
maldahleh pushed a commit to maldahleh/swift that referenced this pull request Oct 26, 2020
…ation

[swiftSyntax] Incremental syntax classification
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.

3 participants