[clang] [clang-format] Add option AllowShortRecordOnASingleLine (PR #154580)

via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 7 03:40:21 PDT 2025


=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoexpo at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/154580 at github.com>


================
@@ -873,9 +897,11 @@ class LineJoiner {
         return 1;
       } else if (Limit != 0 && !Line.startsWithNamespace() &&
                  !startsExternCBlock(Line)) {
-        // We don't merge short records.
-        if (isRecordLBrace(*Line.Last))
+        // Merge short records only when requested.
+        if (isRecordLBrace(*Line.Last) &&
----------------
owenca wrote:

```suggestion
        if (Line.Last->isOneOf(TT_EnumLBrace, TT_RecordLBrace))
          return 0;
        // Merge short records only when requested.
        if (Line.Last->isOneOf(TT_ClassLBrace, TT_StructLBrace,
                               TT_UnionLBrace) &&
```
as `isRecordLBrace()` includes all of the above.

https://github.com/llvm/llvm-project/pull/154580


More information about the cfe-commits mailing list