[PATCH] D50542: [clang-tidy] Add abseil-no-internal-deps check
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 10 02:14:56 PDT 2018
JonasToth added inline comments.
================
Comment at: clang-tidy/abseil/NoInternalDepsCheck.cpp:24
+
+ Finder->addMatcher(
+ nestedNameSpecifierLoc(loc(specifiesNamespace(namespaceDecl(
----------------
Actually that one is generally useful. Accessing the `foo::internal` from outside of `foo` is always a problem. Maybe this matcher can become configurable or just match on any `internal` access from outside the enclosing namespace.
================
Comment at: clang-tidy/abseil/NoInternalDepsCheck.h:21
+/// against doing so.
+/// Should not run on internal Abseil files or Abseil source code.
+///
----------------
Please make this a full sentence, like `This check should not be run on internal ...` (if that is grammatically correct)
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D50542
More information about the cfe-commits
mailing list