[clang-tools-extra] [clang-tidy][NFC] Apply readability-redundant-parentheses (PR #213848)
Baranov Victor via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 3 23:27:20 PDT 2026
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 <bar.victor.2002 at gmail.com>
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);
}
More information about the cfe-commits
mailing list