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

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 24 02:56:12 PDT 2017


djasper added a comment.

Does anything speak against making this behavior happen with AllowShortFunctionsOnASingleLine = SFS_Empty and MergeEmptyOnly.BraceWrapping.AfterFunction = true? I mean without the extra style option?



================
Comment at: unittests/Format/FormatTest.cpp:6067
+  verifyFormat("int f()\n"
+			   "{\n"
+               "  return 42;\n"
----------------
indent


https://reviews.llvm.org/D33447





More information about the cfe-commits mailing list