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

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 20 08:52:40 PDT 2024


================
@@ -436,6 +435,19 @@ TEST(ConfigParseTest, ParsesConfiguration) {
   CHECK_PARSE("BreakBeforeInheritanceComma: true", BreakInheritanceList,
               FormatStyle::BILS_BeforeComma);
 
+  Style.BinPackParameters = FormatStyle::BPPS_BinPack;
+  CHECK_PARSE("BinPackParameters: OnePerLine", BinPackParameters,
+              FormatStyle::BPPS_OnePerLine);
+  CHECK_PARSE("BinPackParameters: BinPack", BinPackParameters,
+              FormatStyle::BPPS_BinPack);
----------------
owenca wrote:

```suggestion
  Style.BinPackParameters = FormatStyle::BPPS_OnePerLine;
  CHECK_PARSE("BinPackParameters: BinPack", BinPackParameters,
              FormatStyle::BPPS_BinPack);
  CHECK_PARSE("BinPackParameters: OnePerLine", BinPackParameters,
              FormatStyle::BPPS_OnePerLine);
```

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


More information about the cfe-commits mailing list