[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