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 | f84f118eb855f7a879fffbd40616c6d542480637 (patch) | |
| tree | 55535f6dd071371b785aeba18eb9c9846eb5a243 /clang/lib/Format | |
| parent | 18850525893c525e24d6f3019645765b579683af (diff) | |
| download | bcm5719-llvm-f84f118eb855f7a879fffbd40616c6d542480637.tar.gz bcm5719-llvm-f84f118eb855f7a879fffbd40616c6d542480637.zip | |
[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
llvm-svn: 322873
Diffstat (limited to 'clang/lib/Format')
| -rw-r--r-- | clang/lib/Format/Format.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp index 3044b3a56e2..88c90a865a8 100644 --- a/clang/lib/Format/Format.cpp +++ b/clang/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, |
