Author: Baranov Victor Date: 2026-08-04T16:37:10+03:00 New Revision: 7e15ae5cef9aef89b7a09a7c6f203f4a8ca6c77d
URL: https://github.com/llvm/llvm-project/commit/7e15ae5cef9aef89b7a09a7c6f203f4a8ca6c77d DIFF: https://github.com/llvm/llvm-project/commit/7e15ae5cef9aef89b7a09a7c6f203f4a8ca6c77d.diff LOG: [clang-tidy][NFC] Apply readability-redundant-parentheses (#213848) Added: Modified: clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp clang-tools-extra/clang-tidy/bugprone/BranchCloneCheck.cpp clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp clang-tools-extra/clang-tidy/cppcoreguidelines/NoMallocCheck.cpp clang-tools-extra/clang-tidy/misc/StaticInitializationCycleCheck.cpp clang-tools-extra/clang-tidy/modernize/AvoidBindCheck.cpp clang-tools-extra/clang-tidy/modernize/MinMaxUseInitializerListCheck.cpp clang-tools-extra/clang-tidy/modernize/UseStartsEndsWithCheck.cpp clang-tools-extra/clang-tidy/readability/ContainerSizeEmptyCheck.cpp clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp clang-tools-extra/clang-tidy/utils/Matchers.h Removed: ################################################################################ diff --git a/clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp b/clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp index d33b6677b86b1..e548a0d56a920 100644 --- a/clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp +++ b/clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp @@ -92,7 +92,7 @@ IdDependentBackwardBranchCheck::hasIdDepVar(const Expr *Expression) { const auto FoundVariable = IdDepVarsMap.find(CheckVariable); if (FoundVariable == IdDepVarsMap.end()) return nullptr; - return &(FoundVariable->second); + return &FoundVariable->second; } for (const auto *Child : Expression->children()) if (const auto *ChildExpression = dyn_cast_if_present<Expr>(Child)) @@ -114,7 +114,7 @@ IdDependentBackwardBranchCheck::hasIdDepField(const Expr *Expression) { const auto FoundField = IdDepFieldsMap.find(CheckField); if (FoundField == IdDepFieldsMap.end()) return nullptr; - return &(FoundField->second); + return &FoundField->second; } for (const auto *Child : Expression->children()) if (const auto *ChildExpression = dyn_cast_if_present<Expr>(Child)) diff --git a/clang-tools-extra/clang-tidy/bugprone/BranchCloneCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/BranchCloneCheck.cpp index ad6b12c1d7d2f..397887f8fe8e0 100644 --- a/clang-tools-extra/clang-tidy/bugprone/BranchCloneCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/BranchCloneCheck.cpp @@ -125,7 +125,7 @@ static bool isIdenticalStmt(const ASTContext &Ctx, const Stmt *Stmt1, return false; // If either expression comes from a macro then don't warn even if // the expressions are identical. - if ((Expr1->getExprLoc().isMacroID()) || (Expr2->getExprLoc().isMacroID())) + if (Expr1->getExprLoc().isMacroID() || Expr2->getExprLoc().isMacroID()) return false; // If all children of two expressions are identical, return true. diff --git a/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp index eab0fe2babf53..c16e623c25eb4 100644 --- a/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/NondeterministicPointerIterationOrderCheck.cpp @@ -48,7 +48,7 @@ void NondeterministicPointerIterationOrderCheck::check( const auto *ForRangePointers = Result.Nodes.getNodeAs<CXXForRangeStmt>("cxxForRangeStmt"); - if (ForRangePointers && !(ForRangePointers->getBeginLoc().isMacroID())) { + if (ForRangePointers && !ForRangePointers->getBeginLoc().isMacroID()) { const auto *RangeInit = Result.Nodes.getNodeAs<Stmt>("rangeinit"); if (const auto *ClassTemplate = Result.Nodes.getNodeAs<ClassTemplateSpecializationDecl>( @@ -67,7 +67,7 @@ void NondeterministicPointerIterationOrderCheck::check( } const auto *SortPointers = Result.Nodes.getNodeAs<Stmt>("sortsemantic"); - if (SortPointers && !(SortPointers->getBeginLoc().isMacroID())) { + if (SortPointers && !SortPointers->getBeginLoc().isMacroID()) { const SourceRange R = SortPointers->getSourceRange(); diag(R.getBegin(), "sorting pointers is nondeterministic") << R; } diff --git a/clang-tools-extra/clang-tidy/cppcoreguidelines/NoMallocCheck.cpp b/clang-tools-extra/clang-tidy/cppcoreguidelines/NoMallocCheck.cpp index 22cd1e4e29a68..8bf1e86db0d77 100644 --- a/clang-tools-extra/clang-tidy/cppcoreguidelines/NoMallocCheck.cpp +++ b/clang-tools-extra/clang-tidy/cppcoreguidelines/NoMallocCheck.cpp @@ -31,14 +31,14 @@ void NoMallocCheck::registerMatchers(MatchFinder *Finder) { // Registering realloc calls, suggest std::vector or std::string. Finder->addMatcher( callExpr(callee(functionDecl( - hasAnyName(utils::options::parseStringList((ReallocList)))))) + hasAnyName(utils::options::parseStringList(ReallocList))))) .bind("realloc"), this); // Registering free calls, will suggest RAII instead. Finder->addMatcher( callExpr(callee(functionDecl( - hasAnyName(utils::options::parseStringList((DeallocList)))))) + hasAnyName(utils::options::parseStringList(DeallocList))))) .bind("free"), this); } diff --git a/clang-tools-extra/clang-tidy/misc/StaticInitializationCycleCheck.cpp b/clang-tools-extra/clang-tidy/misc/StaticInitializationCycleCheck.cpp index a5b45ad935934..790b76b784ea4 100644 --- a/clang-tools-extra/clang-tidy/misc/StaticInitializationCycleCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/StaticInitializationCycleCheck.cpp @@ -362,7 +362,7 @@ reportCycles(ArrayRef<const VarUseNode *> SCC, CycleOs << *N->getDecl() << " -> "; } - CycleOs << *(FoundPath.front()->getDecl()); + CycleOs << *FoundPath.front()->getDecl(); Chk.diag((*VarNode)->getDecl()->getLocation(), "possible cyclical initialization: %0", DiagnosticIDs::Note) diff --git a/clang-tools-extra/clang-tidy/modernize/AvoidBindCheck.cpp b/clang-tools-extra/clang-tidy/modernize/AvoidBindCheck.cpp index c9f63c3b5e2ce..551ae8b1110bc 100644 --- a/clang-tools-extra/clang-tidy/modernize/AvoidBindCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/AvoidBindCheck.cpp @@ -514,7 +514,7 @@ getCallableMaterialization(const MatchFinder::MatchResult &Result) { const auto *CE = dyn_cast<CXXConstructExpr>(NoTemporaries); const auto *FC = dyn_cast<CXXFunctionalCastExpr>(NoTemporaries); - if ((isa<CallExpr>(NoTemporaries)) || (CE && (CE->getNumArgs() > 0)) || + if (isa<CallExpr>(NoTemporaries) || (CE && (CE->getNumArgs() > 0)) || (FC && (FC->getCastKind() == CK_ConstructorConversion))) // CE is something that looks like a call, with arguments - either // a function call or a constructor invocation. diff --git a/clang-tools-extra/clang-tidy/modernize/MinMaxUseInitializerListCheck.cpp b/clang-tools-extra/clang-tidy/modernize/MinMaxUseInitializerListCheck.cpp index acf1050732779..55f0886d12dee 100644 --- a/clang-tools-extra/clang-tidy/modernize/MinMaxUseInitializerListCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/MinMaxUseInitializerListCheck.cpp @@ -61,7 +61,7 @@ static FindArgsResult findArgs(const CallExpr *Call) { Result.Args = SmallVector<const Expr *>(Call->arguments()); } else { // if it has 3 arguments then the last will be the comparison - Result.Compare = *(std::next(Call->arguments().begin(), 2)); + Result.Compare = *std::next(Call->arguments().begin(), 2); Result.Args = SmallVector<const Expr *>(llvm::drop_end(Call->arguments())); } Result.First = Result.Args.front(); diff --git a/clang-tools-extra/clang-tidy/modernize/UseStartsEndsWithCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseStartsEndsWithCheck.cpp index 159773be855fb..d8ab0fedfc112 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseStartsEndsWithCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseStartsEndsWithCheck.cpp @@ -89,7 +89,7 @@ struct NotLengthExprForStringNode { AST_MATCHER_P(Expr, lengthExprForStringNode, std::string, ID) { return Builder->removeBindings(NotLengthExprForStringNode( - ID, DynTypedNode::create(Node), &(Finder->getASTContext()))); + ID, DynTypedNode::create(Node), &Finder->getASTContext())); } } // namespace diff --git a/clang-tools-extra/clang-tidy/readability/ContainerSizeEmptyCheck.cpp b/clang-tools-extra/clang-tidy/readability/ContainerSizeEmptyCheck.cpp index 5dda3f3297d2f..c5fbd3022756d 100644 --- a/clang-tools-extra/clang-tidy/readability/ContainerSizeEmptyCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/ContainerSizeEmptyCheck.cpp @@ -38,7 +38,7 @@ AST_POLYMORPHIC_MATCHER_P2(hasAnyArgumentWithParam, int ParamIndex = 0; for (; ArgIndex < Node.getNumArgs(); ++ArgIndex) { ast_matchers::internal::BoundNodesTreeBuilder ArgMatches(*Builder); - if (ArgMatcher.matches(*(Node.getArg(ArgIndex)->IgnoreParenCasts()), Finder, + if (ArgMatcher.matches(*Node.getArg(ArgIndex)->IgnoreParenCasts(), Finder, &ArgMatches)) { ast_matchers::internal::BoundNodesTreeBuilder ParamMatches(ArgMatches); if (expr(anyOf(cxxConstructExpr(hasDeclaration(cxxConstructorDecl( diff --git a/clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp b/clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp index 8d9e68c8147f8..28410f23a812e 100644 --- a/clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp @@ -216,7 +216,7 @@ void UppercaseLiteralSuffixCheck::check( "'%1', which is not uppercase") << IsInteger << Details->OldSuffix; if (Details->FixIt) // Similarly, a fix-it is not always possible. - Complaint << *(Details->FixIt); + Complaint << *Details->FixIt; } } diff --git a/clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp b/clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp index 08deb6a35e55b..13781ada37557 100644 --- a/clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp +++ b/clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp @@ -443,7 +443,7 @@ ExceptionAnalyzer::ExceptionInfo::filterIgnoredExceptions( if (TD->getDeclName().isIdentifier()) { if ((IgnoreBadAlloc && (TD->getName() == "bad_alloc" && TD->isInStdNamespace())) || - (IgnoredTypes.contains(TD->getName()))) + IgnoredTypes.contains(TD->getName())) TypesToDelete.push_back(T); } } diff --git a/clang-tools-extra/clang-tidy/utils/Matchers.h b/clang-tools-extra/clang-tidy/utils/Matchers.h index fea3ef041df1c..e04da47322623 100644 --- a/clang-tools-extra/clang-tidy/utils/Matchers.h +++ b/clang-tools-extra/clang-tidy/utils/Matchers.h @@ -162,7 +162,7 @@ struct NotIdenticalStatementsPredicate { // to ID node. AST_MATCHER_P(Stmt, isStatementIdenticalToBoundNode, std::string, ID) { const NotIdenticalStatementsPredicate Predicate{ - ID, ::clang::DynTypedNode::create(Node), &(Finder->getASTContext())}; + ID, ::clang::DynTypedNode::create(Node), &Finder->getASTContext()}; return Builder->removeBindings(Predicate); } _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
