[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