This is an archive of the discontinued LLVM Phabricator instance.

Differential D37795

SplitEmptyFunction should be true in the Mozilla coding style
ClosedPublic

Authored by sylvestre.ledru on Sep 13 2017, 1:00 AM.

Details

Reviewers
Typz
djasper
Commits
rG2030ade15231: Merging r313182:
rG82c9a0ec12b5: SplitEmptyFunction should be true in the Mozilla coding style
rC313182: SplitEmptyFunction should be true in the Mozilla coding style
rL318858: Merging r313182:
rL313182: 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

Diff Detail

Event Timeline

sylvestre.ledru created this revision.Sep 13 2017, 1:00 AM
Herald added a subscriber: klimek. · View Herald TranscriptSep 13 2017, 1:00 AM
Typz accepted this revision.Sep 13 2017, 7:33 AM
This revision is now accepted and ready to land.Sep 13 2017, 7:33 AM
sylvestre.ledru closed this revision.Sep 13 2017, 1:04 PM

Revision Contents

PathSize
lib/
Format/
2 lines

Diff 114992

lib/Format/Format.cpp

Context not available.
Expanded.BraceWrapping.AfterFunction = true; Expanded.BraceWrapping.AfterFunction = true;
Expanded.BraceWrapping.AfterStruct = true; Expanded.BraceWrapping.AfterStruct = true;
Expanded.BraceWrapping.AfterUnion = true; Expanded.BraceWrapping.AfterUnion = true;
Expanded.BraceWrapping.SplitEmptyFunction = false; Expanded.BraceWrapping.SplitEmptyFunction = true;
Expanded.BraceWrapping.SplitEmptyRecord = false; Expanded.BraceWrapping.SplitEmptyRecord = false;
break; break;
case FormatStyle::BS_Stroustrup: case FormatStyle::BS_Stroustrup:
Context not available.