[PATCH] D33447: clang-format: add option to merge empty function body
Francois Ferrand via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 24 05:15:35 PDT 2017
Typz added a comment.
I am fine with "removing" the options: I only fear it would incorrectly affect existing users/styles, which would possibly end up in the patch being reverted...
I could not find the info in Mozilla coding style, but looking at the code it seems it should be done indeed, but with an extra space in between, like this:
void foo()
{ }
So if you tell me it's fine I will go ahead and replace the option with the `BraceWrapping.AfterFunction && AllowShortFunctionsOnASingleLine >= SFS_Empty` condition.
https://reviews.llvm.org/D33447
More information about the cfe-commits
mailing list