djasper added a comment.
I think this generally looks good, but needs a few more tests.
================
Comment at: include/clang/Format/Format.h:1204
+ /// \brief If ``false``, spaces will be removed before constructor
initializer
+ /// colon.
----------------
When this file is changed, can you also run docs/tools/dump_format_style.py to
update the docs?
================
Comment at: unittests/Format/FormatTest.cpp:7539
+ verifyFormat("class Foo : public Bar {};", CtorInitializerStyle);
+ verifyFormat("Foo::Foo(): foo(1) {}", CtorInitializerStyle);
+ verifyFormat("for (auto a : b) {\n}", CtorInitializerStyle);
----------------
Can you add tests for the other values of BreakConstructorInitializers
(BCIS_BeforeColon, BCIS_BeforeComma)?
https://reviews.llvm.org/D32525
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
