=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina <itzexpoe...@gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/154...@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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
