[clang-tools-extra] [clang-tidy] Update llvmlibc-implementation-in-namespace to new rules (PR #66504)
Piotr Zegar via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 18 08:27:16 PDT 2023
================
@@ -30,15 +32,18 @@ void ImplementationInNamespaceCheck::check(
return;
if (const auto *NS = dyn_cast<NamespaceDecl>(MatchedDecl)) {
- if (NS->getName() != RequiredNamespace) {
- diag(NS->getLocation(), "'%0' needs to be the outermost namespace")
- << RequiredNamespace;
- }
+ if (!Result.SourceManager->isMacroBodyExpansion(NS->getLocation()))
+ diag(NS->getLocation(),
+ "the outermost namespace should be the '%0' macro")
+ << RequiredNamespaceMacroName;
+ else if (!NS->getName().starts_with(RequiredNamespaceStart))
----------------
PiotrZSL wrote:
getName will assert if Namespace got not identifier. What about anonymous namespaces ? Or declarations without name ?
https://github.com/llvm/llvm-project/pull/66504
More information about the cfe-commits
mailing list