Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1 | //===--- ContinuationIndenter.cpp - Format C++ code -----------------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | /// |
| 10 | /// \file |
| 11 | /// \brief This file implements the continuation indenter. |
| 12 | /// |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 15 | #include "BreakableToken.h" |
| 16 | #include "ContinuationIndenter.h" |
| 17 | #include "WhitespaceManager.h" |
| 18 | #include "clang/Basic/OperatorPrecedence.h" |
| 19 | #include "clang/Basic/SourceManager.h" |
| 20 | #include "clang/Format/Format.h" |
| 21 | #include "llvm/Support/Debug.h" |
| 22 | #include <string> |
| 23 | |
Chandler Carruth | 1034666 | 2014-04-22 03:17:02 +0000 | [diff] [blame] | 24 | #define DEBUG_TYPE "format-formatter" |
| 25 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 26 | namespace clang { |
| 27 | namespace format { |
| 28 | |
| 29 | // Returns the length of everything up to the first possible line break after |
| 30 | // the ), ], } or > matching \c Tok. |
| 31 | static unsigned getLengthToMatchingParen(const FormatToken &Tok) { |
Craig Topper | 2145bc0 | 2014-05-09 08:15:10 +0000 | [diff] [blame] | 32 | if (!Tok.MatchingParen) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 33 | return 0; |
| 34 | FormatToken *End = Tok.MatchingParen; |
| 35 | while (End->Next && !End->Next->CanBreakBefore) { |
| 36 | End = End->Next; |
| 37 | } |
| 38 | return End->TotalLength - Tok.TotalLength + 1; |
| 39 | } |
| 40 | |
Daniel Jasper | 4c6e005 | 2013-08-27 14:24:43 +0000 | [diff] [blame] | 41 | // Returns \c true if \c Tok is the "." or "->" of a call and starts the next |
| 42 | // segment of a builder type call. |
| 43 | static bool startsSegmentOfBuilderTypeCall(const FormatToken &Tok) { |
| 44 | return Tok.isMemberAccess() && Tok.Previous && Tok.Previous->closesScope(); |
| 45 | } |
| 46 | |
Daniel Jasper | ec01cd6 | 2013-10-08 05:11:18 +0000 | [diff] [blame] | 47 | // Returns \c true if \c Current starts a new parameter. |
| 48 | static bool startsNextParameter(const FormatToken &Current, |
| 49 | const FormatStyle &Style) { |
| 50 | const FormatToken &Previous = *Current.Previous; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 51 | if (Current.is(TT_CtorInitializerComma) && |
Daniel Jasper | ec01cd6 | 2013-10-08 05:11:18 +0000 | [diff] [blame] | 52 | Style.BreakConstructorInitializersBeforeComma) |
| 53 | return true; |
| 54 | return Previous.is(tok::comma) && !Current.isTrailingComment() && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 55 | (Previous.isNot(TT_CtorInitializerComma) || |
Daniel Jasper | ec01cd6 | 2013-10-08 05:11:18 +0000 | [diff] [blame] | 56 | !Style.BreakConstructorInitializersBeforeComma); |
| 57 | } |
| 58 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 59 | ContinuationIndenter::ContinuationIndenter(const FormatStyle &Style, |
Daniel Jasper | d0ec0d6 | 2014-11-04 12:41:02 +0000 | [diff] [blame] | 60 | const AdditionalKeywords &Keywords, |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 61 | SourceManager &SourceMgr, |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 62 | WhitespaceManager &Whitespaces, |
| 63 | encoding::Encoding Encoding, |
| 64 | bool BinPackInconclusiveFunctions) |
Daniel Jasper | d0ec0d6 | 2014-11-04 12:41:02 +0000 | [diff] [blame] | 65 | : Style(Style), Keywords(Keywords), SourceMgr(SourceMgr), |
| 66 | Whitespaces(Whitespaces), Encoding(Encoding), |
Alexander Kornienko | ce9161a | 2014-01-02 15:13:14 +0000 | [diff] [blame] | 67 | BinPackInconclusiveFunctions(BinPackInconclusiveFunctions), |
| 68 | CommentPragmasRegex(Style.CommentPragmas) {} |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 69 | |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 70 | LineState ContinuationIndenter::getInitialState(unsigned FirstIndent, |
Daniel Jasper | 1c5d9df | 2013-09-06 07:54:20 +0000 | [diff] [blame] | 71 | const AnnotatedLine *Line, |
| 72 | bool DryRun) { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 73 | LineState State; |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 74 | State.FirstIndent = FirstIndent; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 75 | State.Column = FirstIndent; |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 76 | State.Line = Line; |
| 77 | State.NextToken = Line->First; |
Alexander Kornienko | e2e0387 | 2013-10-14 00:46:35 +0000 | [diff] [blame] | 78 | State.Stack.push_back(ParenState(FirstIndent, Line->Level, FirstIndent, |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 79 | /*AvoidBinPacking=*/false, |
| 80 | /*NoLineBreak=*/false)); |
| 81 | State.LineContainsContinuedForLoopSection = false; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 82 | State.StartOfStringLiteral = 0; |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 83 | State.StartOfLineLevel = 0; |
| 84 | State.LowestLevelOnLine = 0; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 85 | State.IgnoreStackForComparison = false; |
| 86 | |
| 87 | // The first token has already been indented and thus consumed. |
Daniel Jasper | 1c5d9df | 2013-09-06 07:54:20 +0000 | [diff] [blame] | 88 | moveStateToNextToken(State, DryRun, /*Newline=*/false); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 89 | return State; |
| 90 | } |
| 91 | |
| 92 | bool ContinuationIndenter::canBreak(const LineState &State) { |
| 93 | const FormatToken &Current = *State.NextToken; |
| 94 | const FormatToken &Previous = *Current.Previous; |
| 95 | assert(&Previous == Current.Previous); |
Nico Weber | beb0393 | 2015-01-09 23:25:06 +0000 | [diff] [blame] | 96 | if (!Current.CanBreakBefore && |
| 97 | !(State.Stack.back().BreakBeforeClosingBrace && |
Daniel Jasper | bd73bcf | 2015-10-27 13:42:08 +0000 | [diff] [blame] | 98 | Current.closesBlockOrBlockTypeList(Style))) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 99 | return false; |
| 100 | // The opening "{" of a braced list has to be on the same line as the first |
| 101 | // element if it is nested in another braced init list or function call. |
| 102 | if (!Current.MustBreakBefore && Previous.is(tok::l_brace) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 103 | Previous.isNot(TT_DictLiteral) && Previous.BlockKind == BK_BracedInit && |
Daniel Jasper | b05a81d | 2014-05-09 13:11:16 +0000 | [diff] [blame] | 104 | Previous.Previous && |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 105 | Previous.Previous->isOneOf(tok::l_brace, tok::l_paren, tok::comma)) |
| 106 | return false; |
| 107 | // This prevents breaks like: |
| 108 | // ... |
| 109 | // SomeParameter, OtherParameter).DoSomething( |
| 110 | // ... |
| 111 | // As they hide "DoSomething" and are generally bad for readability. |
Daniel Jasper | 8f59ae5 | 2014-03-11 11:03:26 +0000 | [diff] [blame] | 112 | if (Previous.opensScope() && Previous.isNot(tok::l_brace) && |
Daniel Jasper | fcfac10 | 2014-07-15 09:00:34 +0000 | [diff] [blame] | 113 | State.LowestLevelOnLine < State.StartOfLineLevel && |
| 114 | State.LowestLevelOnLine < Current.NestingLevel) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 115 | return false; |
Daniel Jasper | 4c6e005 | 2013-08-27 14:24:43 +0000 | [diff] [blame] | 116 | if (Current.isMemberAccess() && State.Stack.back().ContainsUnwrappedBuilder) |
| 117 | return false; |
Daniel Jasper | 114a2bc | 2014-06-03 12:02:45 +0000 | [diff] [blame] | 118 | |
| 119 | // Don't create a 'hanging' indent if there are multiple blocks in a single |
| 120 | // statement. |
Daniel Jasper | 4b44449 | 2014-11-21 13:38:53 +0000 | [diff] [blame] | 121 | if (Previous.is(tok::l_brace) && State.Stack.size() > 1 && |
| 122 | State.Stack[State.Stack.size() - 2].NestedBlockInlined && |
Daniel Jasper | 114a2bc | 2014-06-03 12:02:45 +0000 | [diff] [blame] | 123 | State.Stack[State.Stack.size() - 2].HasMultipleNestedBlocks) |
| 124 | return false; |
| 125 | |
Daniel Jasper | e068ac7 | 2014-10-27 17:13:59 +0000 | [diff] [blame] | 126 | // Don't break after very short return types (e.g. "void") as that is often |
| 127 | // unexpected. |
Zachary Turner | 448592e | 2015-12-18 22:20:15 +0000 | [diff] [blame] | 128 | if (Current.is(TT_FunctionDeclarationName) && State.Column < 6) { |
| 129 | if (Style.AlwaysBreakAfterReturnType == FormatStyle::RTBS_None) |
| 130 | return false; |
| 131 | } |
Daniel Jasper | e068ac7 | 2014-10-27 17:13:59 +0000 | [diff] [blame] | 132 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 133 | return !State.Stack.back().NoLineBreak; |
| 134 | } |
| 135 | |
| 136 | bool ContinuationIndenter::mustBreak(const LineState &State) { |
| 137 | const FormatToken &Current = *State.NextToken; |
| 138 | const FormatToken &Previous = *Current.Previous; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 139 | if (Current.MustBreakBefore || Current.is(TT_InlineASMColon)) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 140 | return true; |
Daniel Jasper | 1db6c38 | 2013-10-22 15:30:28 +0000 | [diff] [blame] | 141 | if (State.Stack.back().BreakBeforeClosingBrace && |
Daniel Jasper | bd73bcf | 2015-10-27 13:42:08 +0000 | [diff] [blame] | 142 | Current.closesBlockOrBlockTypeList(Style)) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 143 | return true; |
| 144 | if (Previous.is(tok::semi) && State.LineContainsContinuedForLoopSection) |
| 145 | return true; |
Daniel Jasper | ec01cd6 | 2013-10-08 05:11:18 +0000 | [diff] [blame] | 146 | if ((startsNextParameter(Current, Style) || Previous.is(tok::semi) || |
Daniel Jasper | 253dad23 | 2015-11-23 15:55:45 +0000 | [diff] [blame] | 147 | (Previous.is(TT_TemplateCloser) && Current.is(TT_StartOfName)) || |
Daniel Jasper | 3e0dcc2 | 2015-05-27 05:37:40 +0000 | [diff] [blame] | 148 | (Style.BreakBeforeTernaryOperators && Current.is(TT_ConditionalExpr) && |
| 149 | Previous.isNot(tok::question)) || |
Daniel Jasper | 165b29e | 2013-11-08 00:57:11 +0000 | [diff] [blame] | 150 | (!Style.BreakBeforeTernaryOperators && |
Daniel Jasper | 3e0dcc2 | 2015-05-27 05:37:40 +0000 | [diff] [blame] | 151 | Previous.is(TT_ConditionalExpr))) && |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 152 | State.Stack.back().BreakBeforeParameter && !Current.isTrailingComment() && |
| 153 | !Current.isOneOf(tok::r_paren, tok::r_brace)) |
| 154 | return true; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 155 | if (((Previous.is(TT_DictLiteral) && Previous.is(tok::l_brace)) || |
Daniel Jasper | ccff4d1 | 2016-01-04 07:27:33 +0000 | [diff] [blame] | 156 | (Previous.is(TT_ArrayInitializerLSquare) && |
| 157 | Previous.ParameterCount > 1)) && |
Daniel Jasper | db8804b | 2014-04-14 12:11:07 +0000 | [diff] [blame] | 158 | Style.ColumnLimit > 0 && |
Daniel Jasper | 199d0c9 | 2015-06-02 15:14:21 +0000 | [diff] [blame] | 159 | getLengthToMatchingParen(Previous) + State.Column - 1 > |
| 160 | getColumnLimit(State)) |
Daniel Jasper | d489dd3 | 2013-10-20 16:45:46 +0000 | [diff] [blame] | 161 | return true; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 162 | if (Current.is(TT_CtorInitializerColon) && |
Daniel Jasper | 7b259cd | 2015-08-27 11:59:31 +0000 | [diff] [blame] | 163 | (State.Column + State.Line->Last->TotalLength - Current.TotalLength + 2 > |
| 164 | getColumnLimit(State) || |
| 165 | State.Stack.back().BreakBeforeParameter) && |
Daniel Jasper | d74cf40 | 2014-04-08 12:46:38 +0000 | [diff] [blame] | 166 | ((Style.AllowShortFunctionsOnASingleLine != FormatStyle::SFS_All) || |
Daniel Jasper | 5d2587d | 2014-03-27 16:14:13 +0000 | [diff] [blame] | 167 | Style.BreakConstructorInitializersBeforeComma || Style.ColumnLimit != 0)) |
| 168 | return true; |
Daniel Jasper | 2746a30 | 2015-05-06 13:13:03 +0000 | [diff] [blame] | 169 | if (Current.is(TT_SelectorName) && State.Stack.back().ObjCSelectorNameFound && |
| 170 | State.Stack.back().BreakBeforeParameter) |
| 171 | return true; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 172 | |
Daniel Jasper | 2aaedd3 | 2015-06-18 09:12:47 +0000 | [diff] [blame] | 173 | unsigned NewLineColumn = getNewLineColumn(State); |
| 174 | if (State.Column < NewLineColumn) |
Daniel Jasper | 5d2587d | 2014-03-27 16:14:13 +0000 | [diff] [blame] | 175 | return false; |
Daniel Jasper | 173504e | 2015-05-10 21:15:07 +0000 | [diff] [blame] | 176 | |
Daniel Jasper | 2aaedd3 | 2015-06-18 09:12:47 +0000 | [diff] [blame] | 177 | if (Style.AlwaysBreakBeforeMultilineStrings && |
Daniel Jasper | 1bf729c | 2015-06-18 16:05:17 +0000 | [diff] [blame] | 178 | (NewLineColumn == State.FirstIndent + Style.ContinuationIndentWidth || |
Daniel Jasper | 9fb676a | 2015-06-19 10:32:28 +0000 | [diff] [blame] | 179 | Previous.is(tok::comma) || Current.NestingLevel < 2) && |
Daniel Jasper | 2aaedd3 | 2015-06-18 09:12:47 +0000 | [diff] [blame] | 180 | !Previous.isOneOf(tok::kw_return, tok::lessless, tok::at) && |
| 181 | !Previous.isOneOf(TT_InlineASMColon, TT_ConditionalExpr) && |
| 182 | nextIsMultilineString(State)) |
| 183 | return true; |
| 184 | |
Daniel Jasper | 173504e | 2015-05-10 21:15:07 +0000 | [diff] [blame] | 185 | // Using CanBreakBefore here and below takes care of the decision whether the |
| 186 | // current style uses wrapping before or after operators for the given |
| 187 | // operator. |
| 188 | if (Previous.is(TT_BinaryOperator) && Current.CanBreakBefore) { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 189 | // If we need to break somewhere inside the LHS of a binary expression, we |
| 190 | // should also break after the operator. Otherwise, the formatting would |
| 191 | // hide the operator precedence, e.g. in: |
| 192 | // if (aaaaaaaaaaaaaa == |
| 193 | // bbbbbbbbbbbbbb && c) {.. |
| 194 | // For comparisons, we only apply this rule, if the LHS is a binary |
| 195 | // expression itself as otherwise, the line breaks seem superfluous. |
| 196 | // We need special cases for ">>" which we have split into two ">" while |
| 197 | // lexing in order to make template parsing easier. |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 198 | bool IsComparison = (Previous.getPrecedence() == prec::Relational || |
| 199 | Previous.getPrecedence() == prec::Equality) && |
| 200 | Previous.Previous && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 201 | Previous.Previous->isNot(TT_BinaryOperator); // For >>. |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 202 | bool LHSIsBinaryExpr = |
Daniel Jasper | 562ecd4 | 2013-09-06 08:08:14 +0000 | [diff] [blame] | 203 | Previous.Previous && Previous.Previous->EndsBinaryExpression; |
Daniel Jasper | 173504e | 2015-05-10 21:15:07 +0000 | [diff] [blame] | 204 | if ((!IsComparison || LHSIsBinaryExpr) && !Current.isTrailingComment() && |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 205 | Previous.getPrecedence() != prec::Assignment && |
| 206 | State.Stack.back().BreakBeforeParameter) |
| 207 | return true; |
Daniel Jasper | 173504e | 2015-05-10 21:15:07 +0000 | [diff] [blame] | 208 | } else if (Current.is(TT_BinaryOperator) && Current.CanBreakBefore && |
| 209 | State.Stack.back().BreakBeforeParameter) { |
| 210 | return true; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 211 | } |
| 212 | |
| 213 | // Same as above, but for the first "<<" operator. |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 214 | if (Current.is(tok::lessless) && Current.isNot(TT_OverloadedOperator) && |
Alexander Kornienko | 86b2dfd | 2014-03-06 15:13:08 +0000 | [diff] [blame] | 215 | State.Stack.back().BreakBeforeParameter && |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 216 | State.Stack.back().FirstLessLess == 0) |
| 217 | return true; |
| 218 | |
Daniel Jasper | 211e132 | 2014-12-08 20:08:04 +0000 | [diff] [blame] | 219 | if (Current.NestingLevel == 0 && !Current.isTrailingComment()) { |
Daniel Jasper | f090f03 | 2015-05-18 09:47:22 +0000 | [diff] [blame] | 220 | // Always break after "template <...>" and leading annotations. This is only |
| 221 | // for cases where the entire line does not fit on a single line as a |
| 222 | // different LineFormatter would be used otherwise. |
Daniel Jasper | 211e132 | 2014-12-08 20:08:04 +0000 | [diff] [blame] | 223 | if (Previous.ClosesTemplateDeclaration) |
| 224 | return true; |
Daniel Jasper | 47bbda0 | 2015-05-18 13:47:23 +0000 | [diff] [blame] | 225 | if (Previous.is(TT_FunctionAnnotationRParen)) |
Daniel Jasper | f090f03 | 2015-05-18 09:47:22 +0000 | [diff] [blame] | 226 | return true; |
Nico Weber | beb0393 | 2015-01-09 23:25:06 +0000 | [diff] [blame] | 227 | if (Previous.is(TT_LeadingJavaAnnotation) && Current.isNot(tok::l_paren) && |
| 228 | Current.isNot(TT_LeadingJavaAnnotation)) |
Daniel Jasper | 211e132 | 2014-12-08 20:08:04 +0000 | [diff] [blame] | 229 | return true; |
| 230 | } |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 231 | |
Daniel Jasper | 4355e7f | 2014-07-09 07:50:33 +0000 | [diff] [blame] | 232 | // If the return type spans multiple lines, wrap before the function name. |
Birunthan Mohanathas | 525579d | 2015-07-15 19:11:58 +0000 | [diff] [blame] | 233 | if ((Current.is(TT_FunctionDeclarationName) || |
| 234 | (Current.is(tok::kw_operator) && !Previous.is(tok::coloncolon))) && |
Daniel Jasper | 4355e7f | 2014-07-09 07:50:33 +0000 | [diff] [blame] | 235 | State.Stack.back().BreakBeforeParameter) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 236 | return true; |
Daniel Jasper | 4355e7f | 2014-07-09 07:50:33 +0000 | [diff] [blame] | 237 | |
Daniel Jasper | 4c6e005 | 2013-08-27 14:24:43 +0000 | [diff] [blame] | 238 | if (startsSegmentOfBuilderTypeCall(Current) && |
Daniel Jasper | f8151e9 | 2013-08-30 07:12:40 +0000 | [diff] [blame] | 239 | (State.Stack.back().CallContinuation != 0 || |
Daniel Jasper | 05cd929 | 2015-03-26 18:46:28 +0000 | [diff] [blame] | 240 | State.Stack.back().BreakBeforeParameter)) |
Daniel Jasper | 4c6e005 | 2013-08-27 14:24:43 +0000 | [diff] [blame] | 241 | return true; |
Daniel Jasper | 9697281 | 2014-01-05 12:38:10 +0000 | [diff] [blame] | 242 | |
| 243 | // The following could be precomputed as they do not depend on the state. |
| 244 | // However, as they should take effect only if the UnwrappedLine does not fit |
| 245 | // into the ColumnLimit, they are checked here in the ContinuationIndenter. |
Daniel Jasper | 3599567 | 2014-04-29 14:05:20 +0000 | [diff] [blame] | 246 | if (Style.ColumnLimit != 0 && Previous.BlockKind == BK_Block && |
| 247 | Previous.is(tok::l_brace) && !Current.isOneOf(tok::r_brace, tok::comment)) |
Daniel Jasper | 9697281 | 2014-01-05 12:38:10 +0000 | [diff] [blame] | 248 | return true; |
Daniel Jasper | 9697281 | 2014-01-05 12:38:10 +0000 | [diff] [blame] | 249 | |
Daniel Jasper | 6996312 | 2015-02-17 10:05:15 +0000 | [diff] [blame] | 250 | if (Current.is(tok::lessless) && Previous.is(tok::identifier) && |
| 251 | Previous.TokenText == "endl") |
| 252 | return true; |
| 253 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 254 | return false; |
| 255 | } |
| 256 | |
| 257 | unsigned ContinuationIndenter::addTokenToState(LineState &State, bool Newline, |
Daniel Jasper | 8de9ed0 | 2013-08-22 15:00:41 +0000 | [diff] [blame] | 258 | bool DryRun, |
| 259 | unsigned ExtraSpaces) { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 260 | const FormatToken &Current = *State.NextToken; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 261 | |
Manuel Klimek | 819788d | 2014-03-18 11:22:45 +0000 | [diff] [blame] | 262 | assert(!State.Stack.empty()); |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 263 | if ((Current.is(TT_ImplicitStringLiteral) && |
Craig Topper | 2145bc0 | 2014-05-09 08:15:10 +0000 | [diff] [blame] | 264 | (Current.Previous->Tok.getIdentifierInfo() == nullptr || |
Daniel Jasper | 9885784 | 2013-10-30 13:54:53 +0000 | [diff] [blame] | 265 | Current.Previous->Tok.getIdentifierInfo()->getPPKeywordID() == |
| 266 | tok::pp_not_keyword))) { |
Daniel Jasper | 30526e7 | 2015-01-31 07:05:46 +0000 | [diff] [blame] | 267 | unsigned EndColumn = |
| 268 | SourceMgr.getSpellingColumnNumber(Current.WhitespaceRange.getEnd()); |
| 269 | if (Current.LastNewlineOffset != 0) { |
| 270 | // If there is a newline within this token, the final column will solely |
| 271 | // determined by the current end column. |
| 272 | State.Column = EndColumn; |
| 273 | } else { |
| 274 | unsigned StartColumn = |
| 275 | SourceMgr.getSpellingColumnNumber(Current.WhitespaceRange.getBegin()); |
| 276 | assert(EndColumn >= StartColumn); |
| 277 | State.Column += EndColumn - StartColumn; |
| 278 | } |
Daniel Jasper | 240dfda | 2014-03-31 14:23:49 +0000 | [diff] [blame] | 279 | moveStateToNextToken(State, DryRun, /*Newline=*/false); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 280 | return 0; |
| 281 | } |
| 282 | |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 283 | unsigned Penalty = 0; |
| 284 | if (Newline) |
| 285 | Penalty = addTokenOnNewLine(State, DryRun); |
| 286 | else |
Daniel Jasper | 48437ce | 2013-11-20 14:54:39 +0000 | [diff] [blame] | 287 | addTokenOnCurrentLine(State, DryRun, ExtraSpaces); |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 288 | |
| 289 | return moveStateToNextToken(State, DryRun, Newline) + Penalty; |
| 290 | } |
| 291 | |
Daniel Jasper | 48437ce | 2013-11-20 14:54:39 +0000 | [diff] [blame] | 292 | void ContinuationIndenter::addTokenOnCurrentLine(LineState &State, bool DryRun, |
| 293 | unsigned ExtraSpaces) { |
Manuel Klimek | 71814b4 | 2013-10-11 21:25:45 +0000 | [diff] [blame] | 294 | FormatToken &Current = *State.NextToken; |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 295 | const FormatToken &Previous = *State.NextToken->Previous; |
| 296 | if (Current.is(tok::equal) && |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 297 | (State.Line->First->is(tok::kw_for) || Current.NestingLevel == 0) && |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 298 | State.Stack.back().VariablePos == 0) { |
| 299 | State.Stack.back().VariablePos = State.Column; |
| 300 | // Move over * and & if they are bound to the variable name. |
| 301 | const FormatToken *Tok = &Previous; |
| 302 | while (Tok && State.Stack.back().VariablePos >= Tok->ColumnWidth) { |
| 303 | State.Stack.back().VariablePos -= Tok->ColumnWidth; |
| 304 | if (Tok->SpacesRequiredBefore != 0) |
| 305 | break; |
| 306 | Tok = Tok->Previous; |
| 307 | } |
| 308 | if (Previous.PartOfMultiVariableDeclStmt) |
| 309 | State.Stack.back().LastSpace = State.Stack.back().VariablePos; |
| 310 | } |
| 311 | |
| 312 | unsigned Spaces = Current.SpacesRequiredBefore + ExtraSpaces; |
| 313 | |
| 314 | if (!DryRun) |
| 315 | Whitespaces.replaceWhitespace(Current, /*Newlines=*/0, /*IndentLevel=*/0, |
| 316 | Spaces, State.Column + Spaces); |
| 317 | |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 318 | if (Current.is(TT_SelectorName) && |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 319 | !State.Stack.back().ObjCSelectorNameFound) { |
Daniel Jasper | 06a2695 | 2016-01-04 07:29:07 +0000 | [diff] [blame^] | 320 | unsigned MinIndent = |
| 321 | std::max(State.FirstIndent + Style.ContinuationIndentWidth, |
| 322 | State.Stack.back().Indent); |
| 323 | unsigned FirstColonPos = State.Column + Spaces + Current.ColumnWidth; |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 324 | if (Current.LongestObjCSelectorName == 0) |
| 325 | State.Stack.back().AlignColons = false; |
Daniel Jasper | 06a2695 | 2016-01-04 07:29:07 +0000 | [diff] [blame^] | 326 | else if (MinIndent + Current.LongestObjCSelectorName > FirstColonPos) |
| 327 | State.Stack.back().ColonPos = MinIndent + Current.LongestObjCSelectorName; |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 328 | else |
Daniel Jasper | 06a2695 | 2016-01-04 07:29:07 +0000 | [diff] [blame^] | 329 | State.Stack.back().ColonPos = FirstColonPos; |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 330 | } |
| 331 | |
Daniel Jasper | 6501f7e | 2015-10-27 12:38:37 +0000 | [diff] [blame] | 332 | // In "AlwaysBreak" mode, enforce wrapping directly after the parenthesis by |
| 333 | // disallowing any further line breaks if there is no line break after the |
| 334 | // opening parenthesis. Don't break if it doesn't conserve columns. |
| 335 | if (Style.AlignAfterOpenBracket == FormatStyle::BAS_AlwaysBreak && |
| 336 | Previous.is(tok::l_paren) && State.Column > getNewLineColumn(State) && |
| 337 | (!Previous.Previous || |
| 338 | !Previous.Previous->isOneOf(tok::kw_for, tok::kw_while, tok::kw_switch))) |
| 339 | State.Stack.back().NoLineBreak = true; |
| 340 | |
| 341 | if (Style.AlignAfterOpenBracket != FormatStyle::BAS_DontAlign && |
| 342 | Previous.opensScope() && Previous.isNot(TT_ObjCMethodExpr) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 343 | (Current.isNot(TT_LineComment) || Previous.BlockKind == BK_BracedInit)) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 344 | State.Stack.back().Indent = State.Column + Spaces; |
Daniel Jasper | ec01cd6 | 2013-10-08 05:11:18 +0000 | [diff] [blame] | 345 | if (State.Stack.back().AvoidBinPacking && startsNextParameter(Current, Style)) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 346 | State.Stack.back().NoLineBreak = true; |
Daniel Jasper | 775954b | 2015-04-24 10:08:09 +0000 | [diff] [blame] | 347 | if (startsSegmentOfBuilderTypeCall(Current) && |
| 348 | State.Column > getNewLineColumn(State)) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 349 | State.Stack.back().ContainsUnwrappedBuilder = true; |
| 350 | |
Daniel Jasper | 6f2b88a | 2015-06-05 13:18:09 +0000 | [diff] [blame] | 351 | if (Current.is(TT_LambdaArrow) && Style.Language == FormatStyle::LK_Java) |
Daniel Jasper | 05cd929 | 2015-03-26 18:46:28 +0000 | [diff] [blame] | 352 | State.Stack.back().NoLineBreak = true; |
Daniel Jasper | d6f17d8 | 2014-09-12 16:35:28 +0000 | [diff] [blame] | 353 | if (Current.isMemberAccess() && Previous.is(tok::r_paren) && |
| 354 | (Previous.MatchingParen && |
| 355 | (Previous.TotalLength - Previous.MatchingParen->TotalLength > 10))) { |
| 356 | // If there is a function call with long parameters, break before trailing |
| 357 | // calls. This prevents things like: |
| 358 | // EXPECT_CALL(SomeLongParameter).Times( |
| 359 | // 2); |
| 360 | // We don't want to do this for short parameters as they can just be |
| 361 | // indexes. |
| 362 | State.Stack.back().NoLineBreak = true; |
| 363 | } |
| 364 | |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 365 | State.Column += Spaces; |
Daniel Jasper | 8acf822 | 2014-05-07 09:23:05 +0000 | [diff] [blame] | 366 | if (Current.isNot(tok::comment) && Previous.is(tok::l_paren) && |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 367 | Previous.Previous && |
| 368 | Previous.Previous->isOneOf(tok::kw_if, tok::kw_for)) { |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 369 | // Treat the condition inside an if as if it was a second function |
Daniel Jasper | 6633ab8 | 2013-10-18 10:38:14 +0000 | [diff] [blame] | 370 | // parameter, i.e. let nested calls have a continuation indent. |
Daniel Jasper | 8acf822 | 2014-05-07 09:23:05 +0000 | [diff] [blame] | 371 | State.Stack.back().LastSpace = State.Column; |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 372 | State.Stack.back().NestedBlockIndent = State.Column; |
| 373 | } else if (!Current.isOneOf(tok::comment, tok::caret) && |
| 374 | (Previous.is(tok::comma) || |
| 375 | (Previous.is(tok::colon) && Previous.is(TT_ObjCMethodExpr)))) { |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 376 | State.Stack.back().LastSpace = State.Column; |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 377 | } else if ((Previous.isOneOf(TT_BinaryOperator, TT_ConditionalExpr, |
| 378 | TT_CtorInitializerColon)) && |
| 379 | ((Previous.getPrecedence() != prec::Assignment && |
| 380 | (Previous.isNot(tok::lessless) || Previous.OperatorIndex != 0 || |
| 381 | !Previous.LastOperator)) || |
| 382 | Current.StartsBinaryExpression)) { |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 383 | // Always indent relative to the RHS of the expression unless this is a |
| 384 | // simple assignment without binary expression on the RHS. Also indent |
| 385 | // relative to unary operators and the colons of constructor initializers. |
| 386 | State.Stack.back().LastSpace = State.Column; |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 387 | } else if (Previous.is(TT_InheritanceColon)) { |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 388 | State.Stack.back().Indent = State.Column; |
Daniel Jasper | f9a5e40 | 2013-10-08 16:24:07 +0000 | [diff] [blame] | 389 | State.Stack.back().LastSpace = State.Column; |
| 390 | } else if (Previous.opensScope()) { |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 391 | // If a function has a trailing call, indent all parameters from the |
| 392 | // opening parenthesis. This avoids confusing indents like: |
| 393 | // OuterFunction(InnerFunctionCall( // break |
| 394 | // ParameterToInnerFunction)) // break |
| 395 | // .SecondInnerFunctionCall(); |
| 396 | bool HasTrailingCall = false; |
| 397 | if (Previous.MatchingParen) { |
| 398 | const FormatToken *Next = Previous.MatchingParen->getNextNonComment(); |
| 399 | HasTrailingCall = Next && Next->isMemberAccess(); |
| 400 | } |
Daniel Jasper | d97d5d5 | 2015-02-17 09:58:03 +0000 | [diff] [blame] | 401 | if (HasTrailingCall && State.Stack.size() > 1 && |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 402 | State.Stack[State.Stack.size() - 2].CallContinuation == 0) |
| 403 | State.Stack.back().LastSpace = State.Column; |
| 404 | } |
| 405 | } |
| 406 | |
| 407 | unsigned ContinuationIndenter::addTokenOnNewLine(LineState &State, |
| 408 | bool DryRun) { |
Manuel Klimek | 71814b4 | 2013-10-11 21:25:45 +0000 | [diff] [blame] | 409 | FormatToken &Current = *State.NextToken; |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 410 | const FormatToken &Previous = *State.NextToken->Previous; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 411 | |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 412 | // Extra penalty that needs to be added because of the way certain line |
| 413 | // breaks are chosen. |
| 414 | unsigned Penalty = 0; |
Daniel Jasper | 8de9ed0 | 2013-08-22 15:00:41 +0000 | [diff] [blame] | 415 | |
Daniel Jasper | a040774 | 2014-02-11 10:08:11 +0000 | [diff] [blame] | 416 | const FormatToken *PreviousNonComment = Current.getPreviousNonComment(); |
| 417 | const FormatToken *NextNonComment = Previous.getNextNonComment(); |
| 418 | if (!NextNonComment) |
| 419 | NextNonComment = &Current; |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 420 | // The first line break on any NestingLevel causes an extra penalty in order |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 421 | // prefer similar line breaks. |
| 422 | if (!State.Stack.back().ContainsLineBreak) |
| 423 | Penalty += 15; |
| 424 | State.Stack.back().ContainsLineBreak = true; |
Daniel Jasper | 8de9ed0 | 2013-08-22 15:00:41 +0000 | [diff] [blame] | 425 | |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 426 | Penalty += State.NextToken->SplitPenalty; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 427 | |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 428 | // Breaking before the first "<<" is generally not desirable if the LHS is |
Daniel Jasper | 48437ce | 2013-11-20 14:54:39 +0000 | [diff] [blame] | 429 | // short. Also always add the penalty if the LHS is split over mutliple lines |
Daniel Jasper | 2b7556e | 2014-04-03 12:00:27 +0000 | [diff] [blame] | 430 | // to avoid unnecessary line breaks that just work around this penalty. |
Daniel Jasper | a040774 | 2014-02-11 10:08:11 +0000 | [diff] [blame] | 431 | if (NextNonComment->is(tok::lessless) && |
| 432 | State.Stack.back().FirstLessLess == 0 && |
Daniel Jasper | 004177e | 2013-12-19 16:06:40 +0000 | [diff] [blame] | 433 | (State.Column <= Style.ColumnLimit / 3 || |
Daniel Jasper | 48437ce | 2013-11-20 14:54:39 +0000 | [diff] [blame] | 434 | State.Stack.back().BreakBeforeParameter)) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 435 | Penalty += Style.PenaltyBreakFirstLessLess; |
| 436 | |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 437 | State.Column = getNewLineColumn(State); |
Daniel Jasper | ed3f395 | 2015-06-18 12:32:59 +0000 | [diff] [blame] | 438 | |
| 439 | // Indent nested blocks relative to this column, unless in a very specific |
| 440 | // JavaScript special case where: |
| 441 | // |
| 442 | // var loooooong_name = |
| 443 | // function() { |
| 444 | // // code |
| 445 | // } |
| 446 | // |
| 447 | // is common and should be formatted like a free-standing function. |
| 448 | if (Style.Language != FormatStyle::LK_JavaScript || |
| 449 | Current.NestingLevel != 0 || !PreviousNonComment->is(tok::equal) || |
| 450 | !Current.is(Keywords.kw_function)) |
| 451 | State.Stack.back().NestedBlockIndent = State.Column; |
| 452 | |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 453 | if (NextNonComment->isMemberAccess()) { |
| 454 | if (State.Stack.back().CallContinuation == 0) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 455 | State.Stack.back().CallContinuation = State.Column; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 456 | } else if (NextNonComment->is(TT_SelectorName)) { |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 457 | if (!State.Stack.back().ObjCSelectorNameFound) { |
Daniel Jasper | a040774 | 2014-02-11 10:08:11 +0000 | [diff] [blame] | 458 | if (NextNonComment->LongestObjCSelectorName == 0) { |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 459 | State.Stack.back().AlignColons = false; |
| 460 | } else { |
| 461 | State.Stack.back().ColonPos = |
Daniel Jasper | a2a4d9c | 2015-05-13 09:38:25 +0000 | [diff] [blame] | 462 | (Style.IndentWrappedFunctionNames |
| 463 | ? std::max(State.Stack.back().Indent, |
| 464 | State.FirstIndent + Style.ContinuationIndentWidth) |
| 465 | : State.Stack.back().Indent) + |
| 466 | NextNonComment->LongestObjCSelectorName; |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 467 | } |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 468 | } else if (State.Stack.back().AlignColons && |
| 469 | State.Stack.back().ColonPos <= NextNonComment->ColumnWidth) { |
Daniel Jasper | a040774 | 2014-02-11 10:08:11 +0000 | [diff] [blame] | 470 | State.Stack.back().ColonPos = State.Column + NextNonComment->ColumnWidth; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 471 | } |
Daniel Jasper | 1fd6f1f | 2014-03-17 14:32:47 +0000 | [diff] [blame] | 472 | } else if (PreviousNonComment && PreviousNonComment->is(tok::colon) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 473 | PreviousNonComment->isOneOf(TT_ObjCMethodExpr, TT_DictLiteral)) { |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 474 | // FIXME: This is hacky, find a better way. The problem is that in an ObjC |
| 475 | // method expression, the block should be aligned to the line starting it, |
| 476 | // e.g.: |
| 477 | // [aaaaaaaaaaaaaaa aaaaaaaaa: \\ break for some reason |
| 478 | // ^(int *i) { |
| 479 | // // ... |
| 480 | // }]; |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 481 | // Thus, we set LastSpace of the next higher NestingLevel, to which we move |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 482 | // when we consume all of the "}"'s FakeRParens at the "{". |
Daniel Jasper | 9a26e77 | 2013-12-23 11:25:40 +0000 | [diff] [blame] | 483 | if (State.Stack.size() > 1) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 484 | State.Stack[State.Stack.size() - 2].LastSpace = |
| 485 | std::max(State.Stack.back().LastSpace, State.Stack.back().Indent) + |
| 486 | Style.ContinuationIndentWidth; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 487 | } |
| 488 | |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 489 | if ((Previous.isOneOf(tok::comma, tok::semi) && |
| 490 | !State.Stack.back().AvoidBinPacking) || |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 491 | Previous.is(TT_BinaryOperator)) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 492 | State.Stack.back().BreakBeforeParameter = false; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 493 | if (Previous.isOneOf(TT_TemplateCloser, TT_JavaAnnotation) && |
Daniel Jasper | 39af6cd | 2014-11-03 02:27:28 +0000 | [diff] [blame] | 494 | Current.NestingLevel == 0) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 495 | State.Stack.back().BreakBeforeParameter = false; |
Daniel Jasper | a040774 | 2014-02-11 10:08:11 +0000 | [diff] [blame] | 496 | if (NextNonComment->is(tok::question) || |
Daniel Jasper | 165b29e | 2013-11-08 00:57:11 +0000 | [diff] [blame] | 497 | (PreviousNonComment && PreviousNonComment->is(tok::question))) |
| 498 | State.Stack.back().BreakBeforeParameter = true; |
Daniel Jasper | 5962fa8 | 2015-06-03 09:26:03 +0000 | [diff] [blame] | 499 | if (Current.is(TT_BinaryOperator) && Current.CanBreakBefore) |
| 500 | State.Stack.back().BreakBeforeParameter = false; |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 501 | |
| 502 | if (!DryRun) { |
Daniel Jasper | a69ca9b | 2014-06-04 12:40:57 +0000 | [diff] [blame] | 503 | unsigned Newlines = std::max( |
| 504 | 1u, std::min(Current.NewlinesBefore, Style.MaxEmptyLinesToKeep + 1)); |
Alexander Kornienko | e2e0387 | 2013-10-14 00:46:35 +0000 | [diff] [blame] | 505 | Whitespaces.replaceWhitespace(Current, Newlines, |
| 506 | State.Stack.back().IndentLevel, State.Column, |
| 507 | State.Column, State.Line->InPPDirective); |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 508 | } |
| 509 | |
| 510 | if (!Current.isTrailingComment()) |
| 511 | State.Stack.back().LastSpace = State.Column; |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 512 | State.StartOfLineLevel = Current.NestingLevel; |
| 513 | State.LowestLevelOnLine = Current.NestingLevel; |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 514 | |
| 515 | // Any break on this level means that the parent level has been broken |
| 516 | // and we need to avoid bin packing there. |
Daniel Jasper | 4b44449 | 2014-11-21 13:38:53 +0000 | [diff] [blame] | 517 | bool NestedBlockSpecialCase = |
Daniel Jasper | 100ffc6 | 2015-08-21 11:44:57 +0000 | [diff] [blame] | 518 | Style.Language != FormatStyle::LK_Cpp && |
Daniel Jasper | 4b44449 | 2014-11-21 13:38:53 +0000 | [diff] [blame] | 519 | Current.is(tok::r_brace) && State.Stack.size() > 1 && |
| 520 | State.Stack[State.Stack.size() - 2].NestedBlockInlined; |
Daniel Jasper | 1699eca | 2015-06-01 09:56:32 +0000 | [diff] [blame] | 521 | if (!NestedBlockSpecialCase) |
| 522 | for (unsigned i = 0, e = State.Stack.size() - 1; i != e; ++i) |
Daniel Jasper | b16b969 | 2014-05-21 12:51:23 +0000 | [diff] [blame] | 523 | State.Stack[i].BreakBeforeParameter = true; |
Daniel Jasper | b16b969 | 2014-05-21 12:51:23 +0000 | [diff] [blame] | 524 | |
Daniel Jasper | 9e5ede0 | 2013-11-08 19:56:28 +0000 | [diff] [blame] | 525 | if (PreviousNonComment && |
| 526 | !PreviousNonComment->isOneOf(tok::comma, tok::semi) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 527 | (PreviousNonComment->isNot(TT_TemplateCloser) || |
Daniel Jasper | 6c0ee17 | 2014-11-14 13:14:45 +0000 | [diff] [blame] | 528 | Current.NestingLevel != 0) && |
Daniel Jasper | 47bbda0 | 2015-05-18 13:47:23 +0000 | [diff] [blame] | 529 | !PreviousNonComment->isOneOf( |
| 530 | TT_BinaryOperator, TT_FunctionAnnotationRParen, TT_JavaAnnotation, |
| 531 | TT_LeadingJavaAnnotation) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 532 | Current.isNot(TT_BinaryOperator) && !PreviousNonComment->opensScope()) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 533 | State.Stack.back().BreakBeforeParameter = true; |
| 534 | |
Daniel Jasper | 1db6c38 | 2013-10-22 15:30:28 +0000 | [diff] [blame] | 535 | // If we break after { or the [ of an array initializer, we should also break |
| 536 | // before the corresponding } or ]. |
Daniel Jasper | 9081805 | 2014-06-10 10:42:26 +0000 | [diff] [blame] | 537 | if (PreviousNonComment && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 538 | (PreviousNonComment->isOneOf(tok::l_brace, TT_ArrayInitializerLSquare))) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 539 | State.Stack.back().BreakBeforeClosingBrace = true; |
| 540 | |
| 541 | if (State.Stack.back().AvoidBinPacking) { |
| 542 | // If we are breaking after '(', '{', '<', this is not bin packing |
Daniel Jasper | 2a95832 | 2014-05-21 13:26:58 +0000 | [diff] [blame] | 543 | // unless AllowAllParametersOfDeclarationOnNextLine is false or this is a |
| 544 | // dict/object literal. |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 545 | if (!Previous.isOneOf(tok::l_paren, tok::l_brace, TT_BinaryOperator) || |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 546 | (!Style.AllowAllParametersOfDeclarationOnNextLine && |
Daniel Jasper | 2a95832 | 2014-05-21 13:26:58 +0000 | [diff] [blame] | 547 | State.Line->MustBeDeclaration) || |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 548 | Previous.is(TT_DictLiteral)) |
Alexander Kornienko | 1f80396 | 2013-10-01 14:41:18 +0000 | [diff] [blame] | 549 | State.Stack.back().BreakBeforeParameter = true; |
| 550 | } |
| 551 | |
| 552 | return Penalty; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 553 | } |
| 554 | |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 555 | unsigned ContinuationIndenter::getNewLineColumn(const LineState &State) { |
Daniel Jasper | 5d2587d | 2014-03-27 16:14:13 +0000 | [diff] [blame] | 556 | if (!State.NextToken || !State.NextToken->Previous) |
| 557 | return 0; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 558 | FormatToken &Current = *State.NextToken; |
Daniel Jasper | 4281c5a | 2014-10-07 14:45:34 +0000 | [diff] [blame] | 559 | const FormatToken &Previous = *Current.Previous; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 560 | // If we are continuing an expression, we want to use the continuation indent. |
| 561 | unsigned ContinuationIndent = |
| 562 | std::max(State.Stack.back().LastSpace, State.Stack.back().Indent) + |
| 563 | Style.ContinuationIndentWidth; |
| 564 | const FormatToken *PreviousNonComment = Current.getPreviousNonComment(); |
| 565 | const FormatToken *NextNonComment = Previous.getNextNonComment(); |
| 566 | if (!NextNonComment) |
| 567 | NextNonComment = &Current; |
Daniel Jasper | 50b4bd7 | 2014-11-02 19:16:41 +0000 | [diff] [blame] | 568 | |
| 569 | // Java specific bits. |
Daniel Jasper | d0ec0d6 | 2014-11-04 12:41:02 +0000 | [diff] [blame] | 570 | if (Style.Language == FormatStyle::LK_Java && |
| 571 | Current.isOneOf(Keywords.kw_implements, Keywords.kw_extends)) |
Daniel Jasper | 50b4bd7 | 2014-11-02 19:16:41 +0000 | [diff] [blame] | 572 | return std::max(State.Stack.back().LastSpace, |
| 573 | State.Stack.back().Indent + Style.ContinuationIndentWidth); |
| 574 | |
Daniel Jasper | b05a81d | 2014-05-09 13:11:16 +0000 | [diff] [blame] | 575 | if (NextNonComment->is(tok::l_brace) && NextNonComment->BlockKind == BK_Block) |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 576 | return Current.NestingLevel == 0 ? State.FirstIndent |
| 577 | : State.Stack.back().Indent; |
Daniel Jasper | 675b4f8 | 2015-01-19 10:51:23 +0000 | [diff] [blame] | 578 | if (Current.isOneOf(tok::r_brace, tok::r_square) && State.Stack.size() > 1) { |
Daniel Jasper | bd73bcf | 2015-10-27 13:42:08 +0000 | [diff] [blame] | 579 | if (Current.closesBlockOrBlockTypeList(Style)) |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 580 | return State.Stack[State.Stack.size() - 2].NestedBlockIndent; |
| 581 | if (Current.MatchingParen && |
| 582 | Current.MatchingParen->BlockKind == BK_BracedInit) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 583 | return State.Stack[State.Stack.size() - 2].LastSpace; |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 584 | return State.FirstIndent; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 585 | } |
Daniel Jasper | 783bac6 | 2014-04-15 09:54:30 +0000 | [diff] [blame] | 586 | if (Current.is(tok::identifier) && Current.Next && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 587 | Current.Next->is(TT_DictLiteral)) |
Daniel Jasper | 783bac6 | 2014-04-15 09:54:30 +0000 | [diff] [blame] | 588 | return State.Stack.back().Indent; |
Daniel Jasper | 0928553 | 2015-05-17 08:13:23 +0000 | [diff] [blame] | 589 | if (NextNonComment->isStringLiteral() && State.StartOfStringLiteral != 0) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 590 | return State.StartOfStringLiteral; |
Daniel Jasper | 0928553 | 2015-05-17 08:13:23 +0000 | [diff] [blame] | 591 | if (NextNonComment->is(TT_ObjCStringLiteral) && |
| 592 | State.StartOfStringLiteral != 0) |
| 593 | return State.StartOfStringLiteral - 1; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 594 | if (NextNonComment->is(tok::lessless) && |
| 595 | State.Stack.back().FirstLessLess != 0) |
| 596 | return State.Stack.back().FirstLessLess; |
| 597 | if (NextNonComment->isMemberAccess()) { |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 598 | if (State.Stack.back().CallContinuation == 0) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 599 | return ContinuationIndent; |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 600 | return State.Stack.back().CallContinuation; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 601 | } |
| 602 | if (State.Stack.back().QuestionColumn != 0 && |
Daniel Jasper | c0d606a | 2014-04-14 11:08:45 +0000 | [diff] [blame] | 603 | ((NextNonComment->is(tok::colon) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 604 | NextNonComment->is(TT_ConditionalExpr)) || |
| 605 | Previous.is(TT_ConditionalExpr))) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 606 | return State.Stack.back().QuestionColumn; |
| 607 | if (Previous.is(tok::comma) && State.Stack.back().VariablePos != 0) |
| 608 | return State.Stack.back().VariablePos; |
Daniel Jasper | e9ab42d | 2014-10-31 18:23:49 +0000 | [diff] [blame] | 609 | if ((PreviousNonComment && |
| 610 | (PreviousNonComment->ClosesTemplateDeclaration || |
Daniel Jasper | 47bbda0 | 2015-05-18 13:47:23 +0000 | [diff] [blame] | 611 | PreviousNonComment->isOneOf( |
| 612 | TT_AttributeParen, TT_FunctionAnnotationRParen, TT_JavaAnnotation, |
| 613 | TT_LeadingJavaAnnotation))) || |
Daniel Jasper | c75e1ef | 2014-07-09 08:42:42 +0000 | [diff] [blame] | 614 | (!Style.IndentWrappedFunctionNames && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 615 | NextNonComment->isOneOf(tok::kw_operator, TT_FunctionDeclarationName))) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 616 | return std::max(State.Stack.back().LastSpace, State.Stack.back().Indent); |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 617 | if (NextNonComment->is(TT_SelectorName)) { |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 618 | if (!State.Stack.back().ObjCSelectorNameFound) { |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 619 | if (NextNonComment->LongestObjCSelectorName == 0) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 620 | return State.Stack.back().Indent; |
Daniel Jasper | a2a4d9c | 2015-05-13 09:38:25 +0000 | [diff] [blame] | 621 | return (Style.IndentWrappedFunctionNames |
| 622 | ? std::max(State.Stack.back().Indent, |
| 623 | State.FirstIndent + Style.ContinuationIndentWidth) |
| 624 | : State.Stack.back().Indent) + |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 625 | NextNonComment->LongestObjCSelectorName - |
| 626 | NextNonComment->ColumnWidth; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 627 | } |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 628 | if (!State.Stack.back().AlignColons) |
| 629 | return State.Stack.back().Indent; |
| 630 | if (State.Stack.back().ColonPos > NextNonComment->ColumnWidth) |
| 631 | return State.Stack.back().ColonPos - NextNonComment->ColumnWidth; |
| 632 | return State.Stack.back().Indent; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 633 | } |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 634 | if (NextNonComment->is(TT_ArraySubscriptLSquare)) { |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 635 | if (State.Stack.back().StartOfArraySubscripts != 0) |
| 636 | return State.Stack.back().StartOfArraySubscripts; |
Daniel Jasper | 24a1477 | 2014-12-10 17:24:34 +0000 | [diff] [blame] | 637 | return ContinuationIndent; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 638 | } |
Daniel Jasper | 9c95013 | 2015-05-07 14:19:59 +0000 | [diff] [blame] | 639 | |
| 640 | // This ensure that we correctly format ObjC methods calls without inputs, |
| 641 | // i.e. where the last element isn't selector like: [callee method]; |
| 642 | if (NextNonComment->is(tok::identifier) && NextNonComment->FakeRParens == 0 && |
| 643 | NextNonComment->Next && NextNonComment->Next->is(TT_ObjCMethodExpr)) |
Daniel Jasper | eb53668 | 2015-05-06 12:48:06 +0000 | [diff] [blame] | 644 | return State.Stack.back().Indent; |
Daniel Jasper | 9c95013 | 2015-05-07 14:19:59 +0000 | [diff] [blame] | 645 | |
Daniel Jasper | b754a74 | 2015-03-12 15:04:53 +0000 | [diff] [blame] | 646 | if (NextNonComment->isOneOf(TT_StartOfName, TT_PointerOrReference) || |
Daniel Jasper | b2328b1 | 2015-07-06 14:07:51 +0000 | [diff] [blame] | 647 | Previous.isOneOf(tok::coloncolon, tok::equal, TT_JsTypeColon)) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 648 | return ContinuationIndent; |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 649 | if (PreviousNonComment && PreviousNonComment->is(tok::colon) && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 650 | PreviousNonComment->isOneOf(TT_ObjCMethodExpr, TT_DictLiteral)) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 651 | return ContinuationIndent; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 652 | if (NextNonComment->is(TT_CtorInitializerColon)) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 653 | return State.FirstIndent + Style.ConstructorInitializerIndentWidth; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 654 | if (NextNonComment->is(TT_CtorInitializerComma)) |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 655 | return State.Stack.back().Indent; |
Daniel Jasper | 316ab38 | 2014-08-06 13:14:58 +0000 | [diff] [blame] | 656 | if (Previous.is(tok::r_paren) && !Current.isBinaryOperator() && |
Daniel Jasper | 119ff53 | 2014-11-14 12:31:14 +0000 | [diff] [blame] | 657 | !Current.isOneOf(tok::colon, tok::comment)) |
Daniel Jasper | 316ab38 | 2014-08-06 13:14:58 +0000 | [diff] [blame] | 658 | return ContinuationIndent; |
Daniel Jasper | 5d2587d | 2014-03-27 16:14:13 +0000 | [diff] [blame] | 659 | if (State.Stack.back().Indent == State.FirstIndent && PreviousNonComment && |
Daniel Jasper | 9f388d0 | 2014-03-27 14:33:30 +0000 | [diff] [blame] | 660 | PreviousNonComment->isNot(tok::r_brace)) |
| 661 | // Ensure that we fall back to the continuation indent width instead of |
| 662 | // just flushing continuations left. |
| 663 | return State.Stack.back().Indent + Style.ContinuationIndentWidth; |
| 664 | return State.Stack.back().Indent; |
| 665 | } |
| 666 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 667 | unsigned ContinuationIndenter::moveStateToNextToken(LineState &State, |
| 668 | bool DryRun, bool Newline) { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 669 | assert(State.Stack.size()); |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 670 | const FormatToken &Current = *State.NextToken; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 671 | |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 672 | if (Current.is(TT_InheritanceColon)) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 673 | State.Stack.back().AvoidBinPacking = true; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 674 | if (Current.is(tok::lessless) && Current.isNot(TT_OverloadedOperator)) { |
Daniel Jasper | c0d606a | 2014-04-14 11:08:45 +0000 | [diff] [blame] | 675 | if (State.Stack.back().FirstLessLess == 0) |
| 676 | State.Stack.back().FirstLessLess = State.Column; |
| 677 | else |
| 678 | State.Stack.back().LastOperatorWrapped = Newline; |
| 679 | } |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 680 | if ((Current.is(TT_BinaryOperator) && Current.isNot(tok::lessless)) || |
| 681 | Current.is(TT_ConditionalExpr)) |
Daniel Jasper | c0d606a | 2014-04-14 11:08:45 +0000 | [diff] [blame] | 682 | State.Stack.back().LastOperatorWrapped = Newline; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 683 | if (Current.is(TT_ArraySubscriptLSquare) && |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 684 | State.Stack.back().StartOfArraySubscripts == 0) |
| 685 | State.Stack.back().StartOfArraySubscripts = State.Column; |
Daniel Jasper | 165b29e | 2013-11-08 00:57:11 +0000 | [diff] [blame] | 686 | if ((Current.is(tok::question) && Style.BreakBeforeTernaryOperators) || |
| 687 | (Current.getPreviousNonComment() && Current.isNot(tok::colon) && |
| 688 | Current.getPreviousNonComment()->is(tok::question) && |
| 689 | !Style.BreakBeforeTernaryOperators)) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 690 | State.Stack.back().QuestionColumn = State.Column; |
| 691 | if (!Current.opensScope() && !Current.closesScope()) |
| 692 | State.LowestLevelOnLine = |
Daniel Jasper | 05cd586 | 2014-05-08 12:21:30 +0000 | [diff] [blame] | 693 | std::min(State.LowestLevelOnLine, Current.NestingLevel); |
Daniel Jasper | 4c6e005 | 2013-08-27 14:24:43 +0000 | [diff] [blame] | 694 | if (Current.isMemberAccess()) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 695 | State.Stack.back().StartOfFunctionCall = |
Daniel Jasper | 9dedc775 | 2015-04-07 06:41:24 +0000 | [diff] [blame] | 696 | Current.LastOperator ? 0 : State.Column; |
Daniel Jasper | 3c44c22 | 2015-07-16 22:58:24 +0000 | [diff] [blame] | 697 | if (Current.is(TT_SelectorName)) { |
Daniel Jasper | b88b25f | 2013-12-23 07:29:06 +0000 | [diff] [blame] | 698 | State.Stack.back().ObjCSelectorNameFound = true; |
Daniel Jasper | 3c44c22 | 2015-07-16 22:58:24 +0000 | [diff] [blame] | 699 | if (Style.IndentWrappedFunctionNames) { |
| 700 | State.Stack.back().Indent = |
| 701 | State.FirstIndent + Style.ContinuationIndentWidth; |
| 702 | } |
| 703 | } |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 704 | if (Current.is(TT_CtorInitializerColon)) { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 705 | // Indent 2 from the column, so: |
| 706 | // SomeClass::SomeClass() |
| 707 | // : First(...), ... |
| 708 | // Next(...) |
| 709 | // ^ line up here. |
| 710 | State.Stack.back().Indent = |
| 711 | State.Column + (Style.BreakConstructorInitializersBeforeComma ? 0 : 2); |
Daniel Jasper | d6a1cab | 2015-01-12 10:23:24 +0000 | [diff] [blame] | 712 | State.Stack.back().NestedBlockIndent = State.Stack.back().Indent; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 713 | if (Style.ConstructorInitializerAllOnOneLineOrOnePerLine) |
| 714 | State.Stack.back().AvoidBinPacking = true; |
| 715 | State.Stack.back().BreakBeforeParameter = false; |
| 716 | } |
Daniel Jasper | c4144ea | 2015-05-13 16:09:21 +0000 | [diff] [blame] | 717 | if (Current.isOneOf(TT_BinaryOperator, TT_ConditionalExpr) && Newline) |
| 718 | State.Stack.back().NestedBlockIndent = |
| 719 | State.Column + Current.ColumnWidth + 1; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 720 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 721 | // Insert scopes created by fake parenthesis. |
| 722 | const FormatToken *Previous = Current.getPreviousNonComment(); |
Daniel Jasper | b16b969 | 2014-05-21 12:51:23 +0000 | [diff] [blame] | 723 | |
| 724 | // Add special behavior to support a format commonly used for JavaScript |
| 725 | // closures: |
| 726 | // SomeFunction(function() { |
| 727 | // foo(); |
| 728 | // bar(); |
| 729 | // }, a, b, c); |
Daniel Jasper | b2ad4d4 | 2015-06-15 09:23:17 +0000 | [diff] [blame] | 730 | if (Current.isNot(tok::comment) && Previous && |
| 731 | Previous->isOneOf(tok::l_brace, TT_ArrayInitializerLSquare) && |
Daniel Jasper | ccff4d1 | 2016-01-04 07:27:33 +0000 | [diff] [blame] | 732 | !Previous->is(TT_DictLiteral) && State.Stack.size() > 1) { |
Daniel Jasper | 1699eca | 2015-06-01 09:56:32 +0000 | [diff] [blame] | 733 | if (State.Stack[State.Stack.size() - 2].NestedBlockInlined && Newline) |
| 734 | for (unsigned i = 0, e = State.Stack.size() - 1; i != e; ++i) |
Daniel Jasper | 4b44449 | 2014-11-21 13:38:53 +0000 | [diff] [blame] | 735 | State.Stack[i].NoLineBreak = true; |
Daniel Jasper | 4b44449 | 2014-11-21 13:38:53 +0000 | [diff] [blame] | 736 | State.Stack[State.Stack.size() - 2].NestedBlockInlined = false; |
| 737 | } |
| 738 | if (Previous && (Previous->isOneOf(tok::l_paren, tok::comma, tok::colon) || |
| 739 | Previous->isOneOf(TT_BinaryOperator, TT_ConditionalExpr)) && |
| 740 | !Previous->isOneOf(TT_DictLiteral, TT_ObjCMethodExpr)) { |
| 741 | State.Stack.back().NestedBlockInlined = |
| 742 | !Newline && |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 743 | (Previous->isNot(tok::l_paren) || Previous->ParameterCount > 1); |
Daniel Jasper | b16b969 | 2014-05-21 12:51:23 +0000 | [diff] [blame] | 744 | } |
| 745 | |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 746 | moveStatePastFakeLParens(State, Newline); |
| 747 | moveStatePastScopeOpener(State, Newline); |
| 748 | moveStatePastScopeCloser(State); |
| 749 | moveStatePastFakeRParens(State); |
| 750 | |
Daniel Jasper | 0928553 | 2015-05-17 08:13:23 +0000 | [diff] [blame] | 751 | if (Current.isStringLiteral() && State.StartOfStringLiteral == 0) |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 752 | State.StartOfStringLiteral = State.Column; |
Daniel Jasper | 0928553 | 2015-05-17 08:13:23 +0000 | [diff] [blame] | 753 | if (Current.is(TT_ObjCStringLiteral) && State.StartOfStringLiteral == 0) |
| 754 | State.StartOfStringLiteral = State.Column + 1; |
| 755 | else if (!Current.isOneOf(tok::comment, tok::identifier, tok::hash) && |
Daniel Jasper | e6fcf7d | 2015-06-17 13:08:06 +0000 | [diff] [blame] | 756 | !Current.isStringLiteral()) |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 757 | State.StartOfStringLiteral = 0; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 758 | |
| 759 | State.Column += Current.ColumnWidth; |
| 760 | State.NextToken = State.NextToken->Next; |
| 761 | unsigned Penalty = breakProtrudingToken(Current, State, DryRun); |
| 762 | if (State.Column > getColumnLimit(State)) { |
| 763 | unsigned ExcessCharacters = State.Column - getColumnLimit(State); |
| 764 | Penalty += Style.PenaltyExcessCharacter * ExcessCharacters; |
| 765 | } |
| 766 | |
| 767 | if (Current.Role) |
| 768 | Current.Role->formatFromToken(State, this, DryRun); |
| 769 | // If the previous has a special role, let it consume tokens as appropriate. |
| 770 | // It is necessary to start at the previous token for the only implemented |
| 771 | // role (comma separated list). That way, the decision whether or not to break |
| 772 | // after the "{" is already done and both options are tried and evaluated. |
| 773 | // FIXME: This is ugly, find a better way. |
| 774 | if (Previous && Previous->Role) |
| 775 | Penalty += Previous->Role->formatAfterToken(State, this, DryRun); |
| 776 | |
| 777 | return Penalty; |
| 778 | } |
| 779 | |
| 780 | void ContinuationIndenter::moveStatePastFakeLParens(LineState &State, |
| 781 | bool Newline) { |
| 782 | const FormatToken &Current = *State.NextToken; |
| 783 | const FormatToken *Previous = Current.getPreviousNonComment(); |
| 784 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 785 | // Don't add extra indentation for the first fake parenthesis after |
Dinesh Dwivedi | 0db806b | 2014-05-01 17:19:34 +0000 | [diff] [blame] | 786 | // 'return', assignments or opening <({[. The indentation for these cases |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 787 | // is special cased. |
| 788 | bool SkipFirstExtraIndent = |
Daniel Jasper | 98f8ae3 | 2015-03-06 10:57:12 +0000 | [diff] [blame] | 789 | (Previous && (Previous->opensScope() || |
| 790 | Previous->isOneOf(tok::semi, tok::kw_return) || |
Daniel Jasper | 3219e43 | 2014-12-02 13:24:51 +0000 | [diff] [blame] | 791 | (Previous->getPrecedence() == prec::Assignment && |
| 792 | Style.AlignOperands) || |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 793 | Previous->is(TT_ObjCMethodExpr))); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 794 | for (SmallVectorImpl<prec::Level>::const_reverse_iterator |
| 795 | I = Current.FakeLParens.rbegin(), |
| 796 | E = Current.FakeLParens.rend(); |
| 797 | I != E; ++I) { |
| 798 | ParenState NewParenState = State.Stack.back(); |
| 799 | NewParenState.ContainsLineBreak = false; |
Daniel Jasper | eabede6 | 2013-09-30 08:29:03 +0000 | [diff] [blame] | 800 | |
Daniel Jasper | 3aa9a6a | 2014-11-18 23:55:27 +0000 | [diff] [blame] | 801 | // Indent from 'LastSpace' unless these are fake parentheses encapsulating |
| 802 | // a builder type call after 'return' or, if the alignment after opening |
| 803 | // brackets is disabled. |
Daniel Jasper | 4281c5a | 2014-10-07 14:45:34 +0000 | [diff] [blame] | 804 | if (!Current.isTrailingComment() && |
Daniel Jasper | 3219e43 | 2014-12-02 13:24:51 +0000 | [diff] [blame] | 805 | (Style.AlignOperands || *I < prec::Assignment) && |
Daniel Jasper | 6cab678 | 2014-11-20 09:54:49 +0000 | [diff] [blame] | 806 | (!Previous || Previous->isNot(tok::kw_return) || |
| 807 | (Style.Language != FormatStyle::LK_Java && *I > 0)) && |
Daniel Jasper | 6501f7e | 2015-10-27 12:38:37 +0000 | [diff] [blame] | 808 | (Style.AlignAfterOpenBracket != FormatStyle::BAS_DontAlign || |
| 809 | *I != prec::Comma || Current.NestingLevel == 0)) |
Daniel Jasper | eabede6 | 2013-09-30 08:29:03 +0000 | [diff] [blame] | 810 | NewParenState.Indent = |
| 811 | std::max(std::max(State.Column, NewParenState.Indent), |
| 812 | State.Stack.back().LastSpace); |
| 813 | |
Daniel Jasper | 5c33265 | 2014-04-03 12:00:33 +0000 | [diff] [blame] | 814 | // Don't allow the RHS of an operator to be split over multiple lines unless |
| 815 | // there is a line-break right after the operator. |
| 816 | // Exclude relational operators, as there, it is always more desirable to |
| 817 | // have the LHS 'left' of the RHS. |
Daniel Jasper | c0d606a | 2014-04-14 11:08:45 +0000 | [diff] [blame] | 818 | if (Previous && Previous->getPrecedence() > prec::Assignment && |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 819 | Previous->isOneOf(TT_BinaryOperator, TT_ConditionalExpr) && |
Daniel Jasper | c0d606a | 2014-04-14 11:08:45 +0000 | [diff] [blame] | 820 | Previous->getPrecedence() != prec::Relational) { |
Daniel Jasper | 8c6e9ef | 2014-12-02 09:46:56 +0000 | [diff] [blame] | 821 | bool BreakBeforeOperator = |
| 822 | Previous->is(tok::lessless) || |
| 823 | (Previous->is(TT_BinaryOperator) && |
| 824 | Style.BreakBeforeBinaryOperators != FormatStyle::BOS_None) || |
| 825 | (Previous->is(TT_ConditionalExpr) && |
| 826 | Style.BreakBeforeTernaryOperators); |
Daniel Jasper | c0d606a | 2014-04-14 11:08:45 +0000 | [diff] [blame] | 827 | if ((!Newline && !BreakBeforeOperator) || |
| 828 | (!State.Stack.back().LastOperatorWrapped && BreakBeforeOperator)) |
| 829 | NewParenState.NoLineBreak = true; |
| 830 | } |
Daniel Jasper | 5c33265 | 2014-04-03 12:00:33 +0000 | [diff] [blame] | 831 | |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 832 | // Do not indent relative to the fake parentheses inserted for "." or "->". |
| 833 | // This is a special case to make the following to statements consistent: |
| 834 | // OuterFunction(InnerFunctionCall( // break |
| 835 | // ParameterToInnerFunction)); |
| 836 | // OuterFunction(SomeObject.InnerFunctionCall( // break |
| 837 | // ParameterToInnerFunction)); |
| 838 | if (*I > prec::Unknown) |
| 839 | NewParenState.LastSpace = std::max(NewParenState.LastSpace, State.Column); |
Daniel Jasper | 739b85f | 2015-06-29 10:42:59 +0000 | [diff] [blame] | 840 | if (*I != prec::Conditional && !Current.is(TT_UnaryOperator)) |
Daniel Jasper | a536df4 | 2014-12-08 21:28:31 +0000 | [diff] [blame] | 841 | NewParenState.StartOfFunctionCall = State.Column; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 842 | |
| 843 | // Always indent conditional expressions. Never indent expression where |
| 844 | // the 'operator' is ',', ';' or an assignment (i.e. *I <= |
| 845 | // prec::Assignment) as those have different indentation rules. Indent |
| 846 | // other expression, unless the indentation needs to be skipped. |
| 847 | if (*I == prec::Conditional || |
| 848 | (!SkipFirstExtraIndent && *I > prec::Assignment && |
Daniel Jasper | 8c6e9ef | 2014-12-02 09:46:56 +0000 | [diff] [blame] | 849 | !Current.isTrailingComment())) |
Daniel Jasper | 6633ab8 | 2013-10-18 10:38:14 +0000 | [diff] [blame] | 850 | NewParenState.Indent += Style.ContinuationIndentWidth; |
Daniel Jasper | 1db6c38 | 2013-10-22 15:30:28 +0000 | [diff] [blame] | 851 | if ((Previous && !Previous->opensScope()) || *I > prec::Comma) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 852 | NewParenState.BreakBeforeParameter = false; |
| 853 | State.Stack.push_back(NewParenState); |
| 854 | SkipFirstExtraIndent = false; |
| 855 | } |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 856 | } |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 857 | |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 858 | void ContinuationIndenter::moveStatePastFakeRParens(LineState &State) { |
| 859 | for (unsigned i = 0, e = State.NextToken->FakeRParens; i != e; ++i) { |
Daniel Jasper | 335ff26 | 2014-05-28 09:11:53 +0000 | [diff] [blame] | 860 | unsigned VariablePos = State.Stack.back().VariablePos; |
Daniel Jasper | 335ff26 | 2014-05-28 09:11:53 +0000 | [diff] [blame] | 861 | if (State.Stack.size() == 1) { |
| 862 | // Do not pop the last element. |
| 863 | break; |
| 864 | } |
| 865 | State.Stack.pop_back(); |
| 866 | State.Stack.back().VariablePos = VariablePos; |
| 867 | } |
| 868 | } |
| 869 | |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 870 | void ContinuationIndenter::moveStatePastScopeOpener(LineState &State, |
| 871 | bool Newline) { |
| 872 | const FormatToken &Current = *State.NextToken; |
| 873 | if (!Current.opensScope()) |
| 874 | return; |
| 875 | |
| 876 | if (Current.MatchingParen && Current.BlockKind == BK_Block) { |
| 877 | moveStateToNewBlock(State); |
| 878 | return; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 879 | } |
| 880 | |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 881 | unsigned NewIndent; |
| 882 | unsigned NewIndentLevel = State.Stack.back().IndentLevel; |
Daniel Jasper | de7ca75 | 2015-05-04 07:39:00 +0000 | [diff] [blame] | 883 | unsigned LastSpace = State.Stack.back().LastSpace; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 884 | bool AvoidBinPacking; |
| 885 | bool BreakBeforeParameter = false; |
Daniel Jasper | ea40cee | 2015-07-14 11:26:14 +0000 | [diff] [blame] | 886 | unsigned NestedBlockIndent = std::max(State.Stack.back().StartOfFunctionCall, |
| 887 | State.Stack.back().NestedBlockIndent); |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 888 | if (Current.isOneOf(tok::l_brace, TT_ArrayInitializerLSquare)) { |
Daniel Jasper | bd73bcf | 2015-10-27 13:42:08 +0000 | [diff] [blame] | 889 | if (Current.opensBlockOrBlockTypeList(Style)) { |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 890 | NewIndent = State.Stack.back().NestedBlockIndent + Style.IndentWidth; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 891 | NewIndent = std::min(State.Column + 2, NewIndent); |
| 892 | ++NewIndentLevel; |
| 893 | } else { |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 894 | NewIndent = State.Stack.back().LastSpace + Style.ContinuationIndentWidth; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 895 | } |
| 896 | const FormatToken *NextNoComment = Current.getNextNonComment(); |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 897 | AvoidBinPacking = |
| 898 | Current.isOneOf(TT_ArrayInitializerLSquare, TT_DictLiteral) || |
Daniel Jasper | 0843434 | 2015-05-26 07:26:26 +0000 | [diff] [blame] | 899 | Style.Language == FormatStyle::LK_Proto || !Style.BinPackArguments || |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 900 | (NextNoComment && NextNoComment->is(TT_DesignatedInitializerPeriod)); |
Daniel Jasper | ea40cee | 2015-07-14 11:26:14 +0000 | [diff] [blame] | 901 | if (Current.ParameterCount > 1) |
| 902 | NestedBlockIndent = std::max(NestedBlockIndent, State.Column + 1); |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 903 | } else { |
| 904 | NewIndent = Style.ContinuationIndentWidth + |
| 905 | std::max(State.Stack.back().LastSpace, |
| 906 | State.Stack.back().StartOfFunctionCall); |
Daniel Jasper | de7ca75 | 2015-05-04 07:39:00 +0000 | [diff] [blame] | 907 | |
| 908 | // Ensure that different different brackets force relative alignment, e.g.: |
| 909 | // void SomeFunction(vector< // break |
| 910 | // int> v); |
| 911 | // FIXME: We likely want to do this for more combinations of brackets. |
| 912 | // Verify that it is wanted for ObjC, too. |
| 913 | if (Current.Tok.getKind() == tok::less && |
| 914 | Current.ParentBracket == tok::l_paren) { |
| 915 | NewIndent = std::max(NewIndent, State.Stack.back().Indent); |
| 916 | LastSpace = std::max(LastSpace, State.Stack.back().Indent); |
| 917 | } |
| 918 | |
Daniel Jasper | 18210d7 | 2014-10-09 09:52:05 +0000 | [diff] [blame] | 919 | AvoidBinPacking = |
| 920 | (State.Line->MustBeDeclaration && !Style.BinPackParameters) || |
| 921 | (!State.Line->MustBeDeclaration && !Style.BinPackArguments) || |
| 922 | (Style.ExperimentalAutoDetectBinPacking && |
| 923 | (Current.PackingKind == PPK_OnePerLine || |
| 924 | (!BinPackInconclusiveFunctions && |
| 925 | Current.PackingKind == PPK_Inconclusive))); |
Daniel Jasper | 289afc0 | 2015-04-23 09:23:17 +0000 | [diff] [blame] | 926 | if (Current.is(TT_ObjCMethodExpr) && Current.MatchingParen) { |
| 927 | if (Style.ColumnLimit) { |
| 928 | // If this '[' opens an ObjC call, determine whether all parameters fit |
| 929 | // into one line and put one per line if they don't. |
| 930 | if (getLengthToMatchingParen(Current) + State.Column > |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 931 | getColumnLimit(State)) |
Daniel Jasper | 289afc0 | 2015-04-23 09:23:17 +0000 | [diff] [blame] | 932 | BreakBeforeParameter = true; |
| 933 | } else { |
| 934 | // For ColumnLimit = 0, we have to figure out whether there is or has to |
| 935 | // be a line break within this call. |
| 936 | for (const FormatToken *Tok = &Current; |
| 937 | Tok && Tok != Current.MatchingParen; Tok = Tok->Next) { |
Daniel Jasper | e6fcf7d | 2015-06-17 13:08:06 +0000 | [diff] [blame] | 938 | if (Tok->MustBreakBefore || |
Daniel Jasper | 289afc0 | 2015-04-23 09:23:17 +0000 | [diff] [blame] | 939 | (Tok->CanBreakBefore && Tok->NewlinesBefore > 0)) { |
| 940 | BreakBeforeParameter = true; |
| 941 | break; |
| 942 | } |
| 943 | } |
| 944 | } |
| 945 | } |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 946 | } |
Daniel Jasper | bd73bcf | 2015-10-27 13:42:08 +0000 | [diff] [blame] | 947 | // Generally inherit NoLineBreak from the current scope to nested scope. |
| 948 | // However, don't do this for non-empty nested blocks, dict literals and |
| 949 | // array literals as these follow different indentation rules. |
| 950 | bool NoLineBreak = |
| 951 | Current.Children.empty() && |
| 952 | !Current.isOneOf(TT_DictLiteral, TT_ArrayInitializerLSquare) && |
| 953 | (State.Stack.back().NoLineBreak || |
| 954 | (Current.is(TT_TemplateOpener) && |
| 955 | State.Stack.back().ContainsUnwrappedBuilder)); |
Daniel Jasper | de7ca75 | 2015-05-04 07:39:00 +0000 | [diff] [blame] | 956 | State.Stack.push_back(ParenState(NewIndent, NewIndentLevel, LastSpace, |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 957 | AvoidBinPacking, NoLineBreak)); |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 958 | State.Stack.back().NestedBlockIndent = NestedBlockIndent; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 959 | State.Stack.back().BreakBeforeParameter = BreakBeforeParameter; |
Daniel Jasper | 114a2bc | 2014-06-03 12:02:45 +0000 | [diff] [blame] | 960 | State.Stack.back().HasMultipleNestedBlocks = Current.BlockParameterCount > 1; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 961 | } |
| 962 | |
| 963 | void ContinuationIndenter::moveStatePastScopeCloser(LineState &State) { |
| 964 | const FormatToken &Current = *State.NextToken; |
| 965 | if (!Current.closesScope()) |
| 966 | return; |
| 967 | |
| 968 | // If we encounter a closing ), ], } or >, we can remove a level from our |
| 969 | // stacks. |
| 970 | if (State.Stack.size() > 1 && |
| 971 | (Current.isOneOf(tok::r_paren, tok::r_square) || |
| 972 | (Current.is(tok::r_brace) && State.NextToken != State.Line->First) || |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 973 | State.NextToken->is(TT_TemplateCloser))) |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 974 | State.Stack.pop_back(); |
Daniel Jasper | 335ff26 | 2014-05-28 09:11:53 +0000 | [diff] [blame] | 975 | |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 976 | if (Current.is(tok::r_square)) { |
| 977 | // If this ends the array subscript expr, reset the corresponding value. |
| 978 | const FormatToken *NextNonComment = Current.getNextNonComment(); |
| 979 | if (NextNonComment && NextNonComment->isNot(tok::l_square)) |
| 980 | State.Stack.back().StartOfArraySubscripts = 0; |
| 981 | } |
| 982 | } |
| 983 | |
| 984 | void ContinuationIndenter::moveStateToNewBlock(LineState &State) { |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 985 | unsigned NestedBlockIndent = State.Stack.back().NestedBlockIndent; |
Daniel Jasper | 50d634b | 2014-10-28 16:53:38 +0000 | [diff] [blame] | 986 | // ObjC block sometimes follow special indentation rules. |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 987 | unsigned NewIndent = |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 988 | NestedBlockIndent + (State.NextToken->is(TT_ObjCBlockLBrace) |
| 989 | ? Style.ObjCBlockIndentWidth |
| 990 | : Style.IndentWidth); |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 991 | State.Stack.push_back(ParenState( |
| 992 | NewIndent, /*NewIndentLevel=*/State.Stack.back().IndentLevel + 1, |
| 993 | State.Stack.back().LastSpace, /*AvoidBinPacking=*/true, |
Daniel Jasper | 6501f7e | 2015-10-27 12:38:37 +0000 | [diff] [blame] | 994 | /*NoLineBreak=*/false)); |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 995 | State.Stack.back().NestedBlockIndent = NestedBlockIndent; |
Daniel Jasper | 60553be | 2014-05-26 13:10:39 +0000 | [diff] [blame] | 996 | State.Stack.back().BreakBeforeParameter = true; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 997 | } |
| 998 | |
Alexander Kornienko | 917f9e0 | 2013-09-10 12:29:48 +0000 | [diff] [blame] | 999 | unsigned ContinuationIndenter::addMultilineToken(const FormatToken &Current, |
| 1000 | LineState &State) { |
Alexander Kornienko | d7b837e | 2013-08-29 17:32:57 +0000 | [diff] [blame] | 1001 | // Break before further function parameters on all levels. |
| 1002 | for (unsigned i = 0, e = State.Stack.size(); i != e; ++i) |
| 1003 | State.Stack[i].BreakBeforeParameter = true; |
| 1004 | |
Alexander Kornienko | 39856b7 | 2013-09-10 09:38:25 +0000 | [diff] [blame] | 1005 | unsigned ColumnsUsed = State.Column; |
Alexander Kornienko | 632abb9 | 2013-09-02 13:58:14 +0000 | [diff] [blame] | 1006 | // We can only affect layout of the first and the last line, so the penalty |
| 1007 | // for all other lines is constant, and we ignore it. |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 1008 | State.Column = Current.LastLineColumnWidth; |
Alexander Kornienko | 632abb9 | 2013-09-02 13:58:14 +0000 | [diff] [blame] | 1009 | |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 1010 | if (ColumnsUsed > getColumnLimit(State)) |
| 1011 | return Style.PenaltyExcessCharacter * (ColumnsUsed - getColumnLimit(State)); |
Alexander Kornienko | d7b837e | 2013-08-29 17:32:57 +0000 | [diff] [blame] | 1012 | return 0; |
| 1013 | } |
| 1014 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1015 | unsigned ContinuationIndenter::breakProtrudingToken(const FormatToken &Current, |
| 1016 | LineState &State, |
| 1017 | bool DryRun) { |
Alexander Kornienko | 917f9e0 | 2013-09-10 12:29:48 +0000 | [diff] [blame] | 1018 | // Don't break multi-line tokens other than block comments. Instead, just |
| 1019 | // update the state. |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1020 | if (Current.isNot(TT_BlockComment) && Current.IsMultiline) |
Alexander Kornienko | 917f9e0 | 2013-09-10 12:29:48 +0000 | [diff] [blame] | 1021 | return addMultilineToken(Current, State); |
| 1022 | |
Daniel Jasper | 616de864 | 2014-11-23 16:46:28 +0000 | [diff] [blame] | 1023 | // Don't break implicit string literals or import statements. |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1024 | if (Current.is(TT_ImplicitStringLiteral) || |
Daniel Jasper | 616de864 | 2014-11-23 16:46:28 +0000 | [diff] [blame] | 1025 | State.Line->Type == LT_ImportStatement) |
Daniel Jasper | 9885784 | 2013-10-30 13:54:53 +0000 | [diff] [blame] | 1026 | return 0; |
| 1027 | |
Daniel Jasper | 04b6a08 | 2013-12-20 06:22:01 +0000 | [diff] [blame] | 1028 | if (!Current.isStringLiteral() && !Current.is(tok::comment)) |
Daniel Jasper | f93551c | 2013-08-23 10:05:49 +0000 | [diff] [blame] | 1029 | return 0; |
| 1030 | |
Ahmed Charles | b898432 | 2014-03-07 20:03:18 +0000 | [diff] [blame] | 1031 | std::unique_ptr<BreakableToken> Token; |
Alexander Kornienko | 39856b7 | 2013-09-10 09:38:25 +0000 | [diff] [blame] | 1032 | unsigned StartColumn = State.Column - Current.ColumnWidth; |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1033 | unsigned ColumnLimit = getColumnLimit(State); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1034 | |
Daniel Jasper | 04b6a08 | 2013-12-20 06:22:01 +0000 | [diff] [blame] | 1035 | if (Current.isStringLiteral()) { |
Daniel Jasper | 428f0b1 | 2015-01-04 09:11:17 +0000 | [diff] [blame] | 1036 | // FIXME: String literal breaking is currently disabled for Java and JS, as |
| 1037 | // it requires strings to be merged using "+" which we don't support. |
| 1038 | if (Style.Language == FormatStyle::LK_Java || |
| 1039 | Style.Language == FormatStyle::LK_JavaScript) |
| 1040 | return 0; |
| 1041 | |
Alexander Kornienko | 384b40b | 2013-10-11 21:43:05 +0000 | [diff] [blame] | 1042 | // Don't break string literals inside preprocessor directives (except for |
| 1043 | // #define directives, as their contents are stored in separate lines and |
| 1044 | // are not affected by this check). |
| 1045 | // This way we avoid breaking code with line directives and unknown |
| 1046 | // preprocessor directives that contain long string literals. |
| 1047 | if (State.Line->Type == LT_PreprocessorDirective) |
| 1048 | return 0; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1049 | // Exempts unterminated string literals from line breaking. The user will |
| 1050 | // likely want to terminate the string before any line breaking is done. |
| 1051 | if (Current.IsUnterminatedLiteral) |
| 1052 | return 0; |
| 1053 | |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 1054 | StringRef Text = Current.TokenText; |
| 1055 | StringRef Prefix; |
| 1056 | StringRef Postfix; |
Daniel Jasper | 174b012 | 2014-01-09 14:18:12 +0000 | [diff] [blame] | 1057 | bool IsNSStringLiteral = false; |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 1058 | // FIXME: Handle whitespace between '_T', '(', '"..."', and ')'. |
| 1059 | // FIXME: Store Prefix and Suffix (or PrefixLength and SuffixLength to |
| 1060 | // reduce the overhead) for each FormatToken, which is a string, so that we |
| 1061 | // don't run multiple checks here on the hot path. |
Daniel Jasper | 174b012 | 2014-01-09 14:18:12 +0000 | [diff] [blame] | 1062 | if (Text.startswith("\"") && Current.Previous && |
| 1063 | Current.Previous->is(tok::at)) { |
| 1064 | IsNSStringLiteral = true; |
| 1065 | Prefix = "@\""; |
Daniel Jasper | 174b012 | 2014-01-09 14:18:12 +0000 | [diff] [blame] | 1066 | } |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 1067 | if ((Text.endswith(Postfix = "\"") && |
Daniel Jasper | 174b012 | 2014-01-09 14:18:12 +0000 | [diff] [blame] | 1068 | (IsNSStringLiteral || Text.startswith(Prefix = "\"") || |
| 1069 | Text.startswith(Prefix = "u\"") || Text.startswith(Prefix = "U\"") || |
| 1070 | Text.startswith(Prefix = "u8\"") || |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 1071 | Text.startswith(Prefix = "L\""))) || |
Alexander Kornienko | 732b6bd | 2014-12-14 20:47:11 +0000 | [diff] [blame] | 1072 | (Text.startswith(Prefix = "_T(\"") && Text.endswith(Postfix = "\")"))) { |
Alexander Kornienko | 3c3d09c | 2013-09-27 16:14:22 +0000 | [diff] [blame] | 1073 | Token.reset(new BreakableStringLiteral( |
| 1074 | Current, State.Line->Level, StartColumn, Prefix, Postfix, |
| 1075 | State.Line->InPPDirective, Encoding, Style)); |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 1076 | } else { |
| 1077 | return 0; |
| 1078 | } |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1079 | } else if (Current.is(TT_BlockComment) && Current.isTrailingComment()) { |
Daniel Jasper | a0a5039 | 2015-12-01 13:28:53 +0000 | [diff] [blame] | 1080 | if (!Style.ReflowComments || |
| 1081 | CommentPragmasRegex.match(Current.TokenText.substr(2))) |
Alexander Kornienko | ce9161a | 2014-01-02 15:13:14 +0000 | [diff] [blame] | 1082 | return 0; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1083 | Token.reset(new BreakableBlockComment( |
Alexander Kornienko | 3c3d09c | 2013-09-27 16:14:22 +0000 | [diff] [blame] | 1084 | Current, State.Line->Level, StartColumn, Current.OriginalColumn, |
| 1085 | !Current.Previous, State.Line->InPPDirective, Encoding, Style)); |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1086 | } else if (Current.is(TT_LineComment) && |
Craig Topper | 2145bc0 | 2014-05-09 08:15:10 +0000 | [diff] [blame] | 1087 | (Current.Previous == nullptr || |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1088 | Current.Previous->isNot(TT_ImplicitStringLiteral))) { |
Daniel Jasper | a0a5039 | 2015-12-01 13:28:53 +0000 | [diff] [blame] | 1089 | if (!Style.ReflowComments || |
| 1090 | CommentPragmasRegex.match(Current.TokenText.substr(2))) |
Alexander Kornienko | ce9161a | 2014-01-02 15:13:14 +0000 | [diff] [blame] | 1091 | return 0; |
Alexander Kornienko | 3c3d09c | 2013-09-27 16:14:22 +0000 | [diff] [blame] | 1092 | Token.reset(new BreakableLineComment(Current, State.Line->Level, |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1093 | StartColumn, /*InPPDirective=*/false, |
Alexander Kornienko | 3c3d09c | 2013-09-27 16:14:22 +0000 | [diff] [blame] | 1094 | Encoding, Style)); |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1095 | // We don't insert backslashes when breaking line comments. |
| 1096 | ColumnLimit = Style.ColumnLimit; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1097 | } else { |
| 1098 | return 0; |
| 1099 | } |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1100 | if (Current.UnbreakableTailLength >= ColumnLimit) |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1101 | return 0; |
| 1102 | |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1103 | unsigned RemainingSpace = ColumnLimit - Current.UnbreakableTailLength; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1104 | bool BreakInserted = false; |
| 1105 | unsigned Penalty = 0; |
| 1106 | unsigned RemainingTokenColumns = 0; |
| 1107 | for (unsigned LineIndex = 0, EndIndex = Token->getLineCount(); |
| 1108 | LineIndex != EndIndex; ++LineIndex) { |
| 1109 | if (!DryRun) |
| 1110 | Token->replaceWhitespaceBefore(LineIndex, Whitespaces); |
| 1111 | unsigned TailOffset = 0; |
| 1112 | RemainingTokenColumns = |
| 1113 | Token->getLineLengthAfterSplit(LineIndex, TailOffset, StringRef::npos); |
| 1114 | while (RemainingTokenColumns > RemainingSpace) { |
| 1115 | BreakableToken::Split Split = |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1116 | Token->getSplit(LineIndex, TailOffset, ColumnLimit); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1117 | if (Split.first == StringRef::npos) { |
| 1118 | // The last line's penalty is handled in addNextStateToQueue(). |
| 1119 | if (LineIndex < EndIndex - 1) |
| 1120 | Penalty += Style.PenaltyExcessCharacter * |
| 1121 | (RemainingTokenColumns - RemainingSpace); |
| 1122 | break; |
| 1123 | } |
| 1124 | assert(Split.first != 0); |
| 1125 | unsigned NewRemainingTokenColumns = Token->getLineLengthAfterSplit( |
| 1126 | LineIndex, TailOffset + Split.first + Split.second, StringRef::npos); |
Alexander Kornienko | 875395f | 2013-11-12 17:50:13 +0000 | [diff] [blame] | 1127 | |
| 1128 | // We can remove extra whitespace instead of breaking the line. |
| 1129 | if (RemainingTokenColumns + 1 - Split.second <= RemainingSpace) { |
| 1130 | RemainingTokenColumns = 0; |
| 1131 | if (!DryRun) |
| 1132 | Token->replaceWhitespace(LineIndex, TailOffset, Split, Whitespaces); |
| 1133 | break; |
| 1134 | } |
| 1135 | |
Alexander Kornienko | 64a42b8 | 2014-04-15 14:52:43 +0000 | [diff] [blame] | 1136 | // When breaking before a tab character, it may be moved by a few columns, |
| 1137 | // but will still be expanded to the next tab stop, so we don't save any |
| 1138 | // columns. |
| 1139 | if (NewRemainingTokenColumns == RemainingTokenColumns) |
| 1140 | break; |
| 1141 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1142 | assert(NewRemainingTokenColumns < RemainingTokenColumns); |
| 1143 | if (!DryRun) |
| 1144 | Token->insertBreak(LineIndex, TailOffset, Split, Whitespaces); |
Daniel Jasper | 2739af3 | 2013-08-28 10:03:58 +0000 | [diff] [blame] | 1145 | Penalty += Current.SplitPenalty; |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1146 | unsigned ColumnsUsed = |
| 1147 | Token->getLineLengthAfterSplit(LineIndex, TailOffset, Split.first); |
Alexander Kornienko | 3abbb8a | 2013-11-12 17:30:49 +0000 | [diff] [blame] | 1148 | if (ColumnsUsed > ColumnLimit) { |
| 1149 | Penalty += Style.PenaltyExcessCharacter * (ColumnsUsed - ColumnLimit); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1150 | } |
| 1151 | TailOffset += Split.first + Split.second; |
| 1152 | RemainingTokenColumns = NewRemainingTokenColumns; |
| 1153 | BreakInserted = true; |
| 1154 | } |
| 1155 | } |
| 1156 | |
| 1157 | State.Column = RemainingTokenColumns; |
| 1158 | |
| 1159 | if (BreakInserted) { |
| 1160 | // If we break the token inside a parameter list, we need to break before |
| 1161 | // the next parameter on all levels, so that the next parameter is clearly |
| 1162 | // visible. Line comments already introduce a break. |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1163 | if (Current.isNot(TT_LineComment)) { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1164 | for (unsigned i = 0, e = State.Stack.size(); i != e; ++i) |
| 1165 | State.Stack[i].BreakBeforeParameter = true; |
| 1166 | } |
| 1167 | |
Daniel Jasper | 04b6a08 | 2013-12-20 06:22:01 +0000 | [diff] [blame] | 1168 | Penalty += Current.isStringLiteral() ? Style.PenaltyBreakString |
| 1169 | : Style.PenaltyBreakComment; |
Daniel Jasper | 2739af3 | 2013-08-28 10:03:58 +0000 | [diff] [blame] | 1170 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1171 | State.Stack.back().LastSpace = StartColumn; |
| 1172 | } |
| 1173 | return Penalty; |
| 1174 | } |
| 1175 | |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 1176 | unsigned ContinuationIndenter::getColumnLimit(const LineState &State) const { |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1177 | // In preprocessor directives reserve two chars for trailing " \" |
Daniel Jasper | 9fe0e8d | 2013-09-05 09:29:45 +0000 | [diff] [blame] | 1178 | return Style.ColumnLimit - (State.Line->InPPDirective ? 2 : 0); |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1179 | } |
| 1180 | |
Daniel Jasper | c39b56f | 2013-12-16 07:23:08 +0000 | [diff] [blame] | 1181 | bool ContinuationIndenter::nextIsMultilineString(const LineState &State) { |
Daniel Jasper | f438cb7 | 2013-08-23 11:57:34 +0000 | [diff] [blame] | 1182 | const FormatToken &Current = *State.NextToken; |
Daniel Jasper | a98b7b0 | 2014-11-25 10:05:17 +0000 | [diff] [blame] | 1183 | if (!Current.isStringLiteral() || Current.is(TT_ImplicitStringLiteral)) |
Daniel Jasper | f438cb7 | 2013-08-23 11:57:34 +0000 | [diff] [blame] | 1184 | return false; |
Alexander Kornienko | d7b837e | 2013-08-29 17:32:57 +0000 | [diff] [blame] | 1185 | // We never consider raw string literals "multiline" for the purpose of |
Daniel Jasper | c39b56f | 2013-12-16 07:23:08 +0000 | [diff] [blame] | 1186 | // AlwaysBreakBeforeMultilineStrings implementation as they are special-cased |
| 1187 | // (see TokenAnnotator::mustBreakBefore(). |
Alexander Kornienko | d7b837e | 2013-08-29 17:32:57 +0000 | [diff] [blame] | 1188 | if (Current.TokenText.startswith("R\"")) |
| 1189 | return false; |
Alexander Kornienko | 39856b7 | 2013-09-10 09:38:25 +0000 | [diff] [blame] | 1190 | if (Current.IsMultiline) |
Alexander Kornienko | d7b837e | 2013-08-29 17:32:57 +0000 | [diff] [blame] | 1191 | return true; |
Daniel Jasper | f438cb7 | 2013-08-23 11:57:34 +0000 | [diff] [blame] | 1192 | if (Current.getNextNonComment() && |
Daniel Jasper | 04b6a08 | 2013-12-20 06:22:01 +0000 | [diff] [blame] | 1193 | Current.getNextNonComment()->isStringLiteral()) |
Daniel Jasper | f438cb7 | 2013-08-23 11:57:34 +0000 | [diff] [blame] | 1194 | return true; // Implicit concatenation. |
Daniel Jasper | 6fd5d64 | 2015-01-20 12:59:20 +0000 | [diff] [blame] | 1195 | if (Style.ColumnLimit != 0 && |
| 1196 | State.Column + Current.ColumnWidth + Current.UnbreakableTailLength > |
| 1197 | Style.ColumnLimit) |
Daniel Jasper | f438cb7 | 2013-08-23 11:57:34 +0000 | [diff] [blame] | 1198 | return true; // String will be split. |
Alexander Kornienko | d7b837e | 2013-08-29 17:32:57 +0000 | [diff] [blame] | 1199 | return false; |
Daniel Jasper | f438cb7 | 2013-08-23 11:57:34 +0000 | [diff] [blame] | 1200 | } |
| 1201 | |
Daniel Jasper | de0328a | 2013-08-16 11:20:30 +0000 | [diff] [blame] | 1202 | } // namespace format |
| 1203 | } // namespace clang |