[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