r313182 - SplitEmptyFunction should be true in the Mozilla coding style
Sylvestre Ledru via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 13 13:03:29 PDT 2017
Author: sylvestre
Date: Wed Sep 13 13:03:29 2017
New Revision: 313182
URL: http://llvm.org/viewvc/llvm-project?rev=313182&view=rev
Log:
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/trunk/lib/Format/Format.cpp
Modified: cfe/trunk/lib/Format/Format.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Format/Format.cpp?rev=313182&r1=313181&r2=313182&view=diff
==============================================================================
--- cfe/trunk/lib/Format/Format.cpp (original)
+++ cfe/trunk/lib/Format/Format.cpp Wed Sep 13 13:03:29 2017
@@ -515,7 +515,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 cfe-commits
mailing list