[PATCH] D32525: [clang-format] Add SpaceBeforeColon option

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 28 09:04:03 PST 2018


Typz added inline comments.


================
Comment at: unittests/Format/FormatTest.cpp:8969
+               "    barrrrrrrrrrrrrr(1) {}", CtorInitializerStyle);
+  CtorInitializerStyle.BreakConstructorInitializers = FormatStyle::BCIS_BeforeComma;
+  verifyFormat("Fooooooooooo::Fooooooooooo()\n"
----------------
djasper wrote:
> This is a useless test if it doesn't actually have multiple initializers separated by a comma.
since we are interested only in the behavior next to the colon, I think it is relevant... But I'll add extra parameter to make it more realistic,


Repository:
  rC Clang

https://reviews.llvm.org/D32525





More information about the cfe-commits mailing list