djasper added a comment.

I think generally, this makes sense.

Can you add tests in unittests/Format/FormatTest.cpp (probably next to 
TEST_F(FormatTest, FormatsBlocks) {..})?
What's the behavior when there is more than one block?

Also note that we have some requirements for additional options (mainly to 
ensure long-term maintainability):
http://clang.llvm.org/docs/ClangFormatStyleOptions.html#adding-additional-style-options


https://reviews.llvm.org/D27383



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to