[PATCH] D72623: [clangd] Rearrange type, returntype and parameters in hover card

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 14 02:10:00 PST 2020


kadircet added inline comments.


================
Comment at: clang-tools-extra/clangd/Hover.cpp:567
+  } else if (Type) {
+    Output.addParagraph().appendText("Type: ").appendCode(*Type);
   }
----------------
sammccall wrote:
> consider **`∈`**, which would be short and symmetrical with the arrow for return type...
> 
> Too obscure?
it is a good way to make the output shorter, not sure about the symmetry though as type and returntype never show up in the same hovercard.
but I believe it is too obscure.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72623





More information about the cfe-commits mailing list