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

[Alexander Kornienko via cfe-commits]

alexfh added inline comments.

================
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:24
@@ +23,3 @@
+// FIXME: Move this node matcher to ASTMatcher.
+AST_MATCHER(Stmt, unresolvedLookupExpr) {
+  return isa<UnresolvedLookupExpr>(Node);
----------------
That's because we need a node matcher, not narrowing matcher. I guess, this 
should work:

  const internal::VariadicAllOfMatcher<UnresolvedLookupExpr> 
unresolvedLookupExpr;


http://reviews.llvm.org/D20326



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits