[llvm-branch-commits] [cfe-branch] r318858 - Merging r313182:

Tom Stellard via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Nov 22 10:30:28 PST 2017


Author: tstellar
Date: Wed Nov 22 10:30:28 2017
New Revision: 318858

URL: http://llvm.org/viewvc/llvm-project?rev=318858&view=rev
Log:
Merging r313182:

------------------------------------------------------------------------
r313182 | sylvestre | 2017-09-13 13:03:29 -0700 (Wed, 13 Sep 2017) | 13 lines

SplitEmptyFunction should be true in the Mozilla coding style

Summary:
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

Reviewers: Typz, djasper

Reviewed By: Typz

Subscribers: klimek, cfe-commits

Differential Revision: https://reviews.llvm.org/D37795
------------------------------------------------------------------------

Modified:
    cfe/branches/release_50/lib/Format/Format.cpp

Modified: cfe/branches/release_50/lib/Format/Format.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/branches/release_50/lib/Format/Format.cpp?rev=318858&r1=318857&r2=318858&view=diff
==============================================================================
--- cfe/branches/release_50/lib/Format/Format.cpp (original)
+++ cfe/branches/release_50/lib/Format/Format.cpp Wed Nov 22 10:30:28 2017
@@ -506,7 +506,7 @@ static FormatStyle expandPresets(const F
     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:




More information about the llvm-branch-commits mailing list