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

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

malaperle requested changes to this revision.
malaperle added inline comments.
This revision now requires changes to proceed.


================
Comment at: clangd/Protocol.cpp:498
+    // Default Hover values
+    Hover H;
+    return json::obj{
----------------
remove, we have to return the contents of the H that was passed as parameter, not a new one. I hit this bug while testing with no range (hover text in another file)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D35894