18de9ed05SDaniel Jasper //===--- FormatToken.cpp - Format C++ code --------------------------------===//
28de9ed05SDaniel Jasper //
32946cd70SChandler Carruth // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
42946cd70SChandler Carruth // See https://llvm.org/LICENSE.txt for license information.
52946cd70SChandler Carruth // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
68de9ed05SDaniel Jasper //
78de9ed05SDaniel Jasper //===----------------------------------------------------------------------===//
88de9ed05SDaniel Jasper ///
98de9ed05SDaniel Jasper /// \file
109fc8faf9SAdrian Prantl /// This file implements specific functions of \c FormatTokens and their
118de9ed05SDaniel Jasper /// roles.
128de9ed05SDaniel Jasper ///
138de9ed05SDaniel Jasper //===----------------------------------------------------------------------===//
148de9ed05SDaniel Jasper 
15d89ae9d3SDaniel Jasper #include "FormatToken.h"
169670f847SMehdi Amini #include "ContinuationIndenter.h"
178de9ed05SDaniel Jasper #include "llvm/ADT/SmallVector.h"
188de9ed05SDaniel Jasper #include "llvm/Support/Debug.h"
19d9ff035dSDaniel Jasper #include <climits>
208de9ed05SDaniel Jasper 
218de9ed05SDaniel Jasper namespace clang {
228de9ed05SDaniel Jasper namespace format {
238de9ed05SDaniel Jasper 
getTokenTypeName(TokenType Type)2467d81c8aSBirunthan Mohanathas const char *getTokenTypeName(TokenType Type) {
2567d81c8aSBirunthan Mohanathas   static const char *const TokNames[] = {
2667d81c8aSBirunthan Mohanathas #define TYPE(X) #X,
2767d81c8aSBirunthan Mohanathas       LIST_TOKEN_TYPES
2867d81c8aSBirunthan Mohanathas #undef TYPE
2989628f64SManuel Klimek       nullptr};
3067d81c8aSBirunthan Mohanathas 
3167d81c8aSBirunthan Mohanathas   if (Type < NUM_TOKEN_TYPES)
3267d81c8aSBirunthan Mohanathas     return TokNames[Type];
3367d81c8aSBirunthan Mohanathas   llvm_unreachable("unknown TokenType");
3467d81c8aSBirunthan Mohanathas   return nullptr;
3567d81c8aSBirunthan Mohanathas }
3667d81c8aSBirunthan Mohanathas 
37cb51cf40SDaniel Jasper // FIXME: This is copy&pasted from Sema. Put it in a common place and remove
38cb51cf40SDaniel Jasper // duplication.
isSimpleTypeSpecifier() const39cb51cf40SDaniel Jasper bool FormatToken::isSimpleTypeSpecifier() const {
40cb51cf40SDaniel Jasper   switch (Tok.getKind()) {
41cb51cf40SDaniel Jasper   case tok::kw_short:
42cb51cf40SDaniel Jasper   case tok::kw_long:
43cb51cf40SDaniel Jasper   case tok::kw___int64:
44cb51cf40SDaniel Jasper   case tok::kw___int128:
45cb51cf40SDaniel Jasper   case tok::kw_signed:
46cb51cf40SDaniel Jasper   case tok::kw_unsigned:
47cb51cf40SDaniel Jasper   case tok::kw_void:
48cb51cf40SDaniel Jasper   case tok::kw_char:
49cb51cf40SDaniel Jasper   case tok::kw_int:
50cb51cf40SDaniel Jasper   case tok::kw_half:
51cb51cf40SDaniel Jasper   case tok::kw_float:
52cb51cf40SDaniel Jasper   case tok::kw_double:
53ecd682bbSTies Stuij   case tok::kw___bf16:
54cc623ad0SSjoerd Meijer   case tok::kw__Float16:
55bb1ea2d6SNemanja Ivanovic   case tok::kw___float128:
56fae0dfa6SQiu Chaofan   case tok::kw___ibm128:
57cb51cf40SDaniel Jasper   case tok::kw_wchar_t:
58cb51cf40SDaniel Jasper   case tok::kw_bool:
59cb51cf40SDaniel Jasper   case tok::kw___underlying_type:
60cb51cf40SDaniel Jasper   case tok::annot_typename:
613a8244dfSRichard Smith   case tok::kw_char8_t:
62cb51cf40SDaniel Jasper   case tok::kw_char16_t:
63cb51cf40SDaniel Jasper   case tok::kw_char32_t:
64cb51cf40SDaniel Jasper   case tok::kw_typeof:
65cb51cf40SDaniel Jasper   case tok::kw_decltype:
6656fa7d1dSAlex Richardson   case tok::kw__Atomic:
67cb51cf40SDaniel Jasper     return true;
68cb51cf40SDaniel Jasper   default:
69cb51cf40SDaniel Jasper     return false;
70cb51cf40SDaniel Jasper   }
71cb51cf40SDaniel Jasper }
72cb51cf40SDaniel Jasper 
isTypeOrIdentifier() const732a73a1acSmydeveloperday bool FormatToken::isTypeOrIdentifier() const {
742a73a1acSmydeveloperday   return isSimpleTypeSpecifier() || Tok.isOneOf(tok::kw_auto, tok::identifier);
752a73a1acSmydeveloperday }
762a73a1acSmydeveloperday 
opensBlockOrBlockTypeList(const FormatStyle & Style) const774efde1e5SMarek Kurdej bool FormatToken::opensBlockOrBlockTypeList(const FormatStyle &Style) const {
784efde1e5SMarek Kurdej   // C# Does not indent object initialisers as continuations.
794efde1e5SMarek Kurdej   if (is(tok::l_brace) && getBlockKind() == BK_BracedInit && Style.isCSharp())
804efde1e5SMarek Kurdej     return true;
814efde1e5SMarek Kurdej   if (is(TT_TemplateString) && opensScope())
824efde1e5SMarek Kurdej     return true;
834efde1e5SMarek Kurdej   return is(TT_ArrayInitializerLSquare) || is(TT_ProtoExtensionLSquare) ||
844efde1e5SMarek Kurdej          (is(tok::l_brace) &&
854efde1e5SMarek Kurdej           (getBlockKind() == BK_Block || is(TT_DictLiteral) ||
864efde1e5SMarek Kurdej            (!Style.Cpp11BracedListStyle && NestingLevel == 0))) ||
874efde1e5SMarek Kurdej          (is(tok::less) && (Style.Language == FormatStyle::LK_Proto ||
884efde1e5SMarek Kurdej                             Style.Language == FormatStyle::LK_TextProto));
894efde1e5SMarek Kurdej }
904efde1e5SMarek Kurdej 
~TokenRole()91637d1e66SAngel Garcia Gomez TokenRole::~TokenRole() {}
928de9ed05SDaniel Jasper 
precomputeFormattingInfos(const FormatToken * Token)938de9ed05SDaniel Jasper void TokenRole::precomputeFormattingInfos(const FormatToken *Token) {}
948de9ed05SDaniel Jasper 
formatAfterToken(LineState & State,ContinuationIndenter * Indenter,bool DryRun)950160347bSDaniel Jasper unsigned CommaSeparatedList::formatAfterToken(LineState &State,
968de9ed05SDaniel Jasper                                               ContinuationIndenter *Indenter,
978de9ed05SDaniel Jasper                                               bool DryRun) {
9860c27076SDaniel Jasper   if (State.NextToken == nullptr || !State.NextToken->Previous)
998de9ed05SDaniel Jasper     return 0;
1008de9ed05SDaniel Jasper 
101083d1700SDaniel Jasper   if (Formats.size() == 1)
102083d1700SDaniel Jasper     return 0; // Handled by formatFromToken
103083d1700SDaniel Jasper 
1048de9ed05SDaniel Jasper   // Ensure that we start on the opening brace.
10560c27076SDaniel Jasper   const FormatToken *LBrace =
10660c27076SDaniel Jasper       State.NextToken->Previous->getPreviousNonComment();
107e2deb59fSDaniel Jasper   if (!LBrace || !LBrace->isOneOf(tok::l_brace, TT_ArrayInitializerLSquare) ||
108f5acd11dSBruno Ricci       LBrace->is(BK_Block) || LBrace->is(TT_DictLiteral) ||
109bebf7bdfSowenca       LBrace->Next->is(TT_DesignatedInitializerPeriod)) {
1108de9ed05SDaniel Jasper     return 0;
111bebf7bdfSowenca   }
1128de9ed05SDaniel Jasper 
113cb3f0ed8SDaniel Jasper   // Calculate the number of code points we have to format this list. As the
114cb3f0ed8SDaniel Jasper   // first token is already placed, we have to subtract it.
115b05a81deSDaniel Jasper   unsigned RemainingCodePoints =
116b05a81deSDaniel Jasper       Style.ColumnLimit - State.Column + State.NextToken->Previous->ColumnWidth;
117cb3f0ed8SDaniel Jasper 
1188de9ed05SDaniel Jasper   // Find the best ColumnFormat, i.e. the best number of columns to use.
119cb3f0ed8SDaniel Jasper   const ColumnFormat *Format = getColumnFormat(RemainingCodePoints);
120ff8d6136SDaniel Jasper 
1211a1a2ab7SDaniel Jasper   // If no ColumnFormat can be used, the braced list would generally be
1221a1a2ab7SDaniel Jasper   // bin-packed. Add a severe penalty to this so that column layouts are
123f6a24ce4SAlp Toker   // preferred if possible.
1248de9ed05SDaniel Jasper   if (!Format)
1251a1a2ab7SDaniel Jasper     return 10000;
1268de9ed05SDaniel Jasper 
1278de9ed05SDaniel Jasper   // Format the entire list.
1288de9ed05SDaniel Jasper   unsigned Penalty = 0;
1298de9ed05SDaniel Jasper   unsigned Column = 0;
1308de9ed05SDaniel Jasper   unsigned Item = 0;
1318de9ed05SDaniel Jasper   while (State.NextToken != LBrace->MatchingParen) {
1328de9ed05SDaniel Jasper     bool NewLine = false;
1338de9ed05SDaniel Jasper     unsigned ExtraSpaces = 0;
1348de9ed05SDaniel Jasper 
1358de9ed05SDaniel Jasper     // If the previous token was one of our commas, we are now on the next item.
1368de9ed05SDaniel Jasper     if (Item < Commas.size() && State.NextToken->Previous == Commas[Item]) {
1378de9ed05SDaniel Jasper       if (!State.NextToken->isTrailingComment()) {
1388de9ed05SDaniel Jasper         ExtraSpaces += Format->ColumnSizes[Column] - ItemLengths[Item];
1398de9ed05SDaniel Jasper         ++Column;
1408de9ed05SDaniel Jasper       }
1418de9ed05SDaniel Jasper       ++Item;
1428de9ed05SDaniel Jasper     }
1438de9ed05SDaniel Jasper 
1448de9ed05SDaniel Jasper     if (Column == Format->Columns || State.NextToken->MustBreakBefore) {
1458de9ed05SDaniel Jasper       Column = 0;
1468de9ed05SDaniel Jasper       NewLine = true;
1478de9ed05SDaniel Jasper     }
1488de9ed05SDaniel Jasper 
1498de9ed05SDaniel Jasper     // Place token using the continuation indenter and store the penalty.
1508de9ed05SDaniel Jasper     Penalty += Indenter->addTokenToState(State, NewLine, DryRun, ExtraSpaces);
1518de9ed05SDaniel Jasper   }
1528de9ed05SDaniel Jasper   return Penalty;
1538de9ed05SDaniel Jasper }
1548de9ed05SDaniel Jasper 
formatFromToken(LineState & State,ContinuationIndenter * Indenter,bool DryRun)1550160347bSDaniel Jasper unsigned CommaSeparatedList::formatFromToken(LineState &State,
1560160347bSDaniel Jasper                                              ContinuationIndenter *Indenter,
1570160347bSDaniel Jasper                                              bool DryRun) {
158083d1700SDaniel Jasper   // Formatting with 1 Column isn't really a column layout, so we don't need the
159083d1700SDaniel Jasper   // special logic here. We can just avoid bin packing any of the parameters.
160083d1700SDaniel Jasper   if (Formats.size() == 1 || HasNestedBracedList)
1610160347bSDaniel Jasper     State.Stack.back().AvoidBinPacking = true;
1620160347bSDaniel Jasper   return 0;
1630160347bSDaniel Jasper }
1640160347bSDaniel Jasper 
1658de9ed05SDaniel Jasper // Returns the lengths in code points between Begin and End (both included),
1668de9ed05SDaniel Jasper // assuming that the entire sequence is put on a single line.
CodePointsBetween(const FormatToken * Begin,const FormatToken * End)1678de9ed05SDaniel Jasper static unsigned CodePointsBetween(const FormatToken *Begin,
1688de9ed05SDaniel Jasper                                   const FormatToken *End) {
1698863ada8SDaniel Jasper   assert(End->TotalLength >= Begin->TotalLength);
17039856b71SAlexander Kornienko   return End->TotalLength - Begin->TotalLength + Begin->ColumnWidth;
1718de9ed05SDaniel Jasper }
1728de9ed05SDaniel Jasper 
precomputeFormattingInfos(const FormatToken * Token)1738de9ed05SDaniel Jasper void CommaSeparatedList::precomputeFormattingInfos(const FormatToken *Token) {
1748863ada8SDaniel Jasper   // FIXME: At some point we might want to do this for other lists, too.
175e2deb59fSDaniel Jasper   if (!Token->MatchingParen ||
176bebf7bdfSowenca       !Token->isOneOf(tok::l_brace, TT_ArrayInitializerLSquare)) {
1778de9ed05SDaniel Jasper     return;
178bebf7bdfSowenca   }
1798de9ed05SDaniel Jasper 
180839922e9SDaniel Jasper   // In C++11 braced list style, we should not format in columns unless they
1810843434eSDaniel Jasper   // have many items (20 or more) or we allow bin-packing of function call
1820843434eSDaniel Jasper   // arguments.
1830843434eSDaniel Jasper   if (Style.Cpp11BracedListStyle && !Style.BinPackArguments &&
184bebf7bdfSowenca       Commas.size() < 19) {
185ae8e0d8dSDaniel Jasper     return;
186bebf7bdfSowenca   }
187ae8e0d8dSDaniel Jasper 
188e2deb59fSDaniel Jasper   // Limit column layout for JavaScript array initializers to 20 or more items
189e2deb59fSDaniel Jasper   // for now to introduce it carefully. We can become more aggressive if this
190e2deb59fSDaniel Jasper   // necessary.
191e2deb59fSDaniel Jasper   if (Token->is(TT_ArrayInitializerLSquare) && Commas.size() < 19)
192e2deb59fSDaniel Jasper     return;
193e2deb59fSDaniel Jasper 
19413404dadSDaniel Jasper   // Column format doesn't really make sense if we don't align after brackets.
1956501f7e8SDaniel Jasper   if (Style.AlignAfterOpenBracket == FormatStyle::BAS_DontAlign)
19613404dadSDaniel Jasper     return;
19713404dadSDaniel Jasper 
1988de9ed05SDaniel Jasper   FormatToken *ItemBegin = Token->Next;
19960c27076SDaniel Jasper   while (ItemBegin->isTrailingComment())
20060c27076SDaniel Jasper     ItemBegin = ItemBegin->Next;
2018de9ed05SDaniel Jasper   SmallVector<bool, 8> MustBreakBeforeItem;
2028de9ed05SDaniel Jasper 
2038de9ed05SDaniel Jasper   // The lengths of an item if it is put at the end of the line. This includes
2048de9ed05SDaniel Jasper   // trailing comments which are otherwise ignored for column alignment.
2058de9ed05SDaniel Jasper   SmallVector<unsigned, 8> EndOfLineItemLength;
206768a6192SMarek Kurdej   MustBreakBeforeItem.reserve(Commas.size() + 1);
207768a6192SMarek Kurdej   EndOfLineItemLength.reserve(Commas.size() + 1);
208768a6192SMarek Kurdej   ItemLengths.reserve(Commas.size() + 1);
2098de9ed05SDaniel Jasper 
210731dde91SDaniel Jasper   bool HasSeparatingComment = false;
2118de9ed05SDaniel Jasper   for (unsigned i = 0, e = Commas.size() + 1; i != e; ++i) {
21281793bd2SMarek Kurdej     assert(ItemBegin);
2138de9ed05SDaniel Jasper     // Skip comments on their own line.
214731dde91SDaniel Jasper     while (ItemBegin->HasUnescapedNewline && ItemBegin->isTrailingComment()) {
2158de9ed05SDaniel Jasper       ItemBegin = ItemBegin->Next;
216731dde91SDaniel Jasper       HasSeparatingComment = i > 0;
217731dde91SDaniel Jasper     }
2188de9ed05SDaniel Jasper 
2198de9ed05SDaniel Jasper     MustBreakBeforeItem.push_back(ItemBegin->MustBreakBefore);
2207c72d9feSDaniel Jasper     if (ItemBegin->is(tok::l_brace))
2217c72d9feSDaniel Jasper       HasNestedBracedList = true;
2222145bc02SCraig Topper     const FormatToken *ItemEnd = nullptr;
2238de9ed05SDaniel Jasper     if (i == Commas.size()) {
2248de9ed05SDaniel Jasper       ItemEnd = Token->MatchingParen;
2258de9ed05SDaniel Jasper       const FormatToken *NonCommentEnd = ItemEnd->getPreviousNonComment();
2268de9ed05SDaniel Jasper       ItemLengths.push_back(CodePointsBetween(ItemBegin, NonCommentEnd));
22700fb2a11SDaniel Jasper       if (Style.Cpp11BracedListStyle &&
22800fb2a11SDaniel Jasper           !ItemEnd->Previous->isTrailingComment()) {
2298de9ed05SDaniel Jasper         // In Cpp11 braced list style, the } and possibly other subsequent
2308de9ed05SDaniel Jasper         // tokens will need to stay on a line with the last element.
2318de9ed05SDaniel Jasper         while (ItemEnd->Next && !ItemEnd->Next->CanBreakBefore)
2328de9ed05SDaniel Jasper           ItemEnd = ItemEnd->Next;
2338de9ed05SDaniel Jasper       } else {
2348de9ed05SDaniel Jasper         // In other braced lists styles, the "}" can be wrapped to the new line.
2358de9ed05SDaniel Jasper         ItemEnd = Token->MatchingParen->Previous;
2368de9ed05SDaniel Jasper       }
2378de9ed05SDaniel Jasper     } else {
2388de9ed05SDaniel Jasper       ItemEnd = Commas[i];
2398de9ed05SDaniel Jasper       // The comma is counted as part of the item when calculating the length.
2408863ada8SDaniel Jasper       ItemLengths.push_back(CodePointsBetween(ItemBegin, ItemEnd));
241540dbe29SDaniel Jasper 
2428de9ed05SDaniel Jasper       // Consume trailing comments so the are included in EndOfLineItemLength.
2438de9ed05SDaniel Jasper       if (ItemEnd->Next && !ItemEnd->Next->HasUnescapedNewline &&
244bebf7bdfSowenca           ItemEnd->Next->isTrailingComment()) {
2458de9ed05SDaniel Jasper         ItemEnd = ItemEnd->Next;
2468de9ed05SDaniel Jasper       }
247bebf7bdfSowenca     }
2488de9ed05SDaniel Jasper     EndOfLineItemLength.push_back(CodePointsBetween(ItemBegin, ItemEnd));
2498863ada8SDaniel Jasper     // If there is a trailing comma in the list, the next item will start at the
2508863ada8SDaniel Jasper     // closing brace. Don't create an extra item for this.
2518863ada8SDaniel Jasper     if (ItemEnd->getNextNonComment() == Token->MatchingParen)
2528863ada8SDaniel Jasper       break;
2538de9ed05SDaniel Jasper     ItemBegin = ItemEnd->Next;
2548de9ed05SDaniel Jasper   }
2558de9ed05SDaniel Jasper 
25655582073SDaniel Jasper   // Don't use column layout for lists with few elements and in presence of
25755582073SDaniel Jasper   // separating comments.
25855582073SDaniel Jasper   if (Commas.size() < 5 || HasSeparatingComment)
25955582073SDaniel Jasper     return;
26055582073SDaniel Jasper 
26155582073SDaniel Jasper   if (Token->NestingLevel != 0 && Token->is(tok::l_brace) && Commas.size() < 19)
2620160347bSDaniel Jasper     return;
2630160347bSDaniel Jasper 
2648de9ed05SDaniel Jasper   // We can never place more than ColumnLimit / 3 items in a row (because of the
2658de9ed05SDaniel Jasper   // spaces and the comma).
2663aa55db8SBenjamin Kramer   unsigned MaxItems = Style.ColumnLimit / 3;
267*36229fa3Sowenca   SmallVector<unsigned> MinSizeInColumn;
2683aa55db8SBenjamin Kramer   MinSizeInColumn.reserve(MaxItems);
2693aa55db8SBenjamin Kramer   for (unsigned Columns = 1; Columns <= MaxItems; ++Columns) {
2708de9ed05SDaniel Jasper     ColumnFormat Format;
2718de9ed05SDaniel Jasper     Format.Columns = Columns;
2728de9ed05SDaniel Jasper     Format.ColumnSizes.resize(Columns);
2733aa55db8SBenjamin Kramer     MinSizeInColumn.assign(Columns, UINT_MAX);
2747c72d9feSDaniel Jasper     Format.LineCount = 1;
2758de9ed05SDaniel Jasper     bool HasRowWithSufficientColumns = false;
2768de9ed05SDaniel Jasper     unsigned Column = 0;
2778de9ed05SDaniel Jasper     for (unsigned i = 0, e = ItemLengths.size(); i != e; ++i) {
27871814b44SManuel Klimek       assert(i < MustBreakBeforeItem.size());
2798de9ed05SDaniel Jasper       if (MustBreakBeforeItem[i] || Column == Columns) {
2808de9ed05SDaniel Jasper         ++Format.LineCount;
2818de9ed05SDaniel Jasper         Column = 0;
2828de9ed05SDaniel Jasper       }
2838de9ed05SDaniel Jasper       if (Column == Columns - 1)
2848de9ed05SDaniel Jasper         HasRowWithSufficientColumns = true;
285d57843d4SDaniel Jasper       unsigned Length =
2868de9ed05SDaniel Jasper           (Column == Columns - 1) ? EndOfLineItemLength[i] : ItemLengths[i];
287d57843d4SDaniel Jasper       Format.ColumnSizes[Column] = std::max(Format.ColumnSizes[Column], Length);
288d57843d4SDaniel Jasper       MinSizeInColumn[Column] = std::min(MinSizeInColumn[Column], Length);
2898de9ed05SDaniel Jasper       ++Column;
2908de9ed05SDaniel Jasper     }
2918de9ed05SDaniel Jasper     // If all rows are terminated early (e.g. by trailing comments), we don't
2928de9ed05SDaniel Jasper     // need to look further.
2938de9ed05SDaniel Jasper     if (!HasRowWithSufficientColumns)
2948de9ed05SDaniel Jasper       break;
2958de9ed05SDaniel Jasper     Format.TotalWidth = Columns - 1; // Width of the N-1 spaces.
296d57843d4SDaniel Jasper 
297d57843d4SDaniel Jasper     for (unsigned i = 0; i < Columns; ++i)
2988de9ed05SDaniel Jasper       Format.TotalWidth += Format.ColumnSizes[i];
299d57843d4SDaniel Jasper 
300d57843d4SDaniel Jasper     // Don't use this Format, if the difference between the longest and shortest
301d57843d4SDaniel Jasper     // element in a column exceeds a threshold to avoid excessive spaces.
302d57843d4SDaniel Jasper     if ([&] {
303d57843d4SDaniel Jasper           for (unsigned i = 0; i < Columns - 1; ++i)
304d57843d4SDaniel Jasper             if (Format.ColumnSizes[i] - MinSizeInColumn[i] > 10)
305d57843d4SDaniel Jasper               return true;
306d57843d4SDaniel Jasper           return false;
307bebf7bdfSowenca         }()) {
308d57843d4SDaniel Jasper       continue;
309bebf7bdfSowenca     }
3108de9ed05SDaniel Jasper 
3118de9ed05SDaniel Jasper     // Ignore layouts that are bound to violate the column limit.
312e6169665SDaniel Jasper     if (Format.TotalWidth > Style.ColumnLimit && Columns > 1)
3138de9ed05SDaniel Jasper       continue;
3148de9ed05SDaniel Jasper 
3158de9ed05SDaniel Jasper     Formats.push_back(Format);
3168de9ed05SDaniel Jasper   }
3178de9ed05SDaniel Jasper }
3188de9ed05SDaniel Jasper 
3198de9ed05SDaniel Jasper const CommaSeparatedList::ColumnFormat *
getColumnFormat(unsigned RemainingCharacters) const3208de9ed05SDaniel Jasper CommaSeparatedList::getColumnFormat(unsigned RemainingCharacters) const {
3212145bc02SCraig Topper   const ColumnFormat *BestFormat = nullptr;
32201f355feSMarek Kurdej   for (const ColumnFormat &Format : llvm::reverse(Formats)) {
32301f355feSMarek Kurdej     if (Format.TotalWidth <= RemainingCharacters || Format.Columns == 1) {
32401f355feSMarek Kurdej       if (BestFormat && Format.LineCount > BestFormat->LineCount)
3258de9ed05SDaniel Jasper         break;
32601f355feSMarek Kurdej       BestFormat = &Format;
3278de9ed05SDaniel Jasper     }
3288de9ed05SDaniel Jasper   }
3298de9ed05SDaniel Jasper   return BestFormat;
3308de9ed05SDaniel Jasper }
3318de9ed05SDaniel Jasper 
3328de9ed05SDaniel Jasper } // namespace format
3338de9ed05SDaniel Jasper } // namespace clang
334