[clang] [clang-format] Change BinPackParameters to enum and add AlwaysOnePerLine (PR #101882)

[Owen Pan via cfe-commits]

================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits