klimek accepted this revision.
klimek added a comment.
In https://reviews.llvm.org/D46024#1129350, @hans wrote:
> In https://reviews.llvm.org/D46024#1121242, @rkirsling wrote:
> > FWIW, please note that this space-before-brace style is not specific to
> > WebKit; CppCoreGuidelines exhibits it as well:
> > http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es23-prefer-the--initializer-syntax
> This and WebKit's style seem like compelling arguments to support this option.
> klimek, djasper: Do you have any objections against landing this?
Generally LG minus that I'd significantly reduce the number of test cases :)
Comment at: unittests/Format/FormatTest.cpp:6980
+ FormatStyle SpaceBeforeBrace = getLLVMStyle();
There are super many redundant test cases here - I don't think we need to test
that brace init detection works here, again.
I think given the code change we basically need 2 tests:
one where the previous opens a scope, and one where it doesn't.
cfe-commits mailing list