[clang-tools-extra] [clang-tidy][NFC] Fix clang-format (PR #192044)
Baranov Victor via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 14 05:21:27 PDT 2026
https://github.com/vbvictor created https://github.com/llvm/llvm-project/pull/192044
None
>From 9c575f56115767c0afce79e857837c6e20a3482f Mon Sep 17 00:00:00 2001
From: Victor Baranov <bar.victor.2002 at gmail.com>
Date: Tue, 14 Apr 2026 15:20:55 +0300
Subject: [PATCH] [clang-tidy][NFC] Fix clang-format
---
.../bugprone/RedundantBranchConditionCheck.cpp | 15 +++++++--------
.../misc/MisleadingBidirectionalCheck.cpp | 6 ++++--
2 files changed, 11 insertions(+), 10 deletions(-)
diff --git a/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp
index e0d4aa382d961..874664a959d2a 100644
--- a/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp
@@ -166,14 +166,13 @@ void RedundantBranchConditionCheck::check(
CondOp->getRHS()->getBeginLoc().getLocWithOffset(-1);
Diag << FixItHint::CreateRemoval(CharSourceRange::getTokenRange(
CondOp->getLHS()->getBeginLoc(), BeforeRHS));
- } else {
- if (const auto NextToken = utils::lexer::findNextTokenSkippingComments(
- CondOp->getLHS()->getEndLoc(), *Result.SourceManager,
- getLangOpts())) {
- const SourceLocation AfterLHS = NextToken->getLocation();
- Diag << FixItHint::CreateRemoval(CharSourceRange::getTokenRange(
- AfterLHS, CondOp->getRHS()->getEndLoc()));
- }
+ } else if (const auto NextToken =
+ utils::lexer::findNextTokenSkippingComments(
+ CondOp->getLHS()->getEndLoc(), *Result.SourceManager,
+ getLangOpts())) {
+ const SourceLocation AfterLHS = NextToken->getLocation();
+ Diag << FixItHint::CreateRemoval(CharSourceRange::getTokenRange(
+ AfterLHS, CondOp->getRHS()->getEndLoc()));
}
}
}
diff --git a/clang-tools-extra/clang-tidy/misc/MisleadingBidirectionalCheck.cpp b/clang-tools-extra/clang-tidy/misc/MisleadingBidirectionalCheck.cpp
index 9294c2b57e15e..894c6db8975f2 100644
--- a/clang-tools-extra/clang-tidy/misc/MisleadingBidirectionalCheck.cpp
+++ b/clang-tools-extra/clang-tidy/misc/MisleadingBidirectionalCheck.cpp
@@ -66,16 +66,18 @@ static bool containsMisleadingBidi(StringRef Buffer,
if (CodePoint == RLO || CodePoint == RLE || CodePoint == LRO ||
CodePoint == LRE) {
BidiContexts.push_back(PDF);
+ }
// Close PDF Context.
- } else if (CodePoint == PDF) {
+ else if (CodePoint == PDF) {
if (!BidiContexts.empty() && BidiContexts.back() == PDF)
BidiContexts.pop_back();
}
// Open a PDI Context.
else if (CodePoint == RLI || CodePoint == LRI || CodePoint == FSI) {
BidiContexts.push_back(PDI);
+ }
// Close a PDI Context.
- } else if (CodePoint == PDI) {
+ else if (CodePoint == PDI) {
auto R = llvm::find(llvm::reverse(BidiContexts), PDI);
if (R != BidiContexts.rend())
BidiContexts.resize(BidiContexts.rend() - R - 1);
More information about the cfe-commits
mailing list