diff options
author | Daniel Jasper <djasper@google.com> | 2018-03-12 10:32:18 +0000 |
---|---|---|
committer | Daniel Jasper <djasper@google.com> | 2018-03-12 10:32:18 +0000 |
commit | 9c95dfe65863122539837ed174d07cd5ecff6010 (patch) | |
tree | 24d2ef801e40e5e3ecff8f8fdceb9b0f88a0d02e /clang/lib/Format/Format.cpp | |
parent | 19e238746b2def4c2944dd8e2a95a45119a401cb (diff) | |
download | bcm5719-llvm-9c95dfe65863122539837ed174d07cd5ecff6010.tar.gz bcm5719-llvm-9c95dfe65863122539837ed174d07cd5ecff6010.zip |
clang-format: Properly handle implicit string concatenation in text protos
Three issues to fix:
- char_constants weren't properly treated as string literals
- Prevening the break after "label: " does not make sense in concunction
with AlwaysBreakBeforeMultilineStrings. It leads to situations where
clang-format just cannot find a viable format (it must break and yet
it must not break).
- AlwaysBreakBeforeMultilineStrings should not be on for LK_TextProto in
Google style.
llvm-svn: 327255
Diffstat (limited to 'clang/lib/Format/Format.cpp')
-rw-r--r-- | clang/lib/Format/Format.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp index 5807db9407f..5c7ce851783 100644 --- a/clang/lib/Format/Format.cpp +++ b/clang/lib/Format/Format.cpp @@ -766,6 +766,7 @@ FormatStyle getGoogleStyle(FormatStyle::LanguageKind Language) { GoogleStyle.JavaScriptWrapImports = false; } else if (Language == FormatStyle::LK_Proto) { GoogleStyle.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_None; + GoogleStyle.AlwaysBreakBeforeMultilineStrings = false; GoogleStyle.SpacesInContainerLiterals = false; GoogleStyle.Cpp11BracedListStyle = false; // This affects protocol buffer options specifications and text protos. |