From bfee5f73529d1ba48746f1a059fa81e565541a23 Mon Sep 17 00:00:00 2001 From: Alexander Kornienko Date: Fri, 29 Jan 2016 15:22:20 +0000 Subject: [PATCH] [clang-tidy] Fix minor style issues. NFC llvm-svn: 259198 --- clang-tools-extra/clang-tidy/misc/VirtualNearMissCheck.cpp | 3 +-- .../clang-tidy/performance/ImplicitCastInLoopCheck.cpp | 14 +++++++------- 2 files changed, 8 insertions(+), 9 deletions(-) diff --git a/clang-tools-extra/clang-tidy/misc/VirtualNearMissCheck.cpp b/clang-tools-extra/clang-tidy/misc/VirtualNearMissCheck.cpp index f573bb7..24aea8c2 100644 --- a/clang-tools-extra/clang-tidy/misc/VirtualNearMissCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/VirtualNearMissCheck.cpp @@ -168,9 +168,8 @@ static bool checkOverrideByDerivedMethod(const CXXMethodDecl *BaseMD, E = DerivedMD->end_overridden_methods(); I != E; ++I) { const CXXMethodDecl *OverriddenMD = *I; - if (BaseMD->getCanonicalDecl() == OverriddenMD->getCanonicalDecl()) { + if (BaseMD->getCanonicalDecl() == OverriddenMD->getCanonicalDecl()) return true; - } } return false; diff --git a/clang-tools-extra/clang-tidy/performance/ImplicitCastInLoopCheck.cpp b/clang-tools-extra/clang-tidy/performance/ImplicitCastInLoopCheck.cpp index c20598e..5f3d2b2 100644 --- a/clang-tools-extra/clang-tidy/performance/ImplicitCastInLoopCheck.cpp +++ b/clang-tools-extra/clang-tidy/performance/ImplicitCastInLoopCheck.cpp @@ -27,8 +27,8 @@ namespace { // The subtelty is that in some cases (user defined conversions), we can // get to ImplicitCastExpr inside each other, with the outer one a NoOp. In this // case we skip the first cast expr. -bool IsNonTrivialImplicitCast(const Stmt* ST) { - if (const auto* ICE = dyn_cast(ST)) { +bool IsNonTrivialImplicitCast(const Stmt *ST) { + if (const auto *ICE = dyn_cast(ST)) { return (ICE->getCastKind() != CK_NoOp) || IsNonTrivialImplicitCast(ICE->getSubExpr()); } @@ -57,15 +57,15 @@ void ImplicitCastInLoopCheck::registerMatchers(MatchFinder *Finder) { } void ImplicitCastInLoopCheck::check(const MatchFinder::MatchResult &Result) { - const auto* VD = Result.Nodes.getNodeAs("faulty-var"); - const auto* Init = Result.Nodes.getNodeAs("init"); - const auto* OperatorCall = + const auto *VD = Result.Nodes.getNodeAs("faulty-var"); + const auto *Init = Result.Nodes.getNodeAs("init"); + const auto *OperatorCall = Result.Nodes.getNodeAs("operator-call"); - if (const auto* Cleanup = dyn_cast(Init)) + if (const auto *Cleanup = dyn_cast(Init)) Init = Cleanup->getSubExpr(); - const auto* Materialized = dyn_cast(Init); + const auto *Materialized = dyn_cast(Init); if (!Materialized) return; -- 2.7.4