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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
