diff options
author | Daniel Jasper <djasper@google.com> | 2014-06-04 12:40:57 +0000 |
---|---|---|
committer | Daniel Jasper <djasper@google.com> | 2014-06-04 12:40:57 +0000 |
commit | a69ca9be127f26e22053243f7562d1079339314f (patch) | |
tree | 292fc64d2f53004064e8380d5e5182ef01894d65 /clang/lib/Format/ContinuationIndenter.cpp | |
parent | 5d8d72c5df0b7b80b30a2d66acb7a03b7933bdc6 (diff) | |
download | bcm5719-llvm-a69ca9be127f26e22053243f7562d1079339314f.tar.gz bcm5719-llvm-a69ca9be127f26e22053243f7562d1079339314f.zip |
clang-format: Leave empty lines within UnwrappedLines.
These are commonly used to structure things like enums or long braced
lists. There doesn't seem to be a good reason to have the behavior in
such structures be different from the behavior between statements.
llvm-svn: 210183
Diffstat (limited to 'clang/lib/Format/ContinuationIndenter.cpp')
-rw-r--r-- | clang/lib/Format/ContinuationIndenter.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/clang/lib/Format/ContinuationIndenter.cpp b/clang/lib/Format/ContinuationIndenter.cpp index 5e63d0cb9be..308fc623d82 100644 --- a/clang/lib/Format/ContinuationIndenter.cpp +++ b/clang/lib/Format/ContinuationIndenter.cpp @@ -413,10 +413,8 @@ unsigned ContinuationIndenter::addTokenOnNewLine(LineState &State, State.Stack.back().BreakBeforeParameter = true; if (!DryRun) { - unsigned Newlines = 1; - if (Current.is(tok::comment)) - Newlines = std::max(Newlines, std::min(Current.NewlinesBefore, - Style.MaxEmptyLinesToKeep + 1)); + unsigned Newlines = std::max( + 1u, std::min(Current.NewlinesBefore, Style.MaxEmptyLinesToKeep + 1)); Whitespaces.replaceWhitespace(Current, Newlines, State.Stack.back().IndentLevel, State.Column, State.Column, State.Line->InPPDirective); |