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

[via cfe-commits]

=?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>


================
@@ -928,9 +952,15 @@ class LineJoiner {
         return 0;
       Limit -= 2;
       unsigned MergedLines = 0;
-      if (Style.AllowShortBlocksOnASingleLine != FormatStyle::SBS_Never ||
-          (I[1]->First == I[1]->Last && I + 2 != E &&
-           I[2]->First->is(tok::r_brace))) {
+
+      bool TryMergeBlock =
+          Style.AllowShortBlocksOnASingleLine != FormatStyle::SBS_Never;
+      bool TryMergeRecord =
+          Style.AllowShortRecordOnASingleLine == FormatStyle::SRS_Always;
+      bool NextIsEmptyBlock = I[1]->First == I[1]->Last && I + 2 != E &&
----------------
owenca wrote:

Please make them `const`.

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