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

Marc-Andre Laperle via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 23 08:15:37 PDT 2017

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.


More information about the cfe-commits mailing list