[PATCH] D52136: [clang-tidy] Add modernize-concat-nested-namespaces check

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 15 07:38:23 PDT 2018


lebedev.ri added inline comments.


================
Comment at: clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp:73-74
+void ConcatNestedNamespacesCheck::registerMatchers(MatchFinder *Finder) {
+  if (getLangOpts().CPlusPlus)
+    Finder->addMatcher(namespaceDecl().bind("namespace"), this);
+}
----------------
It should only get registered in C++17 or newer mode.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52136





More information about the cfe-commits mailing list