[clang] [clang-format] Change BinPackParameters to enum and add AlwaysOnePerLine (PR #101882)
Owen Pan via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 21 23:40:25 PDT 2024
================
@@ -403,13 +416,29 @@ TEST_F(FormatTestComments, UnderstandsBlockComments) {
verifyFormat("f(/* aaaaaaaaaaaaaaaaaa = */\n"
" aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa);");
- FormatStyle NoBinPacking = getLLVMStyle();
- NoBinPacking.BinPackParameters = false;
+ verifyFormat(
+ "int aaaaaaaaaaaaa(/* 1st */ int bbbbbbbbbb, /* 2nd */ int ccccccccccc,\n"
+ " /* 3rd */ int dddddddddddd);");
+
+ auto Style = getLLVMStyle();
+ Style.BinPackParameters = FormatStyle::BPPS_OnePerLine;
verifyFormat("aaaaaaaa(/* parameter 1 */ aaaaaa,\n"
" /* parameter 2 */ aaaaaa,\n"
" /* parameter 3 */ aaaaaa,\n"
" /* parameter 4 */ aaaaaa);",
- NoBinPacking);
+ Style);
+
+ Style.BinPackParameters = FormatStyle::BPPS_OnePerLine;
----------------
owenca wrote:
```suggestion
```
Redundant.
https://github.com/llvm/llvm-project/pull/101882
More information about the cfe-commits
mailing list