Skip to content

Commit 5f7c21e

Browse files
committed
usings and whitespace
1 parent 8ea2ea1 commit 5f7c21e

File tree

1 file changed

+31
-5
lines changed

1 file changed

+31
-5
lines changed

clang-tools-extra/clang-tidy/bugprone/MoveSharedPointerContentsCheck.cpp

Lines changed: 31 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@
1212
#include "../utils/OptionsUtils.h"
1313
#include "clang/ASTMatchers/ASTMatchFinder.h"
1414

15-
using namespace clang::ast_matchers; // NOLINT
16-
1715
namespace clang::tidy::bugprone {
1816
namespace {
1917

@@ -48,7 +46,36 @@ MoveSharedPointerContentsCheck::MoveSharedPointerContentsCheck(
4846
SharedPointerClasses(utils::options::parseStringList(Options.get(
4947
"SharedPointerClasses", "::std::shared_ptr;::boost::shared_ptr"))) {}
5048

51-
void MoveSharedPointerContentsCheck::registerMatchers(MatchFinder *Finder) {
49+
void MoveSharedPointerContentsCheck::registerMatchers(
50+
ast_matchers::MatchFinder *Finder) {
51+
using ast_matchers::anyOf;
52+
using ast_matchers::callee;
53+
using ast_matchers::callExpr;
54+
using ast_matchers::cxxDependentScopeMemberExpr;
55+
using ast_matchers::cxxMemberCallExpr;
56+
using ast_matchers::cxxMethodDecl;
57+
using ast_matchers::cxxOperatorCallExpr;
58+
using ast_matchers::declRefExpr;
59+
using ast_matchers::functionDecl;
60+
using ast_matchers::hasAnyDeclaration;
61+
using ast_matchers::hasAnyName;
62+
using ast_matchers::hasArgument;
63+
using ast_matchers::hasDescendant;
64+
using ast_matchers::hasMemberName;
65+
using ast_matchers::hasName;
66+
using ast_matchers::hasOperatorName;
67+
using ast_matchers::hasOverloadedOperatorName;
68+
using ast_matchers::hasType;
69+
using ast_matchers::hasUnaryOperand;
70+
using ast_matchers::hasUnderlyingDecl;
71+
using ast_matchers::namedDecl;
72+
using ast_matchers::on;
73+
using ast_matchers::qualType;
74+
using ast_matchers::to;
75+
using ast_matchers::unaryOperator;
76+
using ast_matchers::unresolvedLookupExpr;
77+
using ast_matchers::varDecl;
78+
5279
auto IsStdMove =
5380
callee(functionDecl(hasAnyName("::std::move", "::std::forward")));
5481

@@ -68,7 +95,6 @@ void MoveSharedPointerContentsCheck::registerMatchers(MatchFinder *Finder) {
6895
callExpr(IsStdMove,
6996
hasArgument(
7097
0, unaryOperator(hasOperatorName("*"),
71-
7298
hasUnaryOperand(cxxMemberCallExpr(
7399
callee(cxxMethodDecl(hasName("get"))),
74100
on(hasType(qualType(isSharedPointer(
@@ -112,7 +138,7 @@ void MoveSharedPointerContentsCheck::registerMatchers(MatchFinder *Finder) {
112138
}
113139

114140
void MoveSharedPointerContentsCheck::check(
115-
const MatchFinder::MatchResult &Result) {
141+
const ast_matchers::MatchFinder::MatchResult &Result) {
116142
const CallExpr *Call = nullptr;
117143
for (const llvm::StringRef Binding :
118144
{"unresolved_call", "unresolved_get_call", "get_call", "call"}) {

0 commit comments

Comments
 (0)