[PATCH] D35894: [clangd] Code hover for Clangd

[Marc-Andre Laperle via Phabricator via cfe-commits]

malaperle added a comment.

In https://reviews.llvm.org/D35894#903594, @ilya-biryukov wrote:

> In https://reviews.llvm.org/D35894#903552, @malaperle wrote:
>
> > I think he meant to have multiple sections in the hover, one C/C++ and one not. But we noticed you can have an array of MarkedString in Hover so it should be fine.
>
>
> I totally misunderstood the question. Good to know it all works without changing the LSP.
>  Just to make sure I got it right this time: the use-case is to, e.g., return both a code snippet (having `"language": cpp`) and a documentation string (without `language`) as a result of hover request?


Exactly! Although the without-language part, maybe it'll be "scope" information first. Maybe documentation/comments in a second patch.


https://reviews.llvm.org/D35894