[clang] [clang-format] Add xxxMaxDigitsNoSeparator (PR #164286)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Nov 22 23:18:02 PST 2025
================
@@ -138,20 +152,28 @@ IntegerLiteralSeparatorFixer::process(const Environment &Env,
Text = Text.substr(Start, Length);
}
auto DigitsPerGroup = Decimal;
- auto MinDigits = DecimalMinDigits;
+ auto MinDigitsInsert = DecimalMinDigitsInsert;
+ auto MaxDigitsRemove = DecimalMaxDigitsRemove;
if (IsBase2) {
DigitsPerGroup = Binary;
- MinDigits = BinaryMinDigits;
+ MinDigitsInsert = BinaryMinDigitsInsert;
+ MaxDigitsRemove = BinaryMaxDigitsRemove;
} else if (IsBase16) {
DigitsPerGroup = Hex;
- MinDigits = HexMinDigits;
+ MinDigitsInsert = HexMinDigitsInsert;
+ MaxDigitsRemove = HexMaxDigitsRemove;
}
const auto SeparatorCount = Text.count(Separator);
const int DigitCount = Length - SeparatorCount;
- const bool RemoveSeparator = DigitsPerGroup < 0 || DigitCount < MinDigits;
+ const bool RemoveSeparator =
+ DigitsPerGroup < 0 || DigitCount <= MaxDigitsRemove;
+ const bool AddSeparator =
+ DigitsPerGroup > 0 && DigitCount >= MinDigitsInsert;
+ if (!RemoveSeparator && !AddSeparator)
+ continue;
----------------
owenca wrote:
Delete.
https://github.com/llvm/llvm-project/pull/164286
More information about the cfe-commits
mailing list