[PATCH] D37795: SplitEmptyFunction should be true in the Mozilla coding style

Sylvestre Ledru via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 13 01:00:46 PDT 2017


sylvestre.ledru created this revision.
Herald added a subscriber: klimek.

As defined here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#Classes
See for the downstream bug report: https://bugzilla.mozilla.org/show_bug.cgi?id=1399359


https://reviews.llvm.org/D37795

Files:
  lib/Format/Format.cpp


Index: lib/Format/Format.cpp
===================================================================
--- lib/Format/Format.cpp
+++ lib/Format/Format.cpp
@@ -515,7 +515,7 @@
     Expanded.BraceWrapping.AfterFunction = true;
     Expanded.BraceWrapping.AfterStruct = true;
     Expanded.BraceWrapping.AfterUnion = true;
-    Expanded.BraceWrapping.SplitEmptyFunction = false;
+    Expanded.BraceWrapping.SplitEmptyFunction = true;
     Expanded.BraceWrapping.SplitEmptyRecord = false;
     break;
   case FormatStyle::BS_Stroustrup:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37795.114992.patch
Type: text/x-patch
Size: 527 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170913/86d58347/attachment-0001.bin>


More information about the cfe-commits mailing list