[PATCH] D85272: [clangd] Semantic highlighting for dependent template name in template argument
Nathan Ridge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 5 08:11:23 PDT 2020
nridge added inline comments.
================
Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:305
+ break;
+ default:;
+ }
----------------
hokein wrote:
> nit: move the trailing `;` to a new line.
I have done this, but clang-format moves it back... do we really want to fight wit it?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85272/new/
https://reviews.llvm.org/D85272
More information about the cfe-commits
mailing list