================
@@ -7771,6 +7771,37 @@ TEST_F(FormatTest, ConstructorInitializers) {
                "Constructor() :\n"
                "    // Comment forcing unwanted break.\n"
                "    aaaa(aaaa) {}");
+
+  // Braced initializers with trailing commas.
+  verifyFormat("MyClass::MyClass()\n"
+               "    : aaaa{\n"
+               "          0,\n"
+               "      } {}",
+               "MyClass::MyClass():aaaa{0,}{}", getGoogleStyle());
+  verifyFormat("MyClass::MyClass()\n"
+               "    : aaaa(0),\n"
+               "      bbbb{\n"
+               "          0,\n"
+               "      } {}",
+               "MyClass::MyClass():aaaa(0),bbbb{0,}{}", getGoogleStyle());
+  verifyFormat("MyClass::MyClass()\n"
+               "    : aaaa(0),\n"
+               "      bbbb{\n"
+               "          0,\n"
+               "      },\n"
+               "      cccc{\n"
+               "          0,\n"
+               "      } {}",
+               "MyClass::MyClass():aaaa(0),bbbb{0,},cccc{0,}{}",
+               getGoogleStyle());
----------------
owenca wrote:

I would not add these test cases as they can pass without this patch.

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

Reply via email to