Skip to content
This repository has been archived by the owner on Jul 7, 2022. It is now read-only.

Support inlay hints for clangd as well #32

Open
tom-anders opened this issue Dec 7, 2021 · 6 comments
Open

Support inlay hints for clangd as well #32

tom-anders opened this issue Dec 7, 2021 · 6 comments

Comments

@tom-anders
Copy link

Clangd-14 also has support for inlay hints (currently needs to be enabled via the --inlay-hints flag) via the 'clangd/inlayHints' request. Would be cool if this was supported as well. Probably most of the rust code could be reused.

@jan-xyz
Copy link

jan-xyz commented Apr 17, 2022

Is clangd planning to go to the stable API? It might come natively to Neovim see neovim/neovim#18086

@Decodetalkers
Copy link

Decodetalkers commented Apr 17, 2022

Is clangd planning to go to the stable API? It might come natively to Neovim see neovim/neovim#18086

Clangd14 is using the stable api. And clangd13 is using the experiment api.

@jan-xyz
Copy link

jan-xyz commented Apr 17, 2022

Are you sure? @tom-anders mentioned it uses clangd/inlayHints, which is a custom extension.

Edit: I am pretty sure it's not using the stable API released with 3.17 of the LSP spec.

@Decodetalkers
Copy link

Are you sure? @tom-anders mentioned it uses clangd/inlayHints, which is a custom extension.

Edit: I am pretty sure it's not using the stable API released with 3.17 of the LSP spec.

I see ,but the api is used since version 13 on Linux.. And now vscode has originally support the inlay of clangd14, I predicted that is use the stable api..I may need to search for more information

@Decodetalkers
Copy link

Decodetalkers commented Apr 17, 2022

Are you sure? @tom-anders mentioned it uses clangd/inlayHints, which is a custom extension.

Edit: I am pretty sure it's not using the stable API released with 3.17 of the LSP spec.

llvm/clangd-www#56

It is still clangd/inlay,I mistake for it. It seems that vscode extension check the version of clangd and show the inlay hints..Not because it uses the stable api.

@tom-anders
Copy link
Author

clangd now supports the stable API: llvm/llvm-project@3137ca8

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants