================
@@ -9906,12 +9944,16 @@ TEST_F(FormatTest, ReturnTypeBreakingStyle) {
   verifyFormat("class B {\n"
                "  int f() { return 1; }\n"
                "  int g();\n"
+               "  long\n"
+               "  foooooooooooooooooooooooooooo::baaaaaaaaaaaaaaaaaaaar();\n"
----------------
rmarker wrote:
Yeah, not sure if this example is completely relevant in the tests for these 
subsequent options.
Though to me, it seems that each of the options has a test with the same block 
of code. Since there are other lines which aren't definitions in the same test.
Wondering if it is worth keeping it so that the symmetry in this area is 
preserved?
Though, I don't have strong feelings either way if you think it isn't needed.

https://github.com/llvm/llvm-project/pull/78011
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to