================
@@ -8143,6 +8143,39 @@ TEST_F(FormatTest, BreakFunctionDefinitionParameters) {
                Input, Style);
 }
 
+TEST_F(FormatTest, BreakFunctionDeclarationParameters) {
+  StringRef Input = "void functionDecl(paramA, paramB, paramC);\n"
+                    "void emptyFunctionDefinition() {}\n"
+                    "void functionDefinition(int A, int B, int C) {}\n"
+                    "Class::Class(int A, int B) : m_A(A), m_B(B) {}";
+  verifyFormat(Input);
+
+  FormatStyle Style = getLLVMStyle();
+  EXPECT_FALSE(Style.BreakFunctionDeclarationParameters);
+  Style.BreakFunctionDeclarationParameters = true;
----------------
mydeveloperday wrote:

your documentation suggest that having this as false, doesn't break, but 
actually almost looks like it "joins" the lines, but looking at the code I 
can't see that is the case,

 does false really mean "Leave" or "join"...could you add a test if you mean 
Join but also move it to an enum so we can have a "Leave" case..

How does this interact with other options? doing similar things..

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

Reply via email to