[PATCH] D54878: [clangd] NFC: Eliminate the unused variable warning.

Henry Wong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 25 19:10:23 PST 2018


MTC marked an inline comment as done.
MTC added inline comments.


================
Comment at: clangd/AST.cpp:98
   // The name was empty, so present an anonymous entity.
-  if (auto *NS = llvm::dyn_cast<NamespaceDecl>(&ND))
+  if (isa<NamespaceDecl>(&ND))
     return "(anonymous namespace)";
----------------
MaskRay wrote:
> `isa<NamespaceDecl>(ND)` also works.
Thanks for the tips! I will update it latter.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D54878





More information about the cfe-commits mailing list