[PATCH] D33447: clang-format: add option to merge empty function body

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 23 09:41:31 PDT 2017


Typz added inline comments.


================
Comment at: unittests/Format/FormatTest.cpp:6029
+
+  //TODO: this case does not work, not sure if there is reason...
+  MergeEmptyOnly.BreakBeforeBraces = FormatStyle::BS_Custom;
----------------
When testing this patch, I found a bug when `AllowShortFunctionsOnASingleLine = SFS_Empty` and `MergeEmptyOnly.BraceWrapping.AfterFunction = true` : I did not have the time to fix it yet, any idea what might be causing this?


https://reviews.llvm.org/D33447





More information about the cfe-commits mailing list