[PATCH] D20326: [clang-tidy] Fix a template function false positive in misc-unused-using-decls check.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Tue May 17 11:52:28 PDT 2016


alexfh requested changes to this revision.
This revision now requires changes to proceed.

================
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:24
@@ +23,3 @@
+// FIXME: Add a node matcher for UnresolvedLookupExpr in ASTMatcher.
+AST_MATCHER(CallExpr, hasUnresolvedLookupExpr) {
+  return isa<UnresolvedLookupExpr>(Node.getCallee());
----------------
Sorry for being unclear. Please change this to the node matcher in this patch. You can move the matcher to the ASTMatchers.h as a follow-up. `hasUnresolvedLookupExpr` is not in line with the matchers design: the `CallExpr` node has no property `UnresolvedLookupExpr`. You'll also be able to bind the `UnresolvedLookupExpr` to an identifier and make the code in `check()` slightly shorter.


http://reviews.llvm.org/D20326





More information about the cfe-commits mailing list