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

Sean Lu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 3 02:45:24 PST 2016


yixiang added a comment.

> 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?

Unit tests added.

> 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

Thanks for pointing that out to me.


https://reviews.llvm.org/D27383





More information about the cfe-commits mailing list