JonasToth added a comment. Could it happen that some template specializations or so need to land in `absl`?
================ Comment at: clang-tidy/abseil/NoNamespaceCheck.cpp:28 +void NoNamespaceCheck::check(const MatchFinder::MatchResult &Result) { + const auto *decl = Result.Nodes.getNodeAs<NamespaceDecl>("absl_namespace"); + ---------------- Please follow LLVM naming convention (Capitalize, and Decl should crash with a type) Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D50580 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits