diff options
| author | Ben Hamilton <benhamilton@google.com> | 2018-01-18 18:37:16 +0000 |
|---|---|---|
| committer | Ben Hamilton <benhamilton@google.com> | 2018-01-18 18:37:16 +0000 |
| commit | a8bf4d5625e5df9e4d3d597858ce777161b0aea4 (patch) | |
| tree | cedd3c51dcb2c5bbfb1ef011fe8641d0973d65f3 /lib/Format/Format.cpp | |
| parent | 0a720a33a5c199d88c84d128bab7a54303623fb3 (diff) | |
[ClangFormat] ObjCSpaceBeforeProtocolList should be true in the google style
Summary:
The Google style guide is neutral on whether there should be a
space before the protocol list in an Objective-C @interface or
@implementation.
The majority of Objective-C code in both Apple's public
header files and Google's open-source uses a space before
the protocol list, so this changes the google style to
default ObjCSpaceBeforeProtocolList to true.
Test Plan: make -j12 FormatTests && ./tools/clang/unittests/Format/FormatTests
Reviewers: krasimir, djasper, klimek
Reviewed By: krasimir
Subscribers: klimek, cfe-commits
Differential Revision: https://reviews.llvm.org/D41074
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@322873 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Format/Format.cpp')
| -rw-r--r-- | lib/Format/Format.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Format/Format.cpp b/lib/Format/Format.cpp index 3044b3a56e..88c90a865a 100644 --- a/lib/Format/Format.cpp +++ b/lib/Format/Format.cpp @@ -694,7 +694,7 @@ FormatStyle getGoogleStyle(FormatStyle::LanguageKind Language) { GoogleStyle.IndentCaseLabels = true; GoogleStyle.KeepEmptyLinesAtTheStartOfBlocks = false; GoogleStyle.ObjCSpaceAfterProperty = false; - GoogleStyle.ObjCSpaceBeforeProtocolList = false; + GoogleStyle.ObjCSpaceBeforeProtocolList = true; GoogleStyle.PointerAlignment = FormatStyle::PAS_Left; GoogleStyle.RawStringFormats = {{ FormatStyle::LK_TextProto, |