https://github.com/vbvictor created https://github.com/llvm/llvm-project/pull/213848
None >From c44b168bd94d3e32f32470395c00d5dae36a7130 Mon Sep 17 00:00:00 2001 From: Victor Baranov <[email protected]> Date: Tue, 4 Aug 2026 09:26:57 +0300 Subject: [PATCH] [clang-tidy][NFC] Apply readability-redundant-parentheses --- .../clang-tidy/altera/IdDependentBackwardBranchCheck.cpp | 4 ++-- clang-tools-extra/clang-tidy/bugprone/BranchCloneCheck.cpp | 2 +- .../bugprone/NondeterministicPointerIterationOrderCheck.cpp | 4 ++-- .../clang-tidy/cppcoreguidelines/NoMallocCheck.cpp | 4 ++-- .../clang-tidy/misc/StaticInitializationCycleCheck.cpp | 2 +- clang-tools-extra/clang-tidy/modernize/AvoidBindCheck.cpp | 2 +- .../clang-tidy/modernize/MinMaxUseInitializerListCheck.cpp | 2 +- .../clang-tidy/modernize/UseStartsEndsWithCheck.cpp | 2 +- .../clang-tidy/readability/ContainerSizeEmptyCheck.cpp | 2 +- .../clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp | 2 +- clang-tools-extra/clang-tidy/utils/ExceptionAnalyzer.cpp | 2 +- clang-tools-extra/clang-tidy/utils/Matchers.h | 2 +- 12 files changed, 15 insertions(+), 15 deletions(-) diff --git a/clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp b/clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranchCheck.cpp index 82a02dd6e9344..1bbc601968b01 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) { 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) { 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 588f9bfe076df..8f35e6c80bd4d 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 d7455be873496..96ca8813619a3 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 3d1fc88e7233a..db871b7d0df83 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 dd8fb6889e14d..8f7f07af32a5f 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 3f61a2d5d8bba..842c21ea18570 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 97d71ef774924..53875a5960f3a 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 33fe48048b356..f0b3ff7f30e42 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 812ade0df42c1..79ad5eedd490f 100644 --- a/clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp @@ -215,7 +215,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 af18fda7a3a54..4b087ed1b3993 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
