djasper added inline comments.
================ Comment at: unittests/Format/FormatTest.cpp:8969 + " barrrrrrrrrrrrrr(1) {}", CtorInitializerStyle); + CtorInitializerStyle.BreakConstructorInitializers = FormatStyle::BCIS_BeforeComma; + verifyFormat("Fooooooooooo::Fooooooooooo()\n" ---------------- This is a useless test if it doesn't actually have multiple initializers separated by a comma. ================ Comment at: unittests/Format/FormatTest.cpp:8971 + verifyFormat("Fooooooooooo::Fooooooooooo()\n" + " : barrrrrrrrrrrrrr(1) {}", CtorInitializerStyle); + CtorInitializerStyle.BreakConstructorInitializers = FormatStyle::BCIS_BeforeColon; ---------------- Has this been formatted by clang-format? I think it'd break after the comma. Repository: rC Clang https://reviews.llvm.org/D32525 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits