Skip to content
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

syntax-highlighter: load all tree-sitter highlight configurations at startup #50394

Merged
merged 1 commit into from
Apr 5, 2023

Conversation

Strum355
Copy link
Contributor

@Strum355 Strum355 commented Apr 5, 2023

See comment in main.rs for more details

Test plan

I LOOKED at the flamegraphs (x-axis is NOT time)

new:
image

old:
image

@Strum355 Strum355 added the team/graph Graph Team (previously Code Intel/Language Tools/Language Platform) label Apr 5, 2023
@Strum355 Strum355 self-assigned this Apr 5, 2023
@cla-bot cla-bot bot added the cla-signed label Apr 5, 2023
@Strum355 Strum355 changed the title syntax-highlighter: load all tree-sitter highlight configurations at … syntax-highlighter: load all tree-sitter highlight configurations at startup Apr 5, 2023
@Strum355 Strum355 merged commit 8d58dba into main Apr 5, 2023
@Strum355 Strum355 deleted the nsc/syntax-highlighter-loadinit branch April 5, 2023 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed team/graph Graph Team (previously Code Intel/Language Tools/Language Platform)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants