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