[PATCH] D34395: clang-format: add options to merge empty record body

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 26 01:28:09 PDT 2017


Typz added a comment.

I don't know if some style would want different styles, and I agree with you on principle; but since the brace wrapping is already configured for each kind of record, I choose to keep things consistent and have flags for each kind of record.
But I can merge the options, and keep only `SplitEmptyRecord` and `SplitEmptyNamespace`, if you really think it is better.


https://reviews.llvm.org/D34395





More information about the cfe-commits mailing list