================
@@ -8164,6 +8164,95 @@ TEST_F(FormatTest, BreakFunctionDefinitionParameters) {
                "    int B)\n"
                "    : m_A(A), m_B(B) {}",
                Input, Style);
+
+  Style.BinPackParameters = FormatStyle::BPPS_BinPack;
+  Style.BreakFunctionDefinitionParameters = false;
+  Style.BreakParametersAfter = 2;
+  verifyFormat("void functionDecl(paramA, paramB);\n"
+               "void functionDecl(paramA,\n"
+               "                  paramB,\n"
+               "                  paramC);\n"
+               "void functionDecl(paramA,\n"
+               "                  paramB,\n"
+               "                  paramC,\n"
+               "                  paramD,\n"
+               "                  paramE);\n"
+               "void functionDefinition(int A, int B) {}\n"
+               "void functionDefinition(int A,\n"
+               "                        int B,\n"
+               "                        int C) {}\n"
+               "Class::Class(int A, int B) {}\n"
+               "Class::Class(int A,\n"
+               "             int B,\n"
+               "             int C) {}\n"
+               "call(a, b);\n"
+               "call(a,\n"
+               "     b,\n"
+               "     c);\n"
+               "new Class(a, b);\n"
+               "new Class(a,\n"
+               "          b,\n"
+               "          c);\n"
+               "int x = (a, b);\n"
+               "int y = (a,\n"
+               "         b,\n"
+               "         c);\n"
+               "(a, b);\n"
+               "(a,\n"
+               " b,\n"
+               " c);",
+               Style);
+  Style.BreakParametersAfter = 4;
+  verifyFormat("void functionDecl(paramA);\n"
+               "void functionDecl(paramA, paramB);\n"
+               "void functionDecl(paramA, paramB, paramC);\n"
+               "void functionDecl(paramA, paramB, paramC, paramD);\n"
+               "void functionDecl(paramA,\n"
+               "                  paramB,\n"
+               "                  paramC,\n"
+               "                  paramD,\n"
+               "                  paramE);\n"
+               "void functionDecl(paramA,\n"
+               "                  paramB,\n"
+               "                  paramC,\n"
+               "                  paramD,\n"
+               "                  paramE,\n"
+               "                  paramF);\n"
+               "void functionDefinition(int A, int B, int C, int D) {}\n"
+               "void functionDefinition(int A,\n"
+               "                        int B,\n"
+               "                        int C,\n"
+               "                        int D,\n"
+               "                        int E) {}\n"
+               "Class::Class(int A, int B) {}\n"
+               "Class::Class(int A, int B, int C, int D) {}\n"
+               "Class::Class(int A,\n"
+               "             int B,\n"
+               "             int C,\n"
+               "             int D,\n"
+               "             int E) {}\n"
+               "call(a,\n"
+               "     b,\n"
+               "     c,\n"
+               "     d,\n"
+               "     e);\n"
+               "new Class(a,\n"
+               "          b,\n"
+               "          c,\n"
+               "          d,\n"
+               "          e);\n"
+               "int y = (a,\n"
+               "         b,\n"
+               "         c,\n"
+               "         d,\n"
+               "         e);\n"
+               "(a,\n"
+               " b,\n"
+               " c,\n"
+               " d,\n"
+               " e);",
+               Style);
+  Style.BreakParametersAfter = 0;
----------------
HazardyKnusperkeks wrote:

Unnecessary.

https://github.com/llvm/llvm-project/pull/181281
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to