================ @@ -0,0 +1,120 @@ +//===--- IncorrectEnableSharedFromThisCheck.cpp - clang-tidy --------------===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception +// +//===----------------------------------------------------------------------===// + +#include "IncorrectEnableSharedFromThisCheck.h" +#include "clang/AST/ASTContext.h" +#include "clang/AST/DeclCXX.h" +#include "clang/AST/RecursiveASTVisitor.h" +#include "clang/ASTMatchers/ASTMatchFinder.h" +#include "clang/Basic/Specifiers.h" +#include "llvm/ADT/SmallPtrSet.h" + +using namespace clang::ast_matchers; + +namespace clang::tidy::bugprone { + +void IncorrectEnableSharedFromThisCheck::registerMatchers(MatchFinder *Finder) { + Finder->addMatcher(translationUnitDecl(), this); +} + +void IncorrectEnableSharedFromThisCheck::check( + const MatchFinder::MatchResult &Result) { + + class Visitor : public RecursiveASTVisitor<Visitor> { + IncorrectEnableSharedFromThisCheck &Check; + llvm::SmallPtrSet<const CXXRecordDecl *, 16> EnableSharedClassSet; + + public: + explicit Visitor(IncorrectEnableSharedFromThisCheck &Check) + : Check(Check) {} + + bool VisitCXXRecordDecl(CXXRecordDecl *RDecl) { + for (const auto &Base : RDecl->bases()) { + VisitCXXBaseSpecifier(Base, RDecl); + } + for (const auto &Base : RDecl->bases()) { + const CXXRecordDecl *BaseType = Base.getType()->getAsCXXRecordDecl(); + if (BaseType && BaseType->getQualifiedNameAsString() == + "std::enable_shared_from_this") { + EnableSharedClassSet.insert(RDecl->getCanonicalDecl()); + return true; + } + } + return true; + } + + bool VisitCXXBaseSpecifier(const CXXBaseSpecifier &Base, + CXXRecordDecl *RDecl) { + const CXXRecordDecl *BaseType = Base.getType()->getAsCXXRecordDecl(); + const clang::AccessSpecifier AccessSpec = Base.getAccessSpecifier(); + + if (BaseType && + BaseType->getQualifiedNameAsString() == "enable_shared_from_this") { + if (AccessSpec == clang::AS_public) { + const SourceLocation InsertLocation = Base.getBaseTypeLoc(); + const FixItHint Hint = FixItHint::CreateInsertion(InsertLocation, "std::"); ---------------- MichelleCDjunaidi wrote:
ah, I implemented this missing std:: thing because of this suggestion. I didn't know there was a generic check for it. https://github.com/llvm/llvm-project/pull/102299#discussion_r1706831159 so, do I just remove this part of the check? https://github.com/llvm/llvm-project/pull/102299 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
