diff --git a/clang-tools-extra/clang-tidy/cppcoreguidelines/MisleadingCaptureDefaultByValueCheck.cpp b/clang-tools-extra/clang-tidy/cppcoreguidelines/MisleadingCaptureDefaultByValueCheck.cpp index 87e1b05802763..00dfa17a1ccf6 100644 --- a/clang-tools-extra/clang-tidy/cppcoreguidelines/MisleadingCaptureDefaultByValueCheck.cpp +++ b/clang-tools-extra/clang-tidy/cppcoreguidelines/MisleadingCaptureDefaultByValueCheck.cpp @@ -37,9 +37,8 @@ static SourceLocation findDefaultCaptureEnd(const LambdaExpr *Lambda, SourceLocation AddressofLoc = utils::lexer::findPreviousTokenKind( Capture.getLocation(), SourceMgr, Context.getLangOpts(), tok::amp); return AddressofLoc; - } else { - return Capture.getLocation(); } + return Capture.getLocation(); } } return Lambda->getIntroducerRange().getEnd(); diff --git a/clang-tools-extra/clang-tidy/cppcoreguidelines/RvalueReferenceParamNotMovedCheck.cpp b/clang-tools-extra/clang-tidy/cppcoreguidelines/RvalueReferenceParamNotMovedCheck.cpp index da49d5610e229..efcaffb45d9ad 100644 --- a/clang-tools-extra/clang-tidy/cppcoreguidelines/RvalueReferenceParamNotMovedCheck.cpp +++ b/clang-tools-extra/clang-tidy/cppcoreguidelines/RvalueReferenceParamNotMovedCheck.cpp @@ -31,9 +31,8 @@ AST_MATCHER_P2(Stmt, argumentOf, bool, AllowPartialMove, StatementMatcher, Ref) { if (AllowPartialMove) { return stmt(anyOf(Ref, hasDescendant(Ref))).matches(Node, Finder, Builder); - } else { - return Ref.matches(Node, Finder, Builder); } + return Ref.matches(Node, Finder, Builder); } } // namespace diff --git a/clang-tools-extra/clang-tidy/google/AvoidCStyleCastsCheck.cpp b/clang-tools-extra/clang-tidy/google/AvoidCStyleCastsCheck.cpp index b7dcaf20169eb..8c62fef6f3f3f 100644 --- a/clang-tools-extra/clang-tidy/google/AvoidCStyleCastsCheck.cpp +++ b/clang-tools-extra/clang-tidy/google/AvoidCStyleCastsCheck.cpp @@ -62,7 +62,8 @@ static clang::CharSourceRange getReplaceRange(const ExplicitCastExpr *Expr) { return CharSourceRange::getCharRange( CastExpr->getLParenLoc(), CastExpr->getSubExprAsWritten()->getBeginLoc()); - } else if (const auto *CastExpr = dyn_cast(Expr)) { + } + if (const auto *CastExpr = dyn_cast(Expr)) { return CharSourceRange::getCharRange(CastExpr->getBeginLoc(), CastExpr->getLParenLoc()); } else diff --git a/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp b/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp index 37aa26ded4240..bf340984bd381 100644 --- a/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp @@ -368,14 +368,14 @@ static std::optional getContainerExpr(const Expr *Call) { return ContainerCall{TheCall->getImplicitObjectArgument(), Member->getMemberDecl()->getName(), Member->isArrow(), CallKind}; - } else { - if (TheCall->getDirectCallee() == nullptr || - !MemberNames.contains(TheCall->getDirectCallee()->getName())) - return std::nullopt; - return ContainerCall{TheCall->getArg(0), - TheCall->getDirectCallee()->getName(), false, - CallKind}; } + if (TheCall->getDirectCallee() == nullptr || + !MemberNames.contains(TheCall->getDirectCallee()->getName())) + return std::nullopt; + return ContainerCall{TheCall->getArg(0), + TheCall->getDirectCallee()->getName(), false, + CallKind}; + } else if (const auto *TheCall = dyn_cast_or_null(Dug)) { if (TheCall->getNumArgs() != 1) return std::nullopt; diff --git a/clang-tools-extra/clang-tidy/modernize/UseConstraintsCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseConstraintsCheck.cpp index 75ebfbd381ad8..310b5bf9affa1 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseConstraintsCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseConstraintsCheck.cpp @@ -168,8 +168,9 @@ matchTrailingTemplateParam(const FunctionTemplateDecl *FunctionTemplate) { return {matchEnableIfSpecialization( LastTemplateParam->getTypeSourceInfo()->getTypeLoc()), LastTemplateParam}; - } else if (const auto *LastTemplateParam = - dyn_cast(LastParam)) { + } + if (const auto *LastTemplateParam = + dyn_cast(LastParam)) { if (LastTemplateParam->hasDefaultArgument() && LastTemplateParam->getIdentifier() == nullptr) { return {matchEnableIfSpecialization( diff --git a/clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp b/clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp index 413a37f4af9fa..aa54cf284f627 100644 --- a/clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp +++ b/clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp @@ -73,14 +73,12 @@ bool IsRValueReferenceParam(const Expr *Invocation, Arg->isLValue()) { if (!Invocation->getType()->isRecordType()) return true; - else { - if (const auto *ConstructCallExpr = - dyn_cast(Invocation)) { - if (const auto *ConstructorDecl = ConstructCallExpr->getConstructor()) { - if (!ConstructorDecl->isCopyOrMoveConstructor() && - !ConstructorDecl->isDefaultConstructor()) - return true; - } + if (const auto *ConstructCallExpr = + dyn_cast(Invocation)) { + if (const auto *ConstructorDecl = ConstructCallExpr->getConstructor()) { + if (!ConstructorDecl->isCopyOrMoveConstructor() && + !ConstructorDecl->isDefaultConstructor()) + return true; } } }