[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