-
Notifications
You must be signed in to change notification settings - Fork 430
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
feat(semantic_tokens): link lsp comments, e.g. conditional C ifdefs that won't evaluate #334
Conversation
reorganized lsp tokens list by alphabetical order |
That (comment) should be a default link in Neovim. (There may also be some modifiers that could usefully linked by default.) |
I'm not 100% convinced about this change, but it's fine by me. |
…hat won't evaluate
It's nice for C/C++ macros to have inactive sections highlighted as comments, and it's quite common in most IDEs Updated to link to |
awesome, thanks! |
Just FYI: The reason that Notably, clangd only uses the |
Thanks for the headsup, and I'm sorry about that! I don't use treesitter's comments so I hadn't noticed that. I would totally be open to changing this only for c/cpp files, I do think such a case should be somewhat separate from regular comments anyways which I'm glad clangd does by not producing comment tokens for actual comments. |
…hat won't evaluate (folke#334)
Before:
After:
Would be very useful for C/C++ devs