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

documentation is bad comparing to ccls #645

Closed
GopherJ opened this issue Jan 14, 2021 · 7 comments
Closed

documentation is bad comparing to ccls #645

GopherJ opened this issue Jan 14, 2021 · 7 comments

Comments

@GopherJ
Copy link

GopherJ commented Jan 14, 2021

Please describe the problem.
For hints on what information is helpful, see: https://clangd.llvm.org/troubleshooting.html

If you can, provide a minimal chunk of code that shows the problem (either inline, or attach it if larger).

Logs
Please attach the clangd stderr log if you can. (Usually available from the editor)
If possible, run with --log=verbose - note that the logs will include the contents of open files!
If this is a crash, try to put llvm-symbolizer on your PATH per the troubleshooting instructions.

System information
Output of clangd --version: clangd version 9.0.0-2~ubuntu18.04.2 (tags/RELEASE_900/final)
Editor/LSP plugin: coc.nvim, neovim
Operating system: LInux Mint 19.3

image

@njames93
Copy link

Care to elaborate. Any specific places you feel are lacking documentation. The website does a pretty decent job as it is right now.

@HighCommander4
Copy link

I think the reporter is referring to the way a function's Doxygen documentation is rendered in the posted screenshot (which I assume is of a hover window).

@GopherJ
Copy link
Author

GopherJ commented Jan 14, 2021

@HighCommander4 @njames93 Yes I mean the function documentation, I'm not sure it's clangd's error or coc.nvim's error

See also: clangd/coc-clangd#48

@HighCommander4
Copy link

Dupe of #529?

@kadircet
Copy link
Member

This is a mixture of coc.nvim's markdown mocking and clangd's lack of doxygen comment parsing.

as mentioned in clangd/coc-clangd#48 (comment), we've changed our escaping to be less aggressive with recent versions of clangd. you can try with clangd-11 to have better results, which is available at https://releases.llvm.org/download.html#11.0.0.

though the doxygen comment processing is still lacking as @HighCommander4 pointed out.

@GopherJ
Copy link
Author

GopherJ commented Jan 15, 2021

@kadircet Hi, I justed tested and it's much more, it's not perfect yet but thanks for improving it.

image

I think the documentation is a little bit too wide, but this is maybe the json library's issue. The @param is not highlighted yet, I'll try some other colorscheme to see if it improves

@GopherJ
Copy link
Author

GopherJ commented Jan 15, 2021

@kadircet @HighCommander4 Let's only open #529 thanks for your replies

@GopherJ GopherJ closed this as completed Jan 15, 2021
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

No branches or pull requests

4 participants