[clang] [polly] [clang-format] Add Automatic and ExceptShortType options for AlwaysBreakAfterReturnType. (PR #78011)

via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 02:35:32 PST 2024


================
@@ -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


More information about the llvm-commits mailing list