[PATCH] D33447: clang-format: add option to merge empty function body
Daniel Jasper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 24 04:50:01 PDT 2017
djasper added a comment.
But that style specifically says that it is only done if the initializer list is wrapped:
https://github.com/facebook/hhvm/blob/master/hphp/doc/coding-conventions.md#constructor-initializer-lists
I.e. we would do the right thing for that style if we would set BraceWrapping.AfterFunction to true and AllowShortFunctionsOnASingleLine to SFS_Empty and it would then format:
Constructor() {}
Constructor() //
: initializer(..)
{}
https://reviews.llvm.org/D33447
More information about the cfe-commits
mailing list