[PATCH] D61601: [clangd] Represent Hover result using FormattedString

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet added a comment.

> I'm happy to restructure the `HoverInfo` to your liking if the current one seems problematic for any reason.

It will just cause more code move on my side, no problems apart from that



================
Comment at: clang-tools-extra/clangd/Protocol.cpp:707
+  else
+    return false;
+  return true;
----------------
kadircet wrote:
> Maybe also vlog/elog the unknown kind? So that we can easily catch new additions to spec.
I was referring to the `else` case down below :D


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61601/new/

https://reviews.llvm.org/D61601