[PATCH] D27383: Add a new clang-format style option ObjCBlockResetsIndent.

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 3 00:48:10 PST 2016


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





More information about the cfe-commits mailing list