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
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits