2013-08-16 19:20:30 +08:00
|
|
|
//===--- ContinuationIndenter.cpp - Format C++ code -----------------------===//
|
|
|
|
//
|
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
///
|
|
|
|
/// \file
|
|
|
|
/// \brief This file implements the continuation indenter.
|
|
|
|
///
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
|
|
|
#define DEBUG_TYPE "format-formatter"
|
|
|
|
|
|
|
|
#include "BreakableToken.h"
|
|
|
|
#include "ContinuationIndenter.h"
|
|
|
|
#include "WhitespaceManager.h"
|
|
|
|
#include "clang/Basic/OperatorPrecedence.h"
|
|
|
|
#include "clang/Basic/SourceManager.h"
|
|
|
|
#include "clang/Format/Format.h"
|
|
|
|
#include "llvm/Support/Debug.h"
|
|
|
|
#include <string>
|
|
|
|
|
|
|
|
namespace clang {
|
|
|
|
namespace format {
|
|
|
|
|
|
|
|
// Returns the length of everything up to the first possible line break after
|
|
|
|
// the ), ], } or > matching \c Tok.
|
|
|
|
static unsigned getLengthToMatchingParen(const FormatToken &Tok) {
|
|
|
|
if (Tok.MatchingParen == NULL)
|
|
|
|
return 0;
|
|
|
|
FormatToken *End = Tok.MatchingParen;
|
|
|
|
while (End->Next && !End->Next->CanBreakBefore) {
|
|
|
|
End = End->Next;
|
|
|
|
}
|
|
|
|
return End->TotalLength - Tok.TotalLength + 1;
|
|
|
|
}
|
|
|
|
|
clang-format: Format segments of builder-type calls one per line.
This fixes llvm.org/PR14818.
Before:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT).StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH).Default(ORDER_TEXT);
After:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT)
.StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH)
.Default(ORDER_TEXT);
llvm-svn: 189353
2013-08-27 22:24:43 +08:00
|
|
|
// Returns \c true if \c Tok is the "." or "->" of a call and starts the next
|
|
|
|
// segment of a builder type call.
|
|
|
|
static bool startsSegmentOfBuilderTypeCall(const FormatToken &Tok) {
|
|
|
|
return Tok.isMemberAccess() && Tok.Previous && Tok.Previous->closesScope();
|
|
|
|
}
|
|
|
|
|
2013-10-08 13:11:18 +08:00
|
|
|
// Returns \c true if \c Current starts a new parameter.
|
|
|
|
static bool startsNextParameter(const FormatToken &Current,
|
|
|
|
const FormatStyle &Style) {
|
|
|
|
const FormatToken &Previous = *Current.Previous;
|
|
|
|
if (Current.Type == TT_CtorInitializerComma &&
|
|
|
|
Style.BreakConstructorInitializersBeforeComma)
|
|
|
|
return true;
|
|
|
|
return Previous.is(tok::comma) && !Current.isTrailingComment() &&
|
|
|
|
(Previous.Type != TT_CtorInitializerComma ||
|
|
|
|
!Style.BreakConstructorInitializersBeforeComma);
|
|
|
|
}
|
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
ContinuationIndenter::ContinuationIndenter(const FormatStyle &Style,
|
|
|
|
SourceManager &SourceMgr,
|
|
|
|
WhitespaceManager &Whitespaces,
|
|
|
|
encoding::Encoding Encoding,
|
|
|
|
bool BinPackInconclusiveFunctions)
|
2013-09-05 17:29:45 +08:00
|
|
|
: Style(Style), SourceMgr(SourceMgr), Whitespaces(Whitespaces),
|
|
|
|
Encoding(Encoding),
|
2014-01-02 23:13:14 +08:00
|
|
|
BinPackInconclusiveFunctions(BinPackInconclusiveFunctions),
|
|
|
|
CommentPragmasRegex(Style.CommentPragmas) {}
|
2013-08-16 19:20:30 +08:00
|
|
|
|
2013-09-05 17:29:45 +08:00
|
|
|
LineState ContinuationIndenter::getInitialState(unsigned FirstIndent,
|
2013-09-06 15:54:20 +08:00
|
|
|
const AnnotatedLine *Line,
|
|
|
|
bool DryRun) {
|
2013-08-16 19:20:30 +08:00
|
|
|
LineState State;
|
2013-09-05 17:29:45 +08:00
|
|
|
State.FirstIndent = FirstIndent;
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Column = FirstIndent;
|
2013-09-05 17:29:45 +08:00
|
|
|
State.Line = Line;
|
|
|
|
State.NextToken = Line->First;
|
2013-10-14 08:46:35 +08:00
|
|
|
State.Stack.push_back(ParenState(FirstIndent, Line->Level, FirstIndent,
|
2013-08-16 19:20:30 +08:00
|
|
|
/*AvoidBinPacking=*/false,
|
|
|
|
/*NoLineBreak=*/false));
|
|
|
|
State.LineContainsContinuedForLoopSection = false;
|
|
|
|
State.ParenLevel = 0;
|
|
|
|
State.StartOfStringLiteral = 0;
|
|
|
|
State.StartOfLineLevel = State.ParenLevel;
|
|
|
|
State.LowestLevelOnLine = State.ParenLevel;
|
|
|
|
State.IgnoreStackForComparison = false;
|
|
|
|
|
|
|
|
// The first token has already been indented and thus consumed.
|
2013-09-06 15:54:20 +08:00
|
|
|
moveStateToNextToken(State, DryRun, /*Newline=*/false);
|
2013-08-16 19:20:30 +08:00
|
|
|
return State;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool ContinuationIndenter::canBreak(const LineState &State) {
|
|
|
|
const FormatToken &Current = *State.NextToken;
|
|
|
|
const FormatToken &Previous = *Current.Previous;
|
|
|
|
assert(&Previous == Current.Previous);
|
2013-10-22 23:30:28 +08:00
|
|
|
if (!Current.CanBreakBefore && !(State.Stack.back().BreakBeforeClosingBrace &&
|
|
|
|
Current.closesBlockTypeList(Style)))
|
2013-08-16 19:20:30 +08:00
|
|
|
return false;
|
|
|
|
// The opening "{" of a braced list has to be on the same line as the first
|
|
|
|
// element if it is nested in another braced init list or function call.
|
|
|
|
if (!Current.MustBreakBefore && Previous.is(tok::l_brace) &&
|
2013-10-24 18:31:50 +08:00
|
|
|
Previous.Type != TT_DictLiteral &&
|
2013-09-05 17:29:45 +08:00
|
|
|
Previous.BlockKind == BK_BracedInit && Previous.Previous &&
|
2013-08-16 19:20:30 +08:00
|
|
|
Previous.Previous->isOneOf(tok::l_brace, tok::l_paren, tok::comma))
|
|
|
|
return false;
|
|
|
|
// This prevents breaks like:
|
|
|
|
// ...
|
|
|
|
// SomeParameter, OtherParameter).DoSomething(
|
|
|
|
// ...
|
|
|
|
// As they hide "DoSomething" and are generally bad for readability.
|
|
|
|
if (Previous.opensScope() && State.LowestLevelOnLine < State.StartOfLineLevel)
|
|
|
|
return false;
|
clang-format: Format segments of builder-type calls one per line.
This fixes llvm.org/PR14818.
Before:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT).StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH).Default(ORDER_TEXT);
After:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT)
.StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH)
.Default(ORDER_TEXT);
llvm-svn: 189353
2013-08-27 22:24:43 +08:00
|
|
|
if (Current.isMemberAccess() && State.Stack.back().ContainsUnwrappedBuilder)
|
|
|
|
return false;
|
2013-08-16 19:20:30 +08:00
|
|
|
return !State.Stack.back().NoLineBreak;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool ContinuationIndenter::mustBreak(const LineState &State) {
|
|
|
|
const FormatToken &Current = *State.NextToken;
|
|
|
|
const FormatToken &Previous = *Current.Previous;
|
|
|
|
if (Current.MustBreakBefore || Current.Type == TT_InlineASMColon)
|
|
|
|
return true;
|
2013-10-22 23:30:28 +08:00
|
|
|
if (State.Stack.back().BreakBeforeClosingBrace &&
|
|
|
|
Current.closesBlockTypeList(Style))
|
2013-08-16 19:20:30 +08:00
|
|
|
return true;
|
|
|
|
if (Previous.is(tok::semi) && State.LineContainsContinuedForLoopSection)
|
|
|
|
return true;
|
2013-10-08 13:11:18 +08:00
|
|
|
if ((startsNextParameter(Current, Style) || Previous.is(tok::semi) ||
|
2013-11-08 08:57:11 +08:00
|
|
|
(Style.BreakBeforeTernaryOperators &&
|
|
|
|
(Current.is(tok::question) || (Current.Type == TT_ConditionalExpr &&
|
|
|
|
Previous.isNot(tok::question)))) ||
|
|
|
|
(!Style.BreakBeforeTernaryOperators &&
|
|
|
|
(Previous.is(tok::question) || Previous.Type == TT_ConditionalExpr))) &&
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.back().BreakBeforeParameter && !Current.isTrailingComment() &&
|
|
|
|
!Current.isOneOf(tok::r_paren, tok::r_brace))
|
|
|
|
return true;
|
|
|
|
if (Style.AlwaysBreakBeforeMultilineStrings &&
|
2013-08-23 19:57:34 +08:00
|
|
|
State.Column > State.Stack.back().Indent && // Breaking saves columns.
|
2013-11-09 11:08:25 +08:00
|
|
|
!Previous.isOneOf(tok::kw_return, tok::lessless, tok::at) &&
|
2013-12-16 15:23:08 +08:00
|
|
|
Previous.Type != TT_InlineASMColon && nextIsMultilineString(State))
|
2013-08-16 19:20:30 +08:00
|
|
|
return true;
|
2013-10-24 18:31:50 +08:00
|
|
|
if (((Previous.Type == TT_DictLiteral && Previous.is(tok::l_brace)) ||
|
2013-10-22 23:30:28 +08:00
|
|
|
Previous.Type == TT_ArrayInitializerLSquare) &&
|
2013-10-21 00:45:46 +08:00
|
|
|
getLengthToMatchingParen(Previous) + State.Column > getColumnLimit(State))
|
|
|
|
return true;
|
2013-08-16 19:20:30 +08:00
|
|
|
|
|
|
|
if (!Style.BreakBeforeBinaryOperators) {
|
|
|
|
// If we need to break somewhere inside the LHS of a binary expression, we
|
|
|
|
// should also break after the operator. Otherwise, the formatting would
|
|
|
|
// hide the operator precedence, e.g. in:
|
|
|
|
// if (aaaaaaaaaaaaaa ==
|
|
|
|
// bbbbbbbbbbbbbb && c) {..
|
|
|
|
// For comparisons, we only apply this rule, if the LHS is a binary
|
|
|
|
// expression itself as otherwise, the line breaks seem superfluous.
|
|
|
|
// We need special cases for ">>" which we have split into two ">" while
|
|
|
|
// lexing in order to make template parsing easier.
|
|
|
|
//
|
|
|
|
// FIXME: We'll need something similar for styles that break before binary
|
|
|
|
// operators.
|
|
|
|
bool IsComparison = (Previous.getPrecedence() == prec::Relational ||
|
|
|
|
Previous.getPrecedence() == prec::Equality) &&
|
|
|
|
Previous.Previous &&
|
|
|
|
Previous.Previous->Type != TT_BinaryOperator; // For >>.
|
|
|
|
bool LHSIsBinaryExpr =
|
2013-09-06 16:08:14 +08:00
|
|
|
Previous.Previous && Previous.Previous->EndsBinaryExpression;
|
2013-08-16 19:20:30 +08:00
|
|
|
if (Previous.Type == TT_BinaryOperator &&
|
|
|
|
(!IsComparison || LHSIsBinaryExpr) &&
|
|
|
|
Current.Type != TT_BinaryOperator && // For >>.
|
|
|
|
!Current.isTrailingComment() &&
|
|
|
|
!Previous.isOneOf(tok::lessless, tok::question) &&
|
|
|
|
Previous.getPrecedence() != prec::Assignment &&
|
|
|
|
State.Stack.back().BreakBeforeParameter)
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Same as above, but for the first "<<" operator.
|
|
|
|
if (Current.is(tok::lessless) && State.Stack.back().BreakBeforeParameter &&
|
|
|
|
State.Stack.back().FirstLessLess == 0)
|
|
|
|
return true;
|
|
|
|
|
|
|
|
if (Current.Type == TT_ObjCSelectorName &&
|
2013-12-23 15:29:06 +08:00
|
|
|
State.Stack.back().ObjCSelectorNameFound &&
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.back().BreakBeforeParameter)
|
|
|
|
return true;
|
2013-12-16 22:35:51 +08:00
|
|
|
if (Previous.ClosesTemplateDeclaration && State.ParenLevel == 0 &&
|
|
|
|
!Current.isTrailingComment())
|
2013-08-16 19:20:30 +08:00
|
|
|
return true;
|
|
|
|
|
|
|
|
if ((Current.Type == TT_StartOfName || Current.is(tok::kw_operator)) &&
|
2013-09-05 17:29:45 +08:00
|
|
|
State.Line->MightBeFunctionDecl &&
|
|
|
|
State.Stack.back().BreakBeforeParameter && State.ParenLevel == 0)
|
2013-08-16 19:20:30 +08:00
|
|
|
return true;
|
clang-format: Format segments of builder-type calls one per line.
This fixes llvm.org/PR14818.
Before:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT).StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH).Default(ORDER_TEXT);
After:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT)
.StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH)
.Default(ORDER_TEXT);
llvm-svn: 189353
2013-08-27 22:24:43 +08:00
|
|
|
if (startsSegmentOfBuilderTypeCall(Current) &&
|
2013-08-30 15:12:40 +08:00
|
|
|
(State.Stack.back().CallContinuation != 0 ||
|
|
|
|
(State.Stack.back().BreakBeforeParameter &&
|
|
|
|
State.Stack.back().ContainsUnwrappedBuilder)))
|
clang-format: Format segments of builder-type calls one per line.
This fixes llvm.org/PR14818.
Before:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT).StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH).Default(ORDER_TEXT);
After:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT)
.StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH)
.Default(ORDER_TEXT);
llvm-svn: 189353
2013-08-27 22:24:43 +08:00
|
|
|
return true;
|
2014-01-05 20:38:10 +08:00
|
|
|
|
|
|
|
// The following could be precomputed as they do not depend on the state.
|
|
|
|
// However, as they should take effect only if the UnwrappedLine does not fit
|
|
|
|
// into the ColumnLimit, they are checked here in the ContinuationIndenter.
|
|
|
|
if (Previous.BlockKind == BK_Block && Previous.is(tok::l_brace) &&
|
|
|
|
!Current.isOneOf(tok::r_brace, tok::comment))
|
|
|
|
return true;
|
|
|
|
if (Current.Type == TT_CtorInitializerColon &&
|
|
|
|
(!Style.AllowShortFunctionsOnASingleLine ||
|
|
|
|
Style.BreakConstructorInitializersBeforeComma || Style.ColumnLimit != 0))
|
|
|
|
return true;
|
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
unsigned ContinuationIndenter::addTokenToState(LineState &State, bool Newline,
|
clang-format: Add column layout formatting for braced lists
With this patch, braced lists (with more than 3 elements are formatted in a
column layout if possible). E.g.:
static const uint16_t CallerSavedRegs64Bit[] = {
X86::RAX, X86::RDX, X86::RCX, X86::RSI, X86::RDI,
X86::R8, X86::R9, X86::R10, X86::R11, 0
};
Required other changes:
- FormatTokens can now have a special role that contains extra data and can do
special formattings. A comma separated list is currently the only
implementation.
- Move penalty calculation entirely into ContinuationIndenter (there was a last
piece still in UnwrappedLineFormatter).
Review: http://llvm-reviews.chandlerc.com/D1457
llvm-svn: 189018
2013-08-22 23:00:41 +08:00
|
|
|
bool DryRun,
|
|
|
|
unsigned ExtraSpaces) {
|
2013-08-16 19:20:30 +08:00
|
|
|
const FormatToken &Current = *State.NextToken;
|
|
|
|
|
2013-10-30 21:54:53 +08:00
|
|
|
if (State.Stack.size() == 0 ||
|
|
|
|
(Current.Type == TT_ImplicitStringLiteral &&
|
|
|
|
(Current.Previous->Tok.getIdentifierInfo() == NULL ||
|
|
|
|
Current.Previous->Tok.getIdentifierInfo()->getPPKeywordID() ==
|
|
|
|
tok::pp_not_keyword))) {
|
2013-08-16 19:20:30 +08:00
|
|
|
// FIXME: Is this correct?
|
|
|
|
int WhitespaceLength = SourceMgr.getSpellingColumnNumber(
|
|
|
|
State.NextToken->WhitespaceRange.getEnd()) -
|
|
|
|
SourceMgr.getSpellingColumnNumber(
|
|
|
|
State.NextToken->WhitespaceRange.getBegin());
|
2013-09-10 17:38:25 +08:00
|
|
|
State.Column += WhitespaceLength + State.NextToken->ColumnWidth;
|
2013-08-16 19:20:30 +08:00
|
|
|
State.NextToken = State.NextToken->Next;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-10-01 22:41:18 +08:00
|
|
|
unsigned Penalty = 0;
|
|
|
|
if (Newline)
|
|
|
|
Penalty = addTokenOnNewLine(State, DryRun);
|
|
|
|
else
|
2013-11-20 22:54:39 +08:00
|
|
|
addTokenOnCurrentLine(State, DryRun, ExtraSpaces);
|
2013-10-01 22:41:18 +08:00
|
|
|
|
|
|
|
return moveStateToNextToken(State, DryRun, Newline) + Penalty;
|
|
|
|
}
|
|
|
|
|
2013-11-20 22:54:39 +08:00
|
|
|
void ContinuationIndenter::addTokenOnCurrentLine(LineState &State, bool DryRun,
|
|
|
|
unsigned ExtraSpaces) {
|
2013-10-12 05:25:45 +08:00
|
|
|
FormatToken &Current = *State.NextToken;
|
2013-10-01 22:41:18 +08:00
|
|
|
const FormatToken &Previous = *State.NextToken->Previous;
|
|
|
|
if (Current.is(tok::equal) &&
|
|
|
|
(State.Line->First->is(tok::kw_for) || State.ParenLevel == 0) &&
|
|
|
|
State.Stack.back().VariablePos == 0) {
|
|
|
|
State.Stack.back().VariablePos = State.Column;
|
|
|
|
// Move over * and & if they are bound to the variable name.
|
|
|
|
const FormatToken *Tok = &Previous;
|
|
|
|
while (Tok && State.Stack.back().VariablePos >= Tok->ColumnWidth) {
|
|
|
|
State.Stack.back().VariablePos -= Tok->ColumnWidth;
|
|
|
|
if (Tok->SpacesRequiredBefore != 0)
|
|
|
|
break;
|
|
|
|
Tok = Tok->Previous;
|
|
|
|
}
|
|
|
|
if (Previous.PartOfMultiVariableDeclStmt)
|
|
|
|
State.Stack.back().LastSpace = State.Stack.back().VariablePos;
|
|
|
|
}
|
|
|
|
|
|
|
|
unsigned Spaces = Current.SpacesRequiredBefore + ExtraSpaces;
|
|
|
|
|
|
|
|
if (!DryRun)
|
|
|
|
Whitespaces.replaceWhitespace(Current, /*Newlines=*/0, /*IndentLevel=*/0,
|
|
|
|
Spaces, State.Column + Spaces);
|
|
|
|
|
2013-12-23 15:29:06 +08:00
|
|
|
if (Current.Type == TT_ObjCSelectorName &&
|
|
|
|
!State.Stack.back().ObjCSelectorNameFound) {
|
|
|
|
if (Current.LongestObjCSelectorName == 0)
|
|
|
|
State.Stack.back().AlignColons = false;
|
|
|
|
else if (State.Stack.back().Indent + Current.LongestObjCSelectorName >
|
|
|
|
State.Column + Spaces + Current.ColumnWidth)
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().ColonPos =
|
|
|
|
State.Stack.back().Indent + Current.LongestObjCSelectorName;
|
|
|
|
else
|
|
|
|
State.Stack.back().ColonPos = State.Column + Spaces + Current.ColumnWidth;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (Previous.opensScope() && Previous.Type != TT_ObjCMethodExpr &&
|
2013-12-20 05:41:37 +08:00
|
|
|
(Current.Type != TT_LineComment || Previous.BlockKind == BK_BracedInit))
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().Indent = State.Column + Spaces;
|
2013-10-08 13:11:18 +08:00
|
|
|
if (State.Stack.back().AvoidBinPacking && startsNextParameter(Current, Style))
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().NoLineBreak = true;
|
|
|
|
if (startsSegmentOfBuilderTypeCall(Current))
|
|
|
|
State.Stack.back().ContainsUnwrappedBuilder = true;
|
|
|
|
|
|
|
|
State.Column += Spaces;
|
|
|
|
if (Current.is(tok::l_paren) && Previous.isOneOf(tok::kw_if, tok::kw_for))
|
|
|
|
// Treat the condition inside an if as if it was a second function
|
2013-10-18 18:38:14 +08:00
|
|
|
// parameter, i.e. let nested calls have a continuation indent.
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().LastSpace = State.Column + 1; // 1 is length of "(".
|
2013-12-23 15:29:06 +08:00
|
|
|
else if (Current.isNot(tok::comment) &&
|
|
|
|
(Previous.is(tok::comma) ||
|
|
|
|
(Previous.is(tok::colon) && Previous.Type == TT_ObjCMethodExpr)))
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().LastSpace = State.Column;
|
|
|
|
else if ((Previous.Type == TT_BinaryOperator ||
|
|
|
|
Previous.Type == TT_ConditionalExpr ||
|
|
|
|
Previous.Type == TT_CtorInitializerColon) &&
|
|
|
|
(Previous.getPrecedence() != prec::Assignment ||
|
|
|
|
Current.StartsBinaryExpression))
|
|
|
|
// Always indent relative to the RHS of the expression unless this is a
|
|
|
|
// simple assignment without binary expression on the RHS. Also indent
|
|
|
|
// relative to unary operators and the colons of constructor initializers.
|
|
|
|
State.Stack.back().LastSpace = State.Column;
|
2013-10-09 00:24:07 +08:00
|
|
|
else if (Previous.Type == TT_InheritanceColon) {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().Indent = State.Column;
|
2013-10-09 00:24:07 +08:00
|
|
|
State.Stack.back().LastSpace = State.Column;
|
|
|
|
} else if (Previous.opensScope()) {
|
2013-10-01 22:41:18 +08:00
|
|
|
// If a function has a trailing call, indent all parameters from the
|
|
|
|
// opening parenthesis. This avoids confusing indents like:
|
|
|
|
// OuterFunction(InnerFunctionCall( // break
|
|
|
|
// ParameterToInnerFunction)) // break
|
|
|
|
// .SecondInnerFunctionCall();
|
|
|
|
bool HasTrailingCall = false;
|
|
|
|
if (Previous.MatchingParen) {
|
|
|
|
const FormatToken *Next = Previous.MatchingParen->getNextNonComment();
|
|
|
|
HasTrailingCall = Next && Next->isMemberAccess();
|
|
|
|
}
|
|
|
|
if (HasTrailingCall &&
|
|
|
|
State.Stack[State.Stack.size() - 2].CallContinuation == 0)
|
|
|
|
State.Stack.back().LastSpace = State.Column;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
unsigned ContinuationIndenter::addTokenOnNewLine(LineState &State,
|
|
|
|
bool DryRun) {
|
2013-10-12 05:25:45 +08:00
|
|
|
FormatToken &Current = *State.NextToken;
|
2013-10-01 22:41:18 +08:00
|
|
|
const FormatToken &Previous = *State.NextToken->Previous;
|
2013-10-18 18:38:14 +08:00
|
|
|
// If we are continuing an expression, we want to use the continuation indent.
|
2013-08-16 19:20:30 +08:00
|
|
|
unsigned ContinuationIndent =
|
2013-10-18 18:38:14 +08:00
|
|
|
std::max(State.Stack.back().LastSpace, State.Stack.back().Indent) +
|
|
|
|
Style.ContinuationIndentWidth;
|
2013-10-01 22:41:18 +08:00
|
|
|
// Extra penalty that needs to be added because of the way certain line
|
|
|
|
// breaks are chosen.
|
|
|
|
unsigned Penalty = 0;
|
|
|
|
|
2014-02-11 18:08:11 +08:00
|
|
|
const FormatToken *PreviousNonComment = Current.getPreviousNonComment();
|
|
|
|
const FormatToken *NextNonComment = Previous.getNextNonComment();
|
|
|
|
if (!NextNonComment)
|
|
|
|
NextNonComment = &Current;
|
2013-10-01 22:41:18 +08:00
|
|
|
// The first line break on any ParenLevel causes an extra penalty in order
|
|
|
|
// prefer similar line breaks.
|
|
|
|
if (!State.Stack.back().ContainsLineBreak)
|
|
|
|
Penalty += 15;
|
|
|
|
State.Stack.back().ContainsLineBreak = true;
|
|
|
|
|
|
|
|
Penalty += State.NextToken->SplitPenalty;
|
|
|
|
|
2014-02-11 18:08:11 +08:00
|
|
|
|
2013-10-01 22:41:18 +08:00
|
|
|
// Breaking before the first "<<" is generally not desirable if the LHS is
|
2013-11-20 22:54:39 +08:00
|
|
|
// short. Also always add the penalty if the LHS is split over mutliple lines
|
|
|
|
// to avoid unncessary line breaks that just work around this penalty.
|
2014-02-11 18:08:11 +08:00
|
|
|
if (NextNonComment->is(tok::lessless) &&
|
|
|
|
State.Stack.back().FirstLessLess == 0 &&
|
2013-12-20 00:06:40 +08:00
|
|
|
(State.Column <= Style.ColumnLimit / 3 ||
|
2013-11-20 22:54:39 +08:00
|
|
|
State.Stack.back().BreakBeforeParameter))
|
2013-10-01 22:41:18 +08:00
|
|
|
Penalty += Style.PenaltyBreakFirstLessLess;
|
|
|
|
|
2014-02-11 18:08:11 +08:00
|
|
|
if (NextNonComment->is(tok::l_brace) &&
|
|
|
|
NextNonComment->BlockKind == BK_Block) {
|
2013-11-29 16:46:20 +08:00
|
|
|
State.Column =
|
|
|
|
State.ParenLevel == 0 ? State.FirstIndent : State.Stack.back().Indent;
|
2013-10-22 23:30:28 +08:00
|
|
|
} else if (Current.isOneOf(tok::r_brace, tok::r_square)) {
|
2013-11-07 22:02:28 +08:00
|
|
|
if (Current.closesBlockTypeList(Style) ||
|
|
|
|
(Current.MatchingParen &&
|
|
|
|
Current.MatchingParen->BlockKind == BK_BracedInit))
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.Stack[State.Stack.size() - 2].LastSpace;
|
|
|
|
else
|
2013-09-13 17:20:45 +08:00
|
|
|
State.Column = State.FirstIndent;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->isStringLiteral() &&
|
|
|
|
State.StartOfStringLiteral != 0) {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.StartOfStringLiteral;
|
|
|
|
State.Stack.back().BreakBeforeParameter = true;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->is(tok::lessless) &&
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().FirstLessLess != 0) {
|
|
|
|
State.Column = State.Stack.back().FirstLessLess;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->isMemberAccess()) {
|
2013-10-01 22:41:18 +08:00
|
|
|
if (State.Stack.back().CallContinuation == 0) {
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Column = ContinuationIndent;
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().CallContinuation = State.Column;
|
2013-08-16 19:20:30 +08:00
|
|
|
} else {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.Stack.back().CallContinuation;
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
2013-11-08 08:57:11 +08:00
|
|
|
} else if (State.Stack.back().QuestionColumn != 0 &&
|
2014-02-11 18:08:11 +08:00
|
|
|
(NextNonComment->Type == TT_ConditionalExpr ||
|
2013-11-08 08:57:11 +08:00
|
|
|
Previous.Type == TT_ConditionalExpr)) {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.Stack.back().QuestionColumn;
|
|
|
|
} else if (Previous.is(tok::comma) && State.Stack.back().VariablePos != 0) {
|
|
|
|
State.Column = State.Stack.back().VariablePos;
|
|
|
|
} else if ((PreviousNonComment &&
|
2014-01-29 04:13:43 +08:00
|
|
|
(PreviousNonComment->ClosesTemplateDeclaration ||
|
|
|
|
PreviousNonComment->Type == TT_AttributeParen)) ||
|
2014-02-11 18:08:11 +08:00
|
|
|
((NextNonComment->Type == TT_StartOfName ||
|
|
|
|
NextNonComment->is(tok::kw_operator)) &&
|
2013-10-01 22:41:18 +08:00
|
|
|
State.ParenLevel == 0 &&
|
|
|
|
(!Style.IndentFunctionDeclarationAfterType ||
|
|
|
|
State.Line->StartsDefinition))) {
|
2013-11-22 15:48:15 +08:00
|
|
|
State.Column =
|
|
|
|
std::max(State.Stack.back().LastSpace, State.Stack.back().Indent);
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->Type == TT_ObjCSelectorName) {
|
2013-12-23 15:29:06 +08:00
|
|
|
if (!State.Stack.back().ObjCSelectorNameFound) {
|
2014-02-11 18:08:11 +08:00
|
|
|
if (NextNonComment->LongestObjCSelectorName == 0) {
|
2013-12-23 15:29:06 +08:00
|
|
|
State.Column = State.Stack.back().Indent;
|
|
|
|
State.Stack.back().AlignColons = false;
|
|
|
|
} else {
|
|
|
|
State.Stack.back().ColonPos =
|
2014-02-11 18:08:11 +08:00
|
|
|
State.Stack.back().Indent + NextNonComment->LongestObjCSelectorName;
|
|
|
|
State.Column =
|
|
|
|
State.Stack.back().ColonPos - NextNonComment->ColumnWidth;
|
2013-12-23 15:29:06 +08:00
|
|
|
}
|
|
|
|
} else if (!State.Stack.back().AlignColons) {
|
|
|
|
State.Column = State.Stack.back().Indent;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (State.Stack.back().ColonPos > NextNonComment->ColumnWidth) {
|
|
|
|
State.Column = State.Stack.back().ColonPos - NextNonComment->ColumnWidth;
|
2013-10-01 22:41:18 +08:00
|
|
|
} else {
|
|
|
|
State.Column = State.Stack.back().Indent;
|
2014-02-11 18:08:11 +08:00
|
|
|
State.Stack.back().ColonPos = State.Column + NextNonComment->ColumnWidth;
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->Type == TT_ArraySubscriptLSquare) {
|
2013-10-01 22:41:18 +08:00
|
|
|
if (State.Stack.back().StartOfArraySubscripts != 0)
|
|
|
|
State.Column = State.Stack.back().StartOfArraySubscripts;
|
|
|
|
else
|
|
|
|
State.Column = ContinuationIndent;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->Type == TT_StartOfName ||
|
2013-12-23 15:29:06 +08:00
|
|
|
Previous.isOneOf(tok::coloncolon, tok::equal)) {
|
|
|
|
State.Column = ContinuationIndent;
|
|
|
|
} else if (PreviousNonComment &&
|
|
|
|
PreviousNonComment->Type == TT_ObjCMethodExpr) {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = ContinuationIndent;
|
2013-12-23 15:29:06 +08:00
|
|
|
// FIXME: This is hacky, find a better way. The problem is that in an ObjC
|
|
|
|
// method expression, the block should be aligned to the line starting it,
|
|
|
|
// e.g.:
|
|
|
|
// [aaaaaaaaaaaaaaa aaaaaaaaa: \\ break for some reason
|
|
|
|
// ^(int *i) {
|
|
|
|
// // ...
|
|
|
|
// }];
|
|
|
|
// Thus, we set LastSpace of the next higher ParenLevel, to which we move
|
|
|
|
// when we consume all of the "}"'s FakeRParens at the "{".
|
2013-12-23 19:25:40 +08:00
|
|
|
if (State.Stack.size() > 1)
|
|
|
|
State.Stack[State.Stack.size() - 2].LastSpace = ContinuationIndent;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->Type == TT_CtorInitializerColon) {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.FirstIndent + Style.ConstructorInitializerIndentWidth;
|
2014-02-11 18:08:11 +08:00
|
|
|
} else if (NextNonComment->Type == TT_CtorInitializerComma) {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.Stack.back().Indent;
|
2013-08-16 19:20:30 +08:00
|
|
|
} else {
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Column = State.Stack.back().Indent;
|
2013-10-18 18:38:14 +08:00
|
|
|
// Ensure that we fall back to the continuation indent width instead of just
|
2013-10-01 22:41:18 +08:00
|
|
|
// flushing continuations left.
|
2013-10-30 22:04:10 +08:00
|
|
|
if (State.Column == State.FirstIndent &&
|
|
|
|
PreviousNonComment->isNot(tok::r_brace))
|
2013-10-18 18:38:14 +08:00
|
|
|
State.Column += Style.ContinuationIndentWidth;
|
2013-10-01 22:41:18 +08:00
|
|
|
}
|
2013-08-16 19:20:30 +08:00
|
|
|
|
2013-10-01 22:41:18 +08:00
|
|
|
if ((Previous.isOneOf(tok::comma, tok::semi) &&
|
|
|
|
!State.Stack.back().AvoidBinPacking) ||
|
|
|
|
Previous.Type == TT_BinaryOperator)
|
|
|
|
State.Stack.back().BreakBeforeParameter = false;
|
|
|
|
if (Previous.Type == TT_TemplateCloser && State.ParenLevel == 0)
|
|
|
|
State.Stack.back().BreakBeforeParameter = false;
|
2014-02-11 18:08:11 +08:00
|
|
|
if (NextNonComment->is(tok::question) ||
|
2013-11-08 08:57:11 +08:00
|
|
|
(PreviousNonComment && PreviousNonComment->is(tok::question)))
|
|
|
|
State.Stack.back().BreakBeforeParameter = true;
|
2013-10-01 22:41:18 +08:00
|
|
|
|
|
|
|
if (!DryRun) {
|
|
|
|
unsigned Newlines = 1;
|
|
|
|
if (Current.is(tok::comment))
|
|
|
|
Newlines = std::max(Newlines, std::min(Current.NewlinesBefore,
|
|
|
|
Style.MaxEmptyLinesToKeep + 1));
|
2013-10-14 08:46:35 +08:00
|
|
|
Whitespaces.replaceWhitespace(Current, Newlines,
|
|
|
|
State.Stack.back().IndentLevel, State.Column,
|
|
|
|
State.Column, State.Line->InPPDirective);
|
2013-10-01 22:41:18 +08:00
|
|
|
}
|
2013-08-16 19:20:30 +08:00
|
|
|
|
2013-10-01 22:41:18 +08:00
|
|
|
if (!Current.isTrailingComment())
|
|
|
|
State.Stack.back().LastSpace = State.Column;
|
|
|
|
State.StartOfLineLevel = State.ParenLevel;
|
|
|
|
State.LowestLevelOnLine = State.ParenLevel;
|
|
|
|
|
|
|
|
// Any break on this level means that the parent level has been broken
|
|
|
|
// and we need to avoid bin packing there.
|
|
|
|
for (unsigned i = 0, e = State.Stack.size() - 1; i != e; ++i) {
|
|
|
|
State.Stack[i].BreakBeforeParameter = true;
|
|
|
|
}
|
2013-11-09 03:56:28 +08:00
|
|
|
if (PreviousNonComment &&
|
|
|
|
!PreviousNonComment->isOneOf(tok::comma, tok::semi) &&
|
|
|
|
PreviousNonComment->Type != TT_TemplateCloser &&
|
|
|
|
PreviousNonComment->Type != TT_BinaryOperator &&
|
2013-12-23 15:29:06 +08:00
|
|
|
Current.Type != TT_BinaryOperator &&
|
2013-11-09 03:56:28 +08:00
|
|
|
!PreviousNonComment->opensScope())
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().BreakBeforeParameter = true;
|
|
|
|
|
2013-10-22 23:30:28 +08:00
|
|
|
// If we break after { or the [ of an array initializer, we should also break
|
|
|
|
// before the corresponding } or ].
|
|
|
|
if (Previous.is(tok::l_brace) || Previous.Type == TT_ArrayInitializerLSquare)
|
2013-10-01 22:41:18 +08:00
|
|
|
State.Stack.back().BreakBeforeClosingBrace = true;
|
|
|
|
|
|
|
|
if (State.Stack.back().AvoidBinPacking) {
|
|
|
|
// If we are breaking after '(', '{', '<', this is not bin packing
|
|
|
|
// unless AllowAllParametersOfDeclarationOnNextLine is false.
|
|
|
|
if (!(Previous.isOneOf(tok::l_paren, tok::l_brace) ||
|
|
|
|
Previous.Type == TT_BinaryOperator) ||
|
|
|
|
(!Style.AllowAllParametersOfDeclarationOnNextLine &&
|
|
|
|
State.Line->MustBeDeclaration))
|
|
|
|
State.Stack.back().BreakBeforeParameter = true;
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
|
2013-10-01 22:41:18 +08:00
|
|
|
return Penalty;
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
unsigned ContinuationIndenter::moveStateToNextToken(LineState &State,
|
|
|
|
bool DryRun, bool Newline) {
|
|
|
|
const FormatToken &Current = *State.NextToken;
|
|
|
|
assert(State.Stack.size());
|
|
|
|
|
|
|
|
if (Current.Type == TT_InheritanceColon)
|
|
|
|
State.Stack.back().AvoidBinPacking = true;
|
|
|
|
if (Current.is(tok::lessless) && State.Stack.back().FirstLessLess == 0)
|
|
|
|
State.Stack.back().FirstLessLess = State.Column;
|
2013-10-22 23:30:28 +08:00
|
|
|
if (Current.Type == TT_ArraySubscriptLSquare &&
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.back().StartOfArraySubscripts == 0)
|
|
|
|
State.Stack.back().StartOfArraySubscripts = State.Column;
|
2013-11-08 08:57:11 +08:00
|
|
|
if ((Current.is(tok::question) && Style.BreakBeforeTernaryOperators) ||
|
|
|
|
(Current.getPreviousNonComment() && Current.isNot(tok::colon) &&
|
|
|
|
Current.getPreviousNonComment()->is(tok::question) &&
|
|
|
|
!Style.BreakBeforeTernaryOperators))
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.back().QuestionColumn = State.Column;
|
|
|
|
if (!Current.opensScope() && !Current.closesScope())
|
|
|
|
State.LowestLevelOnLine =
|
|
|
|
std::min(State.LowestLevelOnLine, State.ParenLevel);
|
clang-format: Format segments of builder-type calls one per line.
This fixes llvm.org/PR14818.
Before:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT).StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH).Default(ORDER_TEXT);
After:
return llvm::StringSwitch<Reference::Kind>(name)
.StartsWith(".eh_frame_hdr", ORDER_EH_FRAMEHDR)
.StartsWith(".eh_frame", ORDER_EH_FRAME)
.StartsWith(".init", ORDER_INIT)
.StartsWith(".fini", ORDER_FINI)
.StartsWith(".hash", ORDER_HASH)
.Default(ORDER_TEXT);
llvm-svn: 189353
2013-08-27 22:24:43 +08:00
|
|
|
if (Current.isMemberAccess())
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.back().StartOfFunctionCall =
|
2013-09-10 17:38:25 +08:00
|
|
|
Current.LastInChainOfCalls ? 0 : State.Column + Current.ColumnWidth;
|
2013-12-23 15:29:06 +08:00
|
|
|
if (Current.Type == TT_ObjCSelectorName)
|
|
|
|
State.Stack.back().ObjCSelectorNameFound = true;
|
2013-08-16 19:20:30 +08:00
|
|
|
if (Current.Type == TT_CtorInitializerColon) {
|
|
|
|
// Indent 2 from the column, so:
|
|
|
|
// SomeClass::SomeClass()
|
|
|
|
// : First(...), ...
|
|
|
|
// Next(...)
|
|
|
|
// ^ line up here.
|
|
|
|
State.Stack.back().Indent =
|
|
|
|
State.Column + (Style.BreakConstructorInitializersBeforeComma ? 0 : 2);
|
|
|
|
if (Style.ConstructorInitializerAllOnOneLineOrOnePerLine)
|
|
|
|
State.Stack.back().AvoidBinPacking = true;
|
|
|
|
State.Stack.back().BreakBeforeParameter = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
// In ObjC method declaration we align on the ":" of parameters, but we need
|
2013-10-18 18:38:14 +08:00
|
|
|
// to ensure that we indent parameters on subsequent lines by at least our
|
|
|
|
// continuation indent width.
|
2013-08-16 19:20:30 +08:00
|
|
|
if (Current.Type == TT_ObjCMethodSpecifier)
|
2013-10-18 18:38:14 +08:00
|
|
|
State.Stack.back().Indent += Style.ContinuationIndentWidth;
|
2013-08-16 19:20:30 +08:00
|
|
|
|
|
|
|
// Insert scopes created by fake parenthesis.
|
|
|
|
const FormatToken *Previous = Current.getPreviousNonComment();
|
|
|
|
// Don't add extra indentation for the first fake parenthesis after
|
|
|
|
// 'return', assignements or opening <({[. The indentation for these cases
|
|
|
|
// is special cased.
|
|
|
|
bool SkipFirstExtraIndent =
|
2013-09-30 16:29:03 +08:00
|
|
|
(Previous && (Previous->opensScope() || Previous->is(tok::kw_return) ||
|
clang-format: Improve ObjC variadic and binary expression parameters.
Before:
[self aaaaaaaaaaaaaaa:aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa |
aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa |
aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa];
[self aaaaaaaaaaaaaaa:aaaaaaaaaaaaaaa,
aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa,
aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa];
After:
[self aaaaaaaaaaaaaaa:aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa |
aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa |
aaaaaaaaaaaaaaa | aaaaaaaaaaaaaaa];
[self aaaaaaaaaaaaaaa:aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa,
aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa,
aaaaaaaaaaaaaaa, aaaaaaaaaaaaaaa];
This addresses llvm.org/PR15349 and llvm.org/PR16185.
llvm-svn: 194214
2013-11-08 03:23:49 +08:00
|
|
|
Previous->getPrecedence() == prec::Assignment ||
|
|
|
|
Previous->Type == TT_ObjCMethodExpr));
|
2013-08-16 19:20:30 +08:00
|
|
|
for (SmallVectorImpl<prec::Level>::const_reverse_iterator
|
|
|
|
I = Current.FakeLParens.rbegin(),
|
|
|
|
E = Current.FakeLParens.rend();
|
|
|
|
I != E; ++I) {
|
|
|
|
ParenState NewParenState = State.Stack.back();
|
|
|
|
NewParenState.ContainsLineBreak = false;
|
2013-09-30 16:29:03 +08:00
|
|
|
|
|
|
|
// Indent from 'LastSpace' unless this the fake parentheses encapsulating a
|
|
|
|
// builder type call after 'return'. If such a call is line-wrapped, we
|
|
|
|
// commonly just want to indent from the start of the line.
|
|
|
|
if (!Previous || Previous->isNot(tok::kw_return) || *I > 0)
|
|
|
|
NewParenState.Indent =
|
|
|
|
std::max(std::max(State.Column, NewParenState.Indent),
|
|
|
|
State.Stack.back().LastSpace);
|
|
|
|
|
2013-09-05 17:29:45 +08:00
|
|
|
// Do not indent relative to the fake parentheses inserted for "." or "->".
|
|
|
|
// This is a special case to make the following to statements consistent:
|
|
|
|
// OuterFunction(InnerFunctionCall( // break
|
|
|
|
// ParameterToInnerFunction));
|
|
|
|
// OuterFunction(SomeObject.InnerFunctionCall( // break
|
|
|
|
// ParameterToInnerFunction));
|
|
|
|
if (*I > prec::Unknown)
|
|
|
|
NewParenState.LastSpace = std::max(NewParenState.LastSpace, State.Column);
|
2013-12-18 18:44:36 +08:00
|
|
|
NewParenState.StartOfFunctionCall = State.Column;
|
2013-08-16 19:20:30 +08:00
|
|
|
|
|
|
|
// Always indent conditional expressions. Never indent expression where
|
|
|
|
// the 'operator' is ',', ';' or an assignment (i.e. *I <=
|
|
|
|
// prec::Assignment) as those have different indentation rules. Indent
|
|
|
|
// other expression, unless the indentation needs to be skipped.
|
|
|
|
if (*I == prec::Conditional ||
|
|
|
|
(!SkipFirstExtraIndent && *I > prec::Assignment &&
|
|
|
|
!Style.BreakBeforeBinaryOperators))
|
2013-10-18 18:38:14 +08:00
|
|
|
NewParenState.Indent += Style.ContinuationIndentWidth;
|
2013-10-22 23:30:28 +08:00
|
|
|
if ((Previous && !Previous->opensScope()) || *I > prec::Comma)
|
2013-08-16 19:20:30 +08:00
|
|
|
NewParenState.BreakBeforeParameter = false;
|
|
|
|
State.Stack.push_back(NewParenState);
|
|
|
|
SkipFirstExtraIndent = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If we encounter an opening (, [, { or <, we add a level to our stacks to
|
|
|
|
// prepare for the following tokens.
|
|
|
|
if (Current.opensScope()) {
|
|
|
|
unsigned NewIndent;
|
2013-10-14 08:46:35 +08:00
|
|
|
unsigned NewIndentLevel = State.Stack.back().IndentLevel;
|
2013-08-16 19:20:30 +08:00
|
|
|
bool AvoidBinPacking;
|
2013-10-22 23:30:28 +08:00
|
|
|
bool BreakBeforeParameter = false;
|
|
|
|
if (Current.is(tok::l_brace) ||
|
|
|
|
Current.Type == TT_ArrayInitializerLSquare) {
|
2013-09-05 17:29:45 +08:00
|
|
|
if (Current.MatchingParen && Current.BlockKind == BK_Block) {
|
2013-09-05 18:48:50 +08:00
|
|
|
// If this is an l_brace starting a nested block, we pretend (wrt. to
|
|
|
|
// indentation) that we already consumed the corresponding r_brace.
|
2013-12-18 18:44:36 +08:00
|
|
|
// Thus, we remove all ParenStates caused by fake parentheses that end
|
2013-09-05 18:48:50 +08:00
|
|
|
// at the r_brace. The net effect of this is that we don't indent
|
|
|
|
// relative to the l_brace, if the nested block is the last parameter of
|
|
|
|
// a function. For example, this formats:
|
|
|
|
//
|
|
|
|
// SomeFunction(a, [] {
|
|
|
|
// f(); // break
|
|
|
|
// });
|
|
|
|
//
|
|
|
|
// instead of:
|
|
|
|
// SomeFunction(a, [] {
|
2013-11-25 19:08:59 +08:00
|
|
|
// f(); // break
|
|
|
|
// });
|
2013-09-05 17:29:45 +08:00
|
|
|
for (unsigned i = 0; i != Current.MatchingParen->FakeRParens; ++i)
|
|
|
|
State.Stack.pop_back();
|
2013-12-23 15:29:06 +08:00
|
|
|
bool IsObjCBlock =
|
|
|
|
Previous &&
|
|
|
|
(Previous->is(tok::caret) ||
|
|
|
|
(Previous->is(tok::r_paren) && Previous->MatchingParen &&
|
|
|
|
Previous->MatchingParen->Previous &&
|
|
|
|
Previous->MatchingParen->Previous->is(tok::caret)));
|
|
|
|
// For some reason, ObjC blocks are indented like continuations.
|
|
|
|
NewIndent =
|
|
|
|
State.Stack.back().LastSpace +
|
|
|
|
(IsObjCBlock ? Style.ContinuationIndentWidth : Style.IndentWidth);
|
2013-10-14 08:46:35 +08:00
|
|
|
++NewIndentLevel;
|
2013-10-22 23:30:28 +08:00
|
|
|
BreakBeforeParameter = true;
|
2013-09-05 17:29:45 +08:00
|
|
|
} else {
|
2013-10-14 08:46:35 +08:00
|
|
|
NewIndent = State.Stack.back().LastSpace;
|
2013-10-22 23:45:58 +08:00
|
|
|
if (Current.opensBlockTypeList(Style)) {
|
2013-10-14 08:46:35 +08:00
|
|
|
NewIndent += Style.IndentWidth;
|
2013-12-20 05:41:37 +08:00
|
|
|
NewIndent = std::min(State.Column + 2, NewIndent);
|
2013-10-14 08:46:35 +08:00
|
|
|
++NewIndentLevel;
|
2013-10-22 23:45:58 +08:00
|
|
|
} else {
|
|
|
|
NewIndent += Style.ContinuationIndentWidth;
|
2013-12-20 05:41:37 +08:00
|
|
|
NewIndent = std::min(State.Column + 1, NewIndent);
|
2013-10-14 08:46:35 +08:00
|
|
|
}
|
2013-09-05 17:29:45 +08:00
|
|
|
}
|
2013-08-16 19:20:30 +08:00
|
|
|
const FormatToken *NextNoComment = Current.getNextNonComment();
|
2013-09-13 17:20:45 +08:00
|
|
|
AvoidBinPacking = Current.BlockKind == BK_Block ||
|
2013-10-22 23:30:28 +08:00
|
|
|
Current.Type == TT_ArrayInitializerLSquare ||
|
2013-10-24 18:31:50 +08:00
|
|
|
Current.Type == TT_DictLiteral ||
|
2013-09-13 17:20:45 +08:00
|
|
|
(NextNoComment &&
|
|
|
|
NextNoComment->Type == TT_DesignatedInitializerPeriod);
|
2013-08-16 19:20:30 +08:00
|
|
|
} else {
|
2013-10-18 18:38:14 +08:00
|
|
|
NewIndent = Style.ContinuationIndentWidth +
|
|
|
|
std::max(State.Stack.back().LastSpace,
|
|
|
|
State.Stack.back().StartOfFunctionCall);
|
2013-08-16 19:20:30 +08:00
|
|
|
AvoidBinPacking = !Style.BinPackParameters ||
|
|
|
|
(Style.ExperimentalAutoDetectBinPacking &&
|
|
|
|
(Current.PackingKind == PPK_OnePerLine ||
|
|
|
|
(!BinPackInconclusiveFunctions &&
|
|
|
|
Current.PackingKind == PPK_Inconclusive)));
|
2013-10-22 23:30:28 +08:00
|
|
|
// If this '[' opens an ObjC call, determine whether all parameters fit
|
|
|
|
// into one line and put one per line if they don't.
|
|
|
|
if (Current.Type == TT_ObjCMethodExpr &&
|
|
|
|
getLengthToMatchingParen(Current) + State.Column >
|
|
|
|
getColumnLimit(State))
|
|
|
|
BreakBeforeParameter = true;
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
|
2013-10-18 23:23:06 +08:00
|
|
|
bool NoLineBreak = State.Stack.back().NoLineBreak ||
|
|
|
|
(Current.Type == TT_TemplateOpener &&
|
|
|
|
State.Stack.back().ContainsUnwrappedBuilder);
|
|
|
|
State.Stack.push_back(ParenState(NewIndent, NewIndentLevel,
|
|
|
|
State.Stack.back().LastSpace,
|
|
|
|
AvoidBinPacking, NoLineBreak));
|
2013-10-22 23:30:28 +08:00
|
|
|
State.Stack.back().BreakBeforeParameter = BreakBeforeParameter;
|
2013-08-16 19:20:30 +08:00
|
|
|
++State.ParenLevel;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If we encounter a closing ), ], } or >, we can remove a level from our
|
|
|
|
// stacks.
|
2013-08-28 17:17:37 +08:00
|
|
|
if (State.Stack.size() > 1 &&
|
|
|
|
(Current.isOneOf(tok::r_paren, tok::r_square) ||
|
2013-09-05 17:29:45 +08:00
|
|
|
(Current.is(tok::r_brace) && State.NextToken != State.Line->First) ||
|
2013-08-28 17:17:37 +08:00
|
|
|
State.NextToken->Type == TT_TemplateCloser)) {
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.pop_back();
|
|
|
|
--State.ParenLevel;
|
|
|
|
}
|
|
|
|
if (Current.is(tok::r_square)) {
|
|
|
|
// If this ends the array subscript expr, reset the corresponding value.
|
|
|
|
const FormatToken *NextNonComment = Current.getNextNonComment();
|
|
|
|
if (NextNonComment && NextNonComment->isNot(tok::l_square))
|
|
|
|
State.Stack.back().StartOfArraySubscripts = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Remove scopes created by fake parenthesis.
|
2013-09-05 17:29:45 +08:00
|
|
|
if (Current.isNot(tok::r_brace) ||
|
|
|
|
(Current.MatchingParen && Current.MatchingParen->BlockKind != BK_Block)) {
|
2013-09-05 18:48:50 +08:00
|
|
|
// Don't remove FakeRParens attached to r_braces that surround nested blocks
|
|
|
|
// as they will have been removed early (see above).
|
2013-09-05 17:29:45 +08:00
|
|
|
for (unsigned i = 0, e = Current.FakeRParens; i != e; ++i) {
|
|
|
|
unsigned VariablePos = State.Stack.back().VariablePos;
|
|
|
|
State.Stack.pop_back();
|
|
|
|
State.Stack.back().VariablePos = VariablePos;
|
|
|
|
}
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
|
2013-12-20 14:22:01 +08:00
|
|
|
if (Current.isStringLiteral() && State.StartOfStringLiteral == 0) {
|
2013-08-16 19:20:30 +08:00
|
|
|
State.StartOfStringLiteral = State.Column;
|
2013-12-20 14:22:01 +08:00
|
|
|
} else if (!Current.isOneOf(tok::comment, tok::identifier, tok::hash) &&
|
|
|
|
!Current.isStringLiteral()) {
|
2013-08-16 19:20:30 +08:00
|
|
|
State.StartOfStringLiteral = 0;
|
|
|
|
}
|
|
|
|
|
2013-09-10 17:38:25 +08:00
|
|
|
State.Column += Current.ColumnWidth;
|
2013-08-16 19:20:30 +08:00
|
|
|
State.NextToken = State.NextToken->Next;
|
clang-format: Revamp builder-type call formatting.
Previously builder-type calls were only correctly recognized in
top-level calls.
This fixes llvm.org/PR16981.
Before:
someobj->Add((new util::filetools::Handler(dir))->OnEvent1(
NewPermanentCallback(this, &HandlerHolderClass::EventHandlerCBA))
->OnEvent2(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBB))
->OnEvent3(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBC))
->OnEvent5(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBD))
->OnEvent6(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBE)));
After:
someobj->Add((new util::filetools::Handler(dir))
->OnEvent1(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBA))
->OnEvent2(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBB))
->OnEvent3(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBC))
->OnEvent5(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBD))
->OnEvent6(NewPermanentCallback(
this, &HandlerHolderClass::EventHandlerCBE)));
llvm-svn: 189337
2013-08-27 19:09:05 +08:00
|
|
|
unsigned Penalty = breakProtrudingToken(Current, State, DryRun);
|
2013-09-05 17:29:45 +08:00
|
|
|
if (State.Column > getColumnLimit(State)) {
|
|
|
|
unsigned ExcessCharacters = State.Column - getColumnLimit(State);
|
|
|
|
Penalty += Style.PenaltyExcessCharacter * ExcessCharacters;
|
|
|
|
}
|
clang-format: Add column layout formatting for braced lists
With this patch, braced lists (with more than 3 elements are formatted in a
column layout if possible). E.g.:
static const uint16_t CallerSavedRegs64Bit[] = {
X86::RAX, X86::RDX, X86::RCX, X86::RSI, X86::RDI,
X86::R8, X86::R9, X86::R10, X86::R11, 0
};
Required other changes:
- FormatTokens can now have a special role that contains extra data and can do
special formattings. A comma separated list is currently the only
implementation.
- Move penalty calculation entirely into ContinuationIndenter (there was a last
piece still in UnwrappedLineFormatter).
Review: http://llvm-reviews.chandlerc.com/D1457
llvm-svn: 189018
2013-08-22 23:00:41 +08:00
|
|
|
|
2014-01-09 21:42:56 +08:00
|
|
|
if (Current.Role)
|
|
|
|
Current.Role->formatFromToken(State, this, DryRun);
|
clang-format: Add column layout formatting for braced lists
With this patch, braced lists (with more than 3 elements are formatted in a
column layout if possible). E.g.:
static const uint16_t CallerSavedRegs64Bit[] = {
X86::RAX, X86::RDX, X86::RCX, X86::RSI, X86::RDI,
X86::R8, X86::R9, X86::R10, X86::R11, 0
};
Required other changes:
- FormatTokens can now have a special role that contains extra data and can do
special formattings. A comma separated list is currently the only
implementation.
- Move penalty calculation entirely into ContinuationIndenter (there was a last
piece still in UnwrappedLineFormatter).
Review: http://llvm-reviews.chandlerc.com/D1457
llvm-svn: 189018
2013-08-22 23:00:41 +08:00
|
|
|
// If the previous has a special role, let it consume tokens as appropriate.
|
|
|
|
// It is necessary to start at the previous token for the only implemented
|
|
|
|
// role (comma separated list). That way, the decision whether or not to break
|
|
|
|
// after the "{" is already done and both options are tried and evaluated.
|
|
|
|
// FIXME: This is ugly, find a better way.
|
|
|
|
if (Previous && Previous->Role)
|
2014-01-09 21:42:56 +08:00
|
|
|
Penalty += Previous->Role->formatAfterToken(State, this, DryRun);
|
2013-08-16 19:20:30 +08:00
|
|
|
|
clang-format: Add column layout formatting for braced lists
With this patch, braced lists (with more than 3 elements are formatted in a
column layout if possible). E.g.:
static const uint16_t CallerSavedRegs64Bit[] = {
X86::RAX, X86::RDX, X86::RCX, X86::RSI, X86::RDI,
X86::R8, X86::R9, X86::R10, X86::R11, 0
};
Required other changes:
- FormatTokens can now have a special role that contains extra data and can do
special formattings. A comma separated list is currently the only
implementation.
- Move penalty calculation entirely into ContinuationIndenter (there was a last
piece still in UnwrappedLineFormatter).
Review: http://llvm-reviews.chandlerc.com/D1457
llvm-svn: 189018
2013-08-22 23:00:41 +08:00
|
|
|
return Penalty;
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
|
2013-09-10 20:29:48 +08:00
|
|
|
unsigned ContinuationIndenter::addMultilineToken(const FormatToken &Current,
|
|
|
|
LineState &State) {
|
2013-08-30 01:32:57 +08:00
|
|
|
// Break before further function parameters on all levels.
|
|
|
|
for (unsigned i = 0, e = State.Stack.size(); i != e; ++i)
|
|
|
|
State.Stack[i].BreakBeforeParameter = true;
|
|
|
|
|
2013-09-10 17:38:25 +08:00
|
|
|
unsigned ColumnsUsed = State.Column;
|
2013-09-02 21:58:14 +08:00
|
|
|
// We can only affect layout of the first and the last line, so the penalty
|
|
|
|
// for all other lines is constant, and we ignore it.
|
2013-09-05 22:08:34 +08:00
|
|
|
State.Column = Current.LastLineColumnWidth;
|
2013-09-02 21:58:14 +08:00
|
|
|
|
2013-09-05 17:29:45 +08:00
|
|
|
if (ColumnsUsed > getColumnLimit(State))
|
|
|
|
return Style.PenaltyExcessCharacter * (ColumnsUsed - getColumnLimit(State));
|
2013-08-30 01:32:57 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-09-17 04:20:49 +08:00
|
|
|
static bool getRawStringLiteralPrefixPostfix(StringRef Text,
|
|
|
|
StringRef &Prefix,
|
|
|
|
StringRef &Postfix) {
|
|
|
|
if (Text.startswith(Prefix = "R\"") || Text.startswith(Prefix = "uR\"") ||
|
|
|
|
Text.startswith(Prefix = "UR\"") || Text.startswith(Prefix = "u8R\"") ||
|
|
|
|
Text.startswith(Prefix = "LR\"")) {
|
|
|
|
size_t ParenPos = Text.find('(');
|
|
|
|
if (ParenPos != StringRef::npos) {
|
|
|
|
StringRef Delimiter =
|
|
|
|
Text.substr(Prefix.size(), ParenPos - Prefix.size());
|
|
|
|
Prefix = Text.substr(0, ParenPos + 1);
|
|
|
|
Postfix = Text.substr(Text.size() - 2 - Delimiter.size());
|
|
|
|
return Postfix.front() == ')' && Postfix.back() == '"' &&
|
|
|
|
Postfix.substr(1).startswith(Delimiter);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
unsigned ContinuationIndenter::breakProtrudingToken(const FormatToken &Current,
|
|
|
|
LineState &State,
|
|
|
|
bool DryRun) {
|
2013-09-10 20:29:48 +08:00
|
|
|
// Don't break multi-line tokens other than block comments. Instead, just
|
|
|
|
// update the state.
|
|
|
|
if (Current.Type != TT_BlockComment && Current.IsMultiline)
|
|
|
|
return addMultilineToken(Current, State);
|
|
|
|
|
2013-10-30 21:54:53 +08:00
|
|
|
// Don't break implicit string literals.
|
|
|
|
if (Current.Type == TT_ImplicitStringLiteral)
|
|
|
|
return 0;
|
|
|
|
|
2013-12-20 14:22:01 +08:00
|
|
|
if (!Current.isStringLiteral() && !Current.is(tok::comment))
|
2013-08-23 18:05:49 +08:00
|
|
|
return 0;
|
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
llvm::OwningPtr<BreakableToken> Token;
|
2013-09-10 17:38:25 +08:00
|
|
|
unsigned StartColumn = State.Column - Current.ColumnWidth;
|
2013-11-13 01:30:49 +08:00
|
|
|
unsigned ColumnLimit = getColumnLimit(State);
|
2013-08-16 19:20:30 +08:00
|
|
|
|
2013-12-20 14:22:01 +08:00
|
|
|
if (Current.isStringLiteral()) {
|
2013-10-12 05:43:05 +08:00
|
|
|
// Don't break string literals inside preprocessor directives (except for
|
|
|
|
// #define directives, as their contents are stored in separate lines and
|
|
|
|
// are not affected by this check).
|
|
|
|
// This way we avoid breaking code with line directives and unknown
|
|
|
|
// preprocessor directives that contain long string literals.
|
|
|
|
if (State.Line->Type == LT_PreprocessorDirective)
|
|
|
|
return 0;
|
2013-08-16 19:20:30 +08:00
|
|
|
// Exempts unterminated string literals from line breaking. The user will
|
|
|
|
// likely want to terminate the string before any line breaking is done.
|
|
|
|
if (Current.IsUnterminatedLiteral)
|
|
|
|
return 0;
|
|
|
|
|
2013-09-17 04:20:49 +08:00
|
|
|
StringRef Text = Current.TokenText;
|
|
|
|
StringRef Prefix;
|
|
|
|
StringRef Postfix;
|
2014-01-09 22:18:12 +08:00
|
|
|
bool IsNSStringLiteral = false;
|
2013-09-17 04:20:49 +08:00
|
|
|
// FIXME: Handle whitespace between '_T', '(', '"..."', and ')'.
|
|
|
|
// FIXME: Store Prefix and Suffix (or PrefixLength and SuffixLength to
|
|
|
|
// reduce the overhead) for each FormatToken, which is a string, so that we
|
|
|
|
// don't run multiple checks here on the hot path.
|
2014-01-09 22:18:12 +08:00
|
|
|
if (Text.startswith("\"") && Current.Previous &&
|
|
|
|
Current.Previous->is(tok::at)) {
|
|
|
|
IsNSStringLiteral = true;
|
|
|
|
Prefix = "@\"";
|
|
|
|
}
|
2013-09-17 04:20:49 +08:00
|
|
|
if ((Text.endswith(Postfix = "\"") &&
|
2014-01-09 22:18:12 +08:00
|
|
|
(IsNSStringLiteral || Text.startswith(Prefix = "\"") ||
|
|
|
|
Text.startswith(Prefix = "u\"") || Text.startswith(Prefix = "U\"") ||
|
|
|
|
Text.startswith(Prefix = "u8\"") ||
|
2013-09-17 04:20:49 +08:00
|
|
|
Text.startswith(Prefix = "L\""))) ||
|
|
|
|
(Text.startswith(Prefix = "_T(\"") && Text.endswith(Postfix = "\")")) ||
|
|
|
|
getRawStringLiteralPrefixPostfix(Text, Prefix, Postfix)) {
|
2013-09-28 00:14:22 +08:00
|
|
|
Token.reset(new BreakableStringLiteral(
|
|
|
|
Current, State.Line->Level, StartColumn, Prefix, Postfix,
|
|
|
|
State.Line->InPPDirective, Encoding, Style));
|
2013-09-17 04:20:49 +08:00
|
|
|
} else {
|
|
|
|
return 0;
|
|
|
|
}
|
2013-08-16 19:20:30 +08:00
|
|
|
} else if (Current.Type == TT_BlockComment && Current.isTrailingComment()) {
|
2014-01-02 23:13:14 +08:00
|
|
|
if (CommentPragmasRegex.match(Current.TokenText.substr(2)))
|
|
|
|
return 0;
|
2013-08-16 19:20:30 +08:00
|
|
|
Token.reset(new BreakableBlockComment(
|
2013-09-28 00:14:22 +08:00
|
|
|
Current, State.Line->Level, StartColumn, Current.OriginalColumn,
|
|
|
|
!Current.Previous, State.Line->InPPDirective, Encoding, Style));
|
2013-08-16 19:20:30 +08:00
|
|
|
} else if (Current.Type == TT_LineComment &&
|
|
|
|
(Current.Previous == NULL ||
|
|
|
|
Current.Previous->Type != TT_ImplicitStringLiteral)) {
|
2014-01-02 23:13:14 +08:00
|
|
|
if (CommentPragmasRegex.match(Current.TokenText.substr(2)))
|
|
|
|
return 0;
|
2013-09-28 00:14:22 +08:00
|
|
|
Token.reset(new BreakableLineComment(Current, State.Line->Level,
|
2013-11-13 01:30:49 +08:00
|
|
|
StartColumn, /*InPPDirective=*/false,
|
2013-09-28 00:14:22 +08:00
|
|
|
Encoding, Style));
|
2013-11-13 01:30:49 +08:00
|
|
|
// We don't insert backslashes when breaking line comments.
|
|
|
|
ColumnLimit = Style.ColumnLimit;
|
2013-08-16 19:20:30 +08:00
|
|
|
} else {
|
|
|
|
return 0;
|
|
|
|
}
|
2013-11-13 01:30:49 +08:00
|
|
|
if (Current.UnbreakableTailLength >= ColumnLimit)
|
2013-08-16 19:20:30 +08:00
|
|
|
return 0;
|
|
|
|
|
2013-11-13 01:30:49 +08:00
|
|
|
unsigned RemainingSpace = ColumnLimit - Current.UnbreakableTailLength;
|
2013-08-16 19:20:30 +08:00
|
|
|
bool BreakInserted = false;
|
|
|
|
unsigned Penalty = 0;
|
|
|
|
unsigned RemainingTokenColumns = 0;
|
|
|
|
for (unsigned LineIndex = 0, EndIndex = Token->getLineCount();
|
|
|
|
LineIndex != EndIndex; ++LineIndex) {
|
|
|
|
if (!DryRun)
|
|
|
|
Token->replaceWhitespaceBefore(LineIndex, Whitespaces);
|
|
|
|
unsigned TailOffset = 0;
|
|
|
|
RemainingTokenColumns =
|
|
|
|
Token->getLineLengthAfterSplit(LineIndex, TailOffset, StringRef::npos);
|
|
|
|
while (RemainingTokenColumns > RemainingSpace) {
|
|
|
|
BreakableToken::Split Split =
|
2013-11-13 01:30:49 +08:00
|
|
|
Token->getSplit(LineIndex, TailOffset, ColumnLimit);
|
2013-08-16 19:20:30 +08:00
|
|
|
if (Split.first == StringRef::npos) {
|
|
|
|
// The last line's penalty is handled in addNextStateToQueue().
|
|
|
|
if (LineIndex < EndIndex - 1)
|
|
|
|
Penalty += Style.PenaltyExcessCharacter *
|
|
|
|
(RemainingTokenColumns - RemainingSpace);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
assert(Split.first != 0);
|
|
|
|
unsigned NewRemainingTokenColumns = Token->getLineLengthAfterSplit(
|
|
|
|
LineIndex, TailOffset + Split.first + Split.second, StringRef::npos);
|
2013-11-13 01:50:13 +08:00
|
|
|
|
|
|
|
// We can remove extra whitespace instead of breaking the line.
|
|
|
|
if (RemainingTokenColumns + 1 - Split.second <= RemainingSpace) {
|
|
|
|
RemainingTokenColumns = 0;
|
|
|
|
if (!DryRun)
|
|
|
|
Token->replaceWhitespace(LineIndex, TailOffset, Split, Whitespaces);
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
assert(NewRemainingTokenColumns < RemainingTokenColumns);
|
|
|
|
if (!DryRun)
|
|
|
|
Token->insertBreak(LineIndex, TailOffset, Split, Whitespaces);
|
2013-08-28 18:03:58 +08:00
|
|
|
Penalty += Current.SplitPenalty;
|
2013-08-16 19:20:30 +08:00
|
|
|
unsigned ColumnsUsed =
|
|
|
|
Token->getLineLengthAfterSplit(LineIndex, TailOffset, Split.first);
|
2013-11-13 01:30:49 +08:00
|
|
|
if (ColumnsUsed > ColumnLimit) {
|
|
|
|
Penalty += Style.PenaltyExcessCharacter * (ColumnsUsed - ColumnLimit);
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
TailOffset += Split.first + Split.second;
|
|
|
|
RemainingTokenColumns = NewRemainingTokenColumns;
|
|
|
|
BreakInserted = true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
State.Column = RemainingTokenColumns;
|
|
|
|
|
|
|
|
if (BreakInserted) {
|
|
|
|
// If we break the token inside a parameter list, we need to break before
|
|
|
|
// the next parameter on all levels, so that the next parameter is clearly
|
|
|
|
// visible. Line comments already introduce a break.
|
|
|
|
if (Current.Type != TT_LineComment) {
|
|
|
|
for (unsigned i = 0, e = State.Stack.size(); i != e; ++i)
|
|
|
|
State.Stack[i].BreakBeforeParameter = true;
|
|
|
|
}
|
|
|
|
|
2013-12-20 14:22:01 +08:00
|
|
|
Penalty += Current.isStringLiteral() ? Style.PenaltyBreakString
|
|
|
|
: Style.PenaltyBreakComment;
|
2013-08-28 18:03:58 +08:00
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
State.Stack.back().LastSpace = StartColumn;
|
|
|
|
}
|
|
|
|
return Penalty;
|
|
|
|
}
|
|
|
|
|
2013-09-05 17:29:45 +08:00
|
|
|
unsigned ContinuationIndenter::getColumnLimit(const LineState &State) const {
|
2013-08-16 19:20:30 +08:00
|
|
|
// In preprocessor directives reserve two chars for trailing " \"
|
2013-09-05 17:29:45 +08:00
|
|
|
return Style.ColumnLimit - (State.Line->InPPDirective ? 2 : 0);
|
2013-08-16 19:20:30 +08:00
|
|
|
}
|
|
|
|
|
2013-12-16 15:23:08 +08:00
|
|
|
bool ContinuationIndenter::nextIsMultilineString(const LineState &State) {
|
2013-08-23 19:57:34 +08:00
|
|
|
const FormatToken &Current = *State.NextToken;
|
2013-12-20 14:22:01 +08:00
|
|
|
if (!Current.isStringLiteral())
|
2013-08-23 19:57:34 +08:00
|
|
|
return false;
|
2013-08-30 01:32:57 +08:00
|
|
|
// We never consider raw string literals "multiline" for the purpose of
|
2013-12-16 15:23:08 +08:00
|
|
|
// AlwaysBreakBeforeMultilineStrings implementation as they are special-cased
|
|
|
|
// (see TokenAnnotator::mustBreakBefore().
|
2013-08-30 01:32:57 +08:00
|
|
|
if (Current.TokenText.startswith("R\""))
|
|
|
|
return false;
|
2013-09-10 17:38:25 +08:00
|
|
|
if (Current.IsMultiline)
|
2013-08-30 01:32:57 +08:00
|
|
|
return true;
|
2013-08-23 19:57:34 +08:00
|
|
|
if (Current.getNextNonComment() &&
|
2013-12-20 14:22:01 +08:00
|
|
|
Current.getNextNonComment()->isStringLiteral())
|
2013-08-23 19:57:34 +08:00
|
|
|
return true; // Implicit concatenation.
|
2013-09-10 17:38:25 +08:00
|
|
|
if (State.Column + Current.ColumnWidth + Current.UnbreakableTailLength >
|
2013-08-23 19:57:34 +08:00
|
|
|
Style.ColumnLimit)
|
|
|
|
return true; // String will be split.
|
2013-08-30 01:32:57 +08:00
|
|
|
return false;
|
2013-08-23 19:57:34 +08:00
|
|
|
}
|
|
|
|
|
2013-08-16 19:20:30 +08:00
|
|
|
} // namespace format
|
|
|
|
} // namespace clang
|