annotate clang/lib/Format/WhitespaceManager.cpp @ 176:de4ac79aef9d

...
author Shinji KONO <kono@ie.u-ryukyu.ac.jp>
date Mon, 25 May 2020 17:13:11 +0900
parents 0572611fdcc8
children 2e18cbf3894f
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
150
anatofuz
parents:
diff changeset
1 //===--- WhitespaceManager.cpp - Format C++ code --------------------------===//
anatofuz
parents:
diff changeset
2 //
anatofuz
parents:
diff changeset
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
anatofuz
parents:
diff changeset
4 // See https://llvm.org/LICENSE.txt for license information.
anatofuz
parents:
diff changeset
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
anatofuz
parents:
diff changeset
6 //
anatofuz
parents:
diff changeset
7 //===----------------------------------------------------------------------===//
anatofuz
parents:
diff changeset
8 ///
anatofuz
parents:
diff changeset
9 /// \file
anatofuz
parents:
diff changeset
10 /// This file implements WhitespaceManager class.
anatofuz
parents:
diff changeset
11 ///
anatofuz
parents:
diff changeset
12 //===----------------------------------------------------------------------===//
anatofuz
parents:
diff changeset
13
anatofuz
parents:
diff changeset
14 #include "WhitespaceManager.h"
anatofuz
parents:
diff changeset
15 #include "llvm/ADT/STLExtras.h"
anatofuz
parents:
diff changeset
16
anatofuz
parents:
diff changeset
17 namespace clang {
anatofuz
parents:
diff changeset
18 namespace format {
anatofuz
parents:
diff changeset
19
anatofuz
parents:
diff changeset
20 bool WhitespaceManager::Change::IsBeforeInFile::operator()(
anatofuz
parents:
diff changeset
21 const Change &C1, const Change &C2) const {
anatofuz
parents:
diff changeset
22 return SourceMgr.isBeforeInTranslationUnit(
anatofuz
parents:
diff changeset
23 C1.OriginalWhitespaceRange.getBegin(),
anatofuz
parents:
diff changeset
24 C2.OriginalWhitespaceRange.getBegin());
anatofuz
parents:
diff changeset
25 }
anatofuz
parents:
diff changeset
26
anatofuz
parents:
diff changeset
27 WhitespaceManager::Change::Change(const FormatToken &Tok,
anatofuz
parents:
diff changeset
28 bool CreateReplacement,
anatofuz
parents:
diff changeset
29 SourceRange OriginalWhitespaceRange,
anatofuz
parents:
diff changeset
30 int Spaces, unsigned StartOfTokenColumn,
anatofuz
parents:
diff changeset
31 unsigned NewlinesBefore,
anatofuz
parents:
diff changeset
32 StringRef PreviousLinePostfix,
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
33 StringRef CurrentLinePrefix, bool IsAligned,
150
anatofuz
parents:
diff changeset
34 bool ContinuesPPDirective, bool IsInsideToken)
anatofuz
parents:
diff changeset
35 : Tok(&Tok), CreateReplacement(CreateReplacement),
anatofuz
parents:
diff changeset
36 OriginalWhitespaceRange(OriginalWhitespaceRange),
anatofuz
parents:
diff changeset
37 StartOfTokenColumn(StartOfTokenColumn), NewlinesBefore(NewlinesBefore),
anatofuz
parents:
diff changeset
38 PreviousLinePostfix(PreviousLinePostfix),
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
39 CurrentLinePrefix(CurrentLinePrefix), IsAligned(IsAligned),
150
anatofuz
parents:
diff changeset
40 ContinuesPPDirective(ContinuesPPDirective), Spaces(Spaces),
anatofuz
parents:
diff changeset
41 IsInsideToken(IsInsideToken), IsTrailingComment(false), TokenLength(0),
anatofuz
parents:
diff changeset
42 PreviousEndOfTokenColumn(0), EscapedNewlineColumn(0),
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
43 StartOfBlockComment(nullptr), IndentationOffset(0), ConditionalsLevel(0) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
44 }
150
anatofuz
parents:
diff changeset
45
anatofuz
parents:
diff changeset
46 void WhitespaceManager::replaceWhitespace(FormatToken &Tok, unsigned Newlines,
anatofuz
parents:
diff changeset
47 unsigned Spaces,
anatofuz
parents:
diff changeset
48 unsigned StartOfTokenColumn,
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
49 bool IsAligned, bool InPPDirective) {
150
anatofuz
parents:
diff changeset
50 if (Tok.Finalized)
anatofuz
parents:
diff changeset
51 return;
anatofuz
parents:
diff changeset
52 Tok.Decision = (Newlines > 0) ? FD_Break : FD_Continue;
anatofuz
parents:
diff changeset
53 Changes.push_back(Change(Tok, /*CreateReplacement=*/true, Tok.WhitespaceRange,
anatofuz
parents:
diff changeset
54 Spaces, StartOfTokenColumn, Newlines, "", "",
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
55 IsAligned, InPPDirective && !Tok.IsFirst,
150
anatofuz
parents:
diff changeset
56 /*IsInsideToken=*/false));
anatofuz
parents:
diff changeset
57 }
anatofuz
parents:
diff changeset
58
anatofuz
parents:
diff changeset
59 void WhitespaceManager::addUntouchableToken(const FormatToken &Tok,
anatofuz
parents:
diff changeset
60 bool InPPDirective) {
anatofuz
parents:
diff changeset
61 if (Tok.Finalized)
anatofuz
parents:
diff changeset
62 return;
anatofuz
parents:
diff changeset
63 Changes.push_back(Change(Tok, /*CreateReplacement=*/false,
anatofuz
parents:
diff changeset
64 Tok.WhitespaceRange, /*Spaces=*/0,
anatofuz
parents:
diff changeset
65 Tok.OriginalColumn, Tok.NewlinesBefore, "", "",
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
66 /*IsAligned=*/false, InPPDirective && !Tok.IsFirst,
150
anatofuz
parents:
diff changeset
67 /*IsInsideToken=*/false));
anatofuz
parents:
diff changeset
68 }
anatofuz
parents:
diff changeset
69
anatofuz
parents:
diff changeset
70 llvm::Error
anatofuz
parents:
diff changeset
71 WhitespaceManager::addReplacement(const tooling::Replacement &Replacement) {
anatofuz
parents:
diff changeset
72 return Replaces.add(Replacement);
anatofuz
parents:
diff changeset
73 }
anatofuz
parents:
diff changeset
74
anatofuz
parents:
diff changeset
75 void WhitespaceManager::replaceWhitespaceInToken(
anatofuz
parents:
diff changeset
76 const FormatToken &Tok, unsigned Offset, unsigned ReplaceChars,
anatofuz
parents:
diff changeset
77 StringRef PreviousPostfix, StringRef CurrentPrefix, bool InPPDirective,
anatofuz
parents:
diff changeset
78 unsigned Newlines, int Spaces) {
anatofuz
parents:
diff changeset
79 if (Tok.Finalized)
anatofuz
parents:
diff changeset
80 return;
anatofuz
parents:
diff changeset
81 SourceLocation Start = Tok.getStartOfNonWhitespace().getLocWithOffset(Offset);
anatofuz
parents:
diff changeset
82 Changes.push_back(
anatofuz
parents:
diff changeset
83 Change(Tok, /*CreateReplacement=*/true,
anatofuz
parents:
diff changeset
84 SourceRange(Start, Start.getLocWithOffset(ReplaceChars)), Spaces,
anatofuz
parents:
diff changeset
85 std::max(0, Spaces), Newlines, PreviousPostfix, CurrentPrefix,
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
86 /*IsAligned=*/true, InPPDirective && !Tok.IsFirst,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
87 /*IsInsideToken=*/true));
150
anatofuz
parents:
diff changeset
88 }
anatofuz
parents:
diff changeset
89
anatofuz
parents:
diff changeset
90 const tooling::Replacements &WhitespaceManager::generateReplacements() {
anatofuz
parents:
diff changeset
91 if (Changes.empty())
anatofuz
parents:
diff changeset
92 return Replaces;
anatofuz
parents:
diff changeset
93
anatofuz
parents:
diff changeset
94 llvm::sort(Changes, Change::IsBeforeInFile(SourceMgr));
anatofuz
parents:
diff changeset
95 calculateLineBreakInformation();
anatofuz
parents:
diff changeset
96 alignConsecutiveMacros();
anatofuz
parents:
diff changeset
97 alignConsecutiveDeclarations();
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
98 alignConsecutiveBitFields();
150
anatofuz
parents:
diff changeset
99 alignConsecutiveAssignments();
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
100 alignChainedConditionals();
150
anatofuz
parents:
diff changeset
101 alignTrailingComments();
anatofuz
parents:
diff changeset
102 alignEscapedNewlines();
anatofuz
parents:
diff changeset
103 generateChanges();
anatofuz
parents:
diff changeset
104
anatofuz
parents:
diff changeset
105 return Replaces;
anatofuz
parents:
diff changeset
106 }
anatofuz
parents:
diff changeset
107
anatofuz
parents:
diff changeset
108 void WhitespaceManager::calculateLineBreakInformation() {
anatofuz
parents:
diff changeset
109 Changes[0].PreviousEndOfTokenColumn = 0;
anatofuz
parents:
diff changeset
110 Change *LastOutsideTokenChange = &Changes[0];
anatofuz
parents:
diff changeset
111 for (unsigned i = 1, e = Changes.size(); i != e; ++i) {
anatofuz
parents:
diff changeset
112 SourceLocation OriginalWhitespaceStart =
anatofuz
parents:
diff changeset
113 Changes[i].OriginalWhitespaceRange.getBegin();
anatofuz
parents:
diff changeset
114 SourceLocation PreviousOriginalWhitespaceEnd =
anatofuz
parents:
diff changeset
115 Changes[i - 1].OriginalWhitespaceRange.getEnd();
anatofuz
parents:
diff changeset
116 unsigned OriginalWhitespaceStartOffset =
anatofuz
parents:
diff changeset
117 SourceMgr.getFileOffset(OriginalWhitespaceStart);
anatofuz
parents:
diff changeset
118 unsigned PreviousOriginalWhitespaceEndOffset =
anatofuz
parents:
diff changeset
119 SourceMgr.getFileOffset(PreviousOriginalWhitespaceEnd);
anatofuz
parents:
diff changeset
120 assert(PreviousOriginalWhitespaceEndOffset <=
anatofuz
parents:
diff changeset
121 OriginalWhitespaceStartOffset);
anatofuz
parents:
diff changeset
122 const char *const PreviousOriginalWhitespaceEndData =
anatofuz
parents:
diff changeset
123 SourceMgr.getCharacterData(PreviousOriginalWhitespaceEnd);
anatofuz
parents:
diff changeset
124 StringRef Text(PreviousOriginalWhitespaceEndData,
anatofuz
parents:
diff changeset
125 SourceMgr.getCharacterData(OriginalWhitespaceStart) -
anatofuz
parents:
diff changeset
126 PreviousOriginalWhitespaceEndData);
anatofuz
parents:
diff changeset
127 // Usually consecutive changes would occur in consecutive tokens. This is
anatofuz
parents:
diff changeset
128 // not the case however when analyzing some preprocessor runs of the
anatofuz
parents:
diff changeset
129 // annotated lines. For example, in this code:
anatofuz
parents:
diff changeset
130 //
anatofuz
parents:
diff changeset
131 // #if A // line 1
anatofuz
parents:
diff changeset
132 // int i = 1;
anatofuz
parents:
diff changeset
133 // #else B // line 2
anatofuz
parents:
diff changeset
134 // int i = 2;
anatofuz
parents:
diff changeset
135 // #endif // line 3
anatofuz
parents:
diff changeset
136 //
anatofuz
parents:
diff changeset
137 // one of the runs will produce the sequence of lines marked with line 1, 2
anatofuz
parents:
diff changeset
138 // and 3. So the two consecutive whitespace changes just before '// line 2'
anatofuz
parents:
diff changeset
139 // and before '#endif // line 3' span multiple lines and tokens:
anatofuz
parents:
diff changeset
140 //
anatofuz
parents:
diff changeset
141 // #else B{change X}[// line 2
anatofuz
parents:
diff changeset
142 // int i = 2;
anatofuz
parents:
diff changeset
143 // ]{change Y}#endif // line 3
anatofuz
parents:
diff changeset
144 //
anatofuz
parents:
diff changeset
145 // For this reason, if the text between consecutive changes spans multiple
anatofuz
parents:
diff changeset
146 // newlines, the token length must be adjusted to the end of the original
anatofuz
parents:
diff changeset
147 // line of the token.
anatofuz
parents:
diff changeset
148 auto NewlinePos = Text.find_first_of('\n');
anatofuz
parents:
diff changeset
149 if (NewlinePos == StringRef::npos) {
anatofuz
parents:
diff changeset
150 Changes[i - 1].TokenLength = OriginalWhitespaceStartOffset -
anatofuz
parents:
diff changeset
151 PreviousOriginalWhitespaceEndOffset +
anatofuz
parents:
diff changeset
152 Changes[i].PreviousLinePostfix.size() +
anatofuz
parents:
diff changeset
153 Changes[i - 1].CurrentLinePrefix.size();
anatofuz
parents:
diff changeset
154 } else {
anatofuz
parents:
diff changeset
155 Changes[i - 1].TokenLength =
anatofuz
parents:
diff changeset
156 NewlinePos + Changes[i - 1].CurrentLinePrefix.size();
anatofuz
parents:
diff changeset
157 }
anatofuz
parents:
diff changeset
158
anatofuz
parents:
diff changeset
159 // If there are multiple changes in this token, sum up all the changes until
anatofuz
parents:
diff changeset
160 // the end of the line.
anatofuz
parents:
diff changeset
161 if (Changes[i - 1].IsInsideToken && Changes[i - 1].NewlinesBefore == 0)
anatofuz
parents:
diff changeset
162 LastOutsideTokenChange->TokenLength +=
anatofuz
parents:
diff changeset
163 Changes[i - 1].TokenLength + Changes[i - 1].Spaces;
anatofuz
parents:
diff changeset
164 else
anatofuz
parents:
diff changeset
165 LastOutsideTokenChange = &Changes[i - 1];
anatofuz
parents:
diff changeset
166
anatofuz
parents:
diff changeset
167 Changes[i].PreviousEndOfTokenColumn =
anatofuz
parents:
diff changeset
168 Changes[i - 1].StartOfTokenColumn + Changes[i - 1].TokenLength;
anatofuz
parents:
diff changeset
169
anatofuz
parents:
diff changeset
170 Changes[i - 1].IsTrailingComment =
anatofuz
parents:
diff changeset
171 (Changes[i].NewlinesBefore > 0 || Changes[i].Tok->is(tok::eof) ||
anatofuz
parents:
diff changeset
172 (Changes[i].IsInsideToken && Changes[i].Tok->is(tok::comment))) &&
anatofuz
parents:
diff changeset
173 Changes[i - 1].Tok->is(tok::comment) &&
anatofuz
parents:
diff changeset
174 // FIXME: This is a dirty hack. The problem is that
anatofuz
parents:
diff changeset
175 // BreakableLineCommentSection does comment reflow changes and here is
anatofuz
parents:
diff changeset
176 // the aligning of trailing comments. Consider the case where we reflow
anatofuz
parents:
diff changeset
177 // the second line up in this example:
anatofuz
parents:
diff changeset
178 //
anatofuz
parents:
diff changeset
179 // // line 1
anatofuz
parents:
diff changeset
180 // // line 2
anatofuz
parents:
diff changeset
181 //
anatofuz
parents:
diff changeset
182 // That amounts to 2 changes by BreakableLineCommentSection:
anatofuz
parents:
diff changeset
183 // - the first, delimited by (), for the whitespace between the tokens,
anatofuz
parents:
diff changeset
184 // - and second, delimited by [], for the whitespace at the beginning
anatofuz
parents:
diff changeset
185 // of the second token:
anatofuz
parents:
diff changeset
186 //
anatofuz
parents:
diff changeset
187 // // line 1(
anatofuz
parents:
diff changeset
188 // )[// ]line 2
anatofuz
parents:
diff changeset
189 //
anatofuz
parents:
diff changeset
190 // So in the end we have two changes like this:
anatofuz
parents:
diff changeset
191 //
anatofuz
parents:
diff changeset
192 // // line1()[ ]line 2
anatofuz
parents:
diff changeset
193 //
anatofuz
parents:
diff changeset
194 // Note that the OriginalWhitespaceStart of the second change is the
anatofuz
parents:
diff changeset
195 // same as the PreviousOriginalWhitespaceEnd of the first change.
anatofuz
parents:
diff changeset
196 // In this case, the below check ensures that the second change doesn't
anatofuz
parents:
diff changeset
197 // get treated as a trailing comment change here, since this might
anatofuz
parents:
diff changeset
198 // trigger additional whitespace to be wrongly inserted before "line 2"
anatofuz
parents:
diff changeset
199 // by the comment aligner here.
anatofuz
parents:
diff changeset
200 //
anatofuz
parents:
diff changeset
201 // For a proper solution we need a mechanism to say to WhitespaceManager
anatofuz
parents:
diff changeset
202 // that a particular change breaks the current sequence of trailing
anatofuz
parents:
diff changeset
203 // comments.
anatofuz
parents:
diff changeset
204 OriginalWhitespaceStart != PreviousOriginalWhitespaceEnd;
anatofuz
parents:
diff changeset
205 }
anatofuz
parents:
diff changeset
206 // FIXME: The last token is currently not always an eof token; in those
anatofuz
parents:
diff changeset
207 // cases, setting TokenLength of the last token to 0 is wrong.
anatofuz
parents:
diff changeset
208 Changes.back().TokenLength = 0;
anatofuz
parents:
diff changeset
209 Changes.back().IsTrailingComment = Changes.back().Tok->is(tok::comment);
anatofuz
parents:
diff changeset
210
anatofuz
parents:
diff changeset
211 const WhitespaceManager::Change *LastBlockComment = nullptr;
anatofuz
parents:
diff changeset
212 for (auto &Change : Changes) {
anatofuz
parents:
diff changeset
213 // Reset the IsTrailingComment flag for changes inside of trailing comments
anatofuz
parents:
diff changeset
214 // so they don't get realigned later. Comment line breaks however still need
anatofuz
parents:
diff changeset
215 // to be aligned.
anatofuz
parents:
diff changeset
216 if (Change.IsInsideToken && Change.NewlinesBefore == 0)
anatofuz
parents:
diff changeset
217 Change.IsTrailingComment = false;
anatofuz
parents:
diff changeset
218 Change.StartOfBlockComment = nullptr;
anatofuz
parents:
diff changeset
219 Change.IndentationOffset = 0;
anatofuz
parents:
diff changeset
220 if (Change.Tok->is(tok::comment)) {
anatofuz
parents:
diff changeset
221 if (Change.Tok->is(TT_LineComment) || !Change.IsInsideToken)
anatofuz
parents:
diff changeset
222 LastBlockComment = &Change;
anatofuz
parents:
diff changeset
223 else {
anatofuz
parents:
diff changeset
224 if ((Change.StartOfBlockComment = LastBlockComment))
anatofuz
parents:
diff changeset
225 Change.IndentationOffset =
anatofuz
parents:
diff changeset
226 Change.StartOfTokenColumn -
anatofuz
parents:
diff changeset
227 Change.StartOfBlockComment->StartOfTokenColumn;
anatofuz
parents:
diff changeset
228 }
anatofuz
parents:
diff changeset
229 } else {
anatofuz
parents:
diff changeset
230 LastBlockComment = nullptr;
anatofuz
parents:
diff changeset
231 }
anatofuz
parents:
diff changeset
232 }
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
233
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
234 // Compute conditional nesting level
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
235 // Level is increased for each conditional, unless this conditional continues
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
236 // a chain of conditional, i.e. starts immediately after the colon of another
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
237 // conditional.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
238 SmallVector<bool, 16> ScopeStack;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
239 int ConditionalsLevel = 0;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
240 for (auto &Change : Changes) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
241 for (unsigned i = 0, e = Change.Tok->FakeLParens.size(); i != e; ++i) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
242 bool isNestedConditional =
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
243 Change.Tok->FakeLParens[e - 1 - i] == prec::Conditional &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
244 !(i == 0 && Change.Tok->Previous &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
245 Change.Tok->Previous->is(TT_ConditionalExpr) &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
246 Change.Tok->Previous->is(tok::colon));
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
247 if (isNestedConditional)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
248 ++ConditionalsLevel;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
249 ScopeStack.push_back(isNestedConditional);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
250 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
251
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
252 Change.ConditionalsLevel = ConditionalsLevel;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
253
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
254 for (unsigned i = Change.Tok->FakeRParens; i > 0 && ScopeStack.size();
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
255 --i) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
256 if (ScopeStack.pop_back_val())
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
257 --ConditionalsLevel;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
258 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
259 }
150
anatofuz
parents:
diff changeset
260 }
anatofuz
parents:
diff changeset
261
anatofuz
parents:
diff changeset
262 // Align a single sequence of tokens, see AlignTokens below.
anatofuz
parents:
diff changeset
263 template <typename F>
anatofuz
parents:
diff changeset
264 static void
anatofuz
parents:
diff changeset
265 AlignTokenSequence(unsigned Start, unsigned End, unsigned Column, F &&Matches,
anatofuz
parents:
diff changeset
266 SmallVector<WhitespaceManager::Change, 16> &Changes) {
anatofuz
parents:
diff changeset
267 bool FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
268 int Shift = 0;
anatofuz
parents:
diff changeset
269
anatofuz
parents:
diff changeset
270 // ScopeStack keeps track of the current scope depth. It contains indices of
anatofuz
parents:
diff changeset
271 // the first token on each scope.
anatofuz
parents:
diff changeset
272 // We only run the "Matches" function on tokens from the outer-most scope.
anatofuz
parents:
diff changeset
273 // However, we do need to pay special attention to one class of tokens
anatofuz
parents:
diff changeset
274 // that are not in the outer-most scope, and that is function parameters
anatofuz
parents:
diff changeset
275 // which are split across multiple lines, as illustrated by this example:
anatofuz
parents:
diff changeset
276 // double a(int x);
anatofuz
parents:
diff changeset
277 // int b(int y,
anatofuz
parents:
diff changeset
278 // double z);
anatofuz
parents:
diff changeset
279 // In the above example, we need to take special care to ensure that
anatofuz
parents:
diff changeset
280 // 'double z' is indented along with it's owning function 'b'.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
281 // Special handling is required for 'nested' ternary operators.
150
anatofuz
parents:
diff changeset
282 SmallVector<unsigned, 16> ScopeStack;
anatofuz
parents:
diff changeset
283
anatofuz
parents:
diff changeset
284 for (unsigned i = Start; i != End; ++i) {
anatofuz
parents:
diff changeset
285 if (ScopeStack.size() != 0 &&
anatofuz
parents:
diff changeset
286 Changes[i].indentAndNestingLevel() <
anatofuz
parents:
diff changeset
287 Changes[ScopeStack.back()].indentAndNestingLevel())
anatofuz
parents:
diff changeset
288 ScopeStack.pop_back();
anatofuz
parents:
diff changeset
289
anatofuz
parents:
diff changeset
290 // Compare current token to previous non-comment token to ensure whether
anatofuz
parents:
diff changeset
291 // it is in a deeper scope or not.
anatofuz
parents:
diff changeset
292 unsigned PreviousNonComment = i - 1;
anatofuz
parents:
diff changeset
293 while (PreviousNonComment > Start &&
anatofuz
parents:
diff changeset
294 Changes[PreviousNonComment].Tok->is(tok::comment))
anatofuz
parents:
diff changeset
295 PreviousNonComment--;
anatofuz
parents:
diff changeset
296 if (i != Start && Changes[i].indentAndNestingLevel() >
anatofuz
parents:
diff changeset
297 Changes[PreviousNonComment].indentAndNestingLevel())
anatofuz
parents:
diff changeset
298 ScopeStack.push_back(i);
anatofuz
parents:
diff changeset
299
anatofuz
parents:
diff changeset
300 bool InsideNestedScope = ScopeStack.size() != 0;
anatofuz
parents:
diff changeset
301
anatofuz
parents:
diff changeset
302 if (Changes[i].NewlinesBefore > 0 && !InsideNestedScope) {
anatofuz
parents:
diff changeset
303 Shift = 0;
anatofuz
parents:
diff changeset
304 FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
305 }
anatofuz
parents:
diff changeset
306
anatofuz
parents:
diff changeset
307 // If this is the first matching token to be aligned, remember by how many
anatofuz
parents:
diff changeset
308 // spaces it has to be shifted, so the rest of the changes on the line are
anatofuz
parents:
diff changeset
309 // shifted by the same amount
anatofuz
parents:
diff changeset
310 if (!FoundMatchOnLine && !InsideNestedScope && Matches(Changes[i])) {
anatofuz
parents:
diff changeset
311 FoundMatchOnLine = true;
anatofuz
parents:
diff changeset
312 Shift = Column - Changes[i].StartOfTokenColumn;
anatofuz
parents:
diff changeset
313 Changes[i].Spaces += Shift;
anatofuz
parents:
diff changeset
314 }
anatofuz
parents:
diff changeset
315
anatofuz
parents:
diff changeset
316 // This is for function parameters that are split across multiple lines,
anatofuz
parents:
diff changeset
317 // as mentioned in the ScopeStack comment.
anatofuz
parents:
diff changeset
318 if (InsideNestedScope && Changes[i].NewlinesBefore > 0) {
anatofuz
parents:
diff changeset
319 unsigned ScopeStart = ScopeStack.back();
anatofuz
parents:
diff changeset
320 if (Changes[ScopeStart - 1].Tok->is(TT_FunctionDeclarationName) ||
anatofuz
parents:
diff changeset
321 (ScopeStart > Start + 1 &&
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
322 Changes[ScopeStart - 2].Tok->is(TT_FunctionDeclarationName)) ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
323 Changes[i].Tok->is(TT_ConditionalExpr) ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
324 (Changes[i].Tok->Previous &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
325 Changes[i].Tok->Previous->is(TT_ConditionalExpr)))
150
anatofuz
parents:
diff changeset
326 Changes[i].Spaces += Shift;
anatofuz
parents:
diff changeset
327 }
anatofuz
parents:
diff changeset
328
anatofuz
parents:
diff changeset
329 assert(Shift >= 0);
anatofuz
parents:
diff changeset
330 Changes[i].StartOfTokenColumn += Shift;
anatofuz
parents:
diff changeset
331 if (i + 1 != Changes.size())
anatofuz
parents:
diff changeset
332 Changes[i + 1].PreviousEndOfTokenColumn += Shift;
anatofuz
parents:
diff changeset
333 }
anatofuz
parents:
diff changeset
334 }
anatofuz
parents:
diff changeset
335
anatofuz
parents:
diff changeset
336 // Walk through a subset of the changes, starting at StartAt, and find
anatofuz
parents:
diff changeset
337 // sequences of matching tokens to align. To do so, keep track of the lines and
anatofuz
parents:
diff changeset
338 // whether or not a matching token was found on a line. If a matching token is
anatofuz
parents:
diff changeset
339 // found, extend the current sequence. If the current line cannot be part of a
anatofuz
parents:
diff changeset
340 // sequence, e.g. because there is an empty line before it or it contains only
anatofuz
parents:
diff changeset
341 // non-matching tokens, finalize the previous sequence.
anatofuz
parents:
diff changeset
342 // The value returned is the token on which we stopped, either because we
anatofuz
parents:
diff changeset
343 // exhausted all items inside Changes, or because we hit a scope level higher
anatofuz
parents:
diff changeset
344 // than our initial scope.
anatofuz
parents:
diff changeset
345 // This function is recursive. Each invocation processes only the scope level
anatofuz
parents:
diff changeset
346 // equal to the initial level, which is the level of Changes[StartAt].
anatofuz
parents:
diff changeset
347 // If we encounter a scope level greater than the initial level, then we call
anatofuz
parents:
diff changeset
348 // ourselves recursively, thereby avoiding the pollution of the current state
anatofuz
parents:
diff changeset
349 // with the alignment requirements of the nested sub-level. This recursive
anatofuz
parents:
diff changeset
350 // behavior is necessary for aligning function prototypes that have one or more
anatofuz
parents:
diff changeset
351 // arguments.
anatofuz
parents:
diff changeset
352 // If this function encounters a scope level less than the initial level,
anatofuz
parents:
diff changeset
353 // it returns the current position.
anatofuz
parents:
diff changeset
354 // There is a non-obvious subtlety in the recursive behavior: Even though we
anatofuz
parents:
diff changeset
355 // defer processing of nested levels to recursive invocations of this
anatofuz
parents:
diff changeset
356 // function, when it comes time to align a sequence of tokens, we run the
anatofuz
parents:
diff changeset
357 // alignment on the entire sequence, including the nested levels.
anatofuz
parents:
diff changeset
358 // When doing so, most of the nested tokens are skipped, because their
anatofuz
parents:
diff changeset
359 // alignment was already handled by the recursive invocations of this function.
anatofuz
parents:
diff changeset
360 // However, the special exception is that we do NOT skip function parameters
anatofuz
parents:
diff changeset
361 // that are split across multiple lines. See the test case in FormatTest.cpp
anatofuz
parents:
diff changeset
362 // that mentions "split function parameter alignment" for an example of this.
anatofuz
parents:
diff changeset
363 template <typename F>
anatofuz
parents:
diff changeset
364 static unsigned AlignTokens(const FormatStyle &Style, F &&Matches,
anatofuz
parents:
diff changeset
365 SmallVector<WhitespaceManager::Change, 16> &Changes,
anatofuz
parents:
diff changeset
366 unsigned StartAt) {
anatofuz
parents:
diff changeset
367 unsigned MinColumn = 0;
anatofuz
parents:
diff changeset
368 unsigned MaxColumn = UINT_MAX;
anatofuz
parents:
diff changeset
369
anatofuz
parents:
diff changeset
370 // Line number of the start and the end of the current token sequence.
anatofuz
parents:
diff changeset
371 unsigned StartOfSequence = 0;
anatofuz
parents:
diff changeset
372 unsigned EndOfSequence = 0;
anatofuz
parents:
diff changeset
373
anatofuz
parents:
diff changeset
374 // Measure the scope level (i.e. depth of (), [], {}) of the first token, and
anatofuz
parents:
diff changeset
375 // abort when we hit any token in a higher scope than the starting one.
anatofuz
parents:
diff changeset
376 auto IndentAndNestingLevel = StartAt < Changes.size()
anatofuz
parents:
diff changeset
377 ? Changes[StartAt].indentAndNestingLevel()
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
378 : std::tuple<unsigned, unsigned, unsigned>();
150
anatofuz
parents:
diff changeset
379
anatofuz
parents:
diff changeset
380 // Keep track of the number of commas before the matching tokens, we will only
anatofuz
parents:
diff changeset
381 // align a sequence of matching tokens if they are preceded by the same number
anatofuz
parents:
diff changeset
382 // of commas.
anatofuz
parents:
diff changeset
383 unsigned CommasBeforeLastMatch = 0;
anatofuz
parents:
diff changeset
384 unsigned CommasBeforeMatch = 0;
anatofuz
parents:
diff changeset
385
anatofuz
parents:
diff changeset
386 // Whether a matching token has been found on the current line.
anatofuz
parents:
diff changeset
387 bool FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
388
anatofuz
parents:
diff changeset
389 // Aligns a sequence of matching tokens, on the MinColumn column.
anatofuz
parents:
diff changeset
390 //
anatofuz
parents:
diff changeset
391 // Sequences start from the first matching token to align, and end at the
anatofuz
parents:
diff changeset
392 // first token of the first line that doesn't need to be aligned.
anatofuz
parents:
diff changeset
393 //
anatofuz
parents:
diff changeset
394 // We need to adjust the StartOfTokenColumn of each Change that is on a line
anatofuz
parents:
diff changeset
395 // containing any matching token to be aligned and located after such token.
anatofuz
parents:
diff changeset
396 auto AlignCurrentSequence = [&] {
anatofuz
parents:
diff changeset
397 if (StartOfSequence > 0 && StartOfSequence < EndOfSequence)
anatofuz
parents:
diff changeset
398 AlignTokenSequence(StartOfSequence, EndOfSequence, MinColumn, Matches,
anatofuz
parents:
diff changeset
399 Changes);
anatofuz
parents:
diff changeset
400 MinColumn = 0;
anatofuz
parents:
diff changeset
401 MaxColumn = UINT_MAX;
anatofuz
parents:
diff changeset
402 StartOfSequence = 0;
anatofuz
parents:
diff changeset
403 EndOfSequence = 0;
anatofuz
parents:
diff changeset
404 };
anatofuz
parents:
diff changeset
405
anatofuz
parents:
diff changeset
406 unsigned i = StartAt;
anatofuz
parents:
diff changeset
407 for (unsigned e = Changes.size(); i != e; ++i) {
anatofuz
parents:
diff changeset
408 if (Changes[i].indentAndNestingLevel() < IndentAndNestingLevel)
anatofuz
parents:
diff changeset
409 break;
anatofuz
parents:
diff changeset
410
anatofuz
parents:
diff changeset
411 if (Changes[i].NewlinesBefore != 0) {
anatofuz
parents:
diff changeset
412 CommasBeforeMatch = 0;
anatofuz
parents:
diff changeset
413 EndOfSequence = i;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
414 // If there is a blank line, there is a forced-align-break (eg,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
415 // preprocessor), or if the last line didn't contain any matching token,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
416 // the sequence ends here.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
417 if (Changes[i].NewlinesBefore > 1 ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
418 Changes[i].Tok->MustBreakAlignBefore || !FoundMatchOnLine)
150
anatofuz
parents:
diff changeset
419 AlignCurrentSequence();
anatofuz
parents:
diff changeset
420
anatofuz
parents:
diff changeset
421 FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
422 }
anatofuz
parents:
diff changeset
423
anatofuz
parents:
diff changeset
424 if (Changes[i].Tok->is(tok::comma)) {
anatofuz
parents:
diff changeset
425 ++CommasBeforeMatch;
anatofuz
parents:
diff changeset
426 } else if (Changes[i].indentAndNestingLevel() > IndentAndNestingLevel) {
anatofuz
parents:
diff changeset
427 // Call AlignTokens recursively, skipping over this scope block.
anatofuz
parents:
diff changeset
428 unsigned StoppedAt = AlignTokens(Style, Matches, Changes, i);
anatofuz
parents:
diff changeset
429 i = StoppedAt - 1;
anatofuz
parents:
diff changeset
430 continue;
anatofuz
parents:
diff changeset
431 }
anatofuz
parents:
diff changeset
432
anatofuz
parents:
diff changeset
433 if (!Matches(Changes[i]))
anatofuz
parents:
diff changeset
434 continue;
anatofuz
parents:
diff changeset
435
anatofuz
parents:
diff changeset
436 // If there is more than one matching token per line, or if the number of
anatofuz
parents:
diff changeset
437 // preceding commas, do not match anymore, end the sequence.
anatofuz
parents:
diff changeset
438 if (FoundMatchOnLine || CommasBeforeMatch != CommasBeforeLastMatch)
anatofuz
parents:
diff changeset
439 AlignCurrentSequence();
anatofuz
parents:
diff changeset
440
anatofuz
parents:
diff changeset
441 CommasBeforeLastMatch = CommasBeforeMatch;
anatofuz
parents:
diff changeset
442 FoundMatchOnLine = true;
anatofuz
parents:
diff changeset
443
anatofuz
parents:
diff changeset
444 if (StartOfSequence == 0)
anatofuz
parents:
diff changeset
445 StartOfSequence = i;
anatofuz
parents:
diff changeset
446
anatofuz
parents:
diff changeset
447 unsigned ChangeMinColumn = Changes[i].StartOfTokenColumn;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
448 int LineLengthAfter = Changes[i].TokenLength;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
449 for (unsigned j = i + 1; j != e && Changes[j].NewlinesBefore == 0; ++j) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
450 LineLengthAfter += Changes[j].Spaces;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
451 // Changes are generally 1:1 with the tokens, but a change could also be
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
452 // inside of a token, in which case it's counted more than once: once for
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
453 // the whitespace surrounding the token (!IsInsideToken) and once for
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
454 // each whitespace change within it (IsInsideToken).
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
455 // Therefore, changes inside of a token should only count the space.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
456 if (!Changes[j].IsInsideToken)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
457 LineLengthAfter += Changes[j].TokenLength;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
458 }
150
anatofuz
parents:
diff changeset
459 unsigned ChangeMaxColumn = Style.ColumnLimit - LineLengthAfter;
anatofuz
parents:
diff changeset
460
anatofuz
parents:
diff changeset
461 // If we are restricted by the maximum column width, end the sequence.
anatofuz
parents:
diff changeset
462 if (ChangeMinColumn > MaxColumn || ChangeMaxColumn < MinColumn ||
anatofuz
parents:
diff changeset
463 CommasBeforeLastMatch != CommasBeforeMatch) {
anatofuz
parents:
diff changeset
464 AlignCurrentSequence();
anatofuz
parents:
diff changeset
465 StartOfSequence = i;
anatofuz
parents:
diff changeset
466 }
anatofuz
parents:
diff changeset
467
anatofuz
parents:
diff changeset
468 MinColumn = std::max(MinColumn, ChangeMinColumn);
anatofuz
parents:
diff changeset
469 MaxColumn = std::min(MaxColumn, ChangeMaxColumn);
anatofuz
parents:
diff changeset
470 }
anatofuz
parents:
diff changeset
471
anatofuz
parents:
diff changeset
472 EndOfSequence = i;
anatofuz
parents:
diff changeset
473 AlignCurrentSequence();
anatofuz
parents:
diff changeset
474 return i;
anatofuz
parents:
diff changeset
475 }
anatofuz
parents:
diff changeset
476
anatofuz
parents:
diff changeset
477 // Aligns a sequence of matching tokens, on the MinColumn column.
anatofuz
parents:
diff changeset
478 //
anatofuz
parents:
diff changeset
479 // Sequences start from the first matching token to align, and end at the
anatofuz
parents:
diff changeset
480 // first token of the first line that doesn't need to be aligned.
anatofuz
parents:
diff changeset
481 //
anatofuz
parents:
diff changeset
482 // We need to adjust the StartOfTokenColumn of each Change that is on a line
anatofuz
parents:
diff changeset
483 // containing any matching token to be aligned and located after such token.
anatofuz
parents:
diff changeset
484 static void AlignMacroSequence(
anatofuz
parents:
diff changeset
485 unsigned &StartOfSequence, unsigned &EndOfSequence, unsigned &MinColumn,
anatofuz
parents:
diff changeset
486 unsigned &MaxColumn, bool &FoundMatchOnLine,
anatofuz
parents:
diff changeset
487 std::function<bool(const WhitespaceManager::Change &C)> AlignMacrosMatches,
anatofuz
parents:
diff changeset
488 SmallVector<WhitespaceManager::Change, 16> &Changes) {
anatofuz
parents:
diff changeset
489 if (StartOfSequence > 0 && StartOfSequence < EndOfSequence) {
anatofuz
parents:
diff changeset
490
anatofuz
parents:
diff changeset
491 FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
492 int Shift = 0;
anatofuz
parents:
diff changeset
493
anatofuz
parents:
diff changeset
494 for (unsigned I = StartOfSequence; I != EndOfSequence; ++I) {
anatofuz
parents:
diff changeset
495 if (Changes[I].NewlinesBefore > 0) {
anatofuz
parents:
diff changeset
496 Shift = 0;
anatofuz
parents:
diff changeset
497 FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
498 }
anatofuz
parents:
diff changeset
499
anatofuz
parents:
diff changeset
500 // If this is the first matching token to be aligned, remember by how many
anatofuz
parents:
diff changeset
501 // spaces it has to be shifted, so the rest of the changes on the line are
anatofuz
parents:
diff changeset
502 // shifted by the same amount
anatofuz
parents:
diff changeset
503 if (!FoundMatchOnLine && AlignMacrosMatches(Changes[I])) {
anatofuz
parents:
diff changeset
504 FoundMatchOnLine = true;
anatofuz
parents:
diff changeset
505 Shift = MinColumn - Changes[I].StartOfTokenColumn;
anatofuz
parents:
diff changeset
506 Changes[I].Spaces += Shift;
anatofuz
parents:
diff changeset
507 }
anatofuz
parents:
diff changeset
508
anatofuz
parents:
diff changeset
509 assert(Shift >= 0);
anatofuz
parents:
diff changeset
510 Changes[I].StartOfTokenColumn += Shift;
anatofuz
parents:
diff changeset
511 if (I + 1 != Changes.size())
anatofuz
parents:
diff changeset
512 Changes[I + 1].PreviousEndOfTokenColumn += Shift;
anatofuz
parents:
diff changeset
513 }
anatofuz
parents:
diff changeset
514 }
anatofuz
parents:
diff changeset
515
anatofuz
parents:
diff changeset
516 MinColumn = 0;
anatofuz
parents:
diff changeset
517 MaxColumn = UINT_MAX;
anatofuz
parents:
diff changeset
518 StartOfSequence = 0;
anatofuz
parents:
diff changeset
519 EndOfSequence = 0;
anatofuz
parents:
diff changeset
520 }
anatofuz
parents:
diff changeset
521
anatofuz
parents:
diff changeset
522 void WhitespaceManager::alignConsecutiveMacros() {
anatofuz
parents:
diff changeset
523 if (!Style.AlignConsecutiveMacros)
anatofuz
parents:
diff changeset
524 return;
anatofuz
parents:
diff changeset
525
anatofuz
parents:
diff changeset
526 auto AlignMacrosMatches = [](const Change &C) {
anatofuz
parents:
diff changeset
527 const FormatToken *Current = C.Tok;
anatofuz
parents:
diff changeset
528 unsigned SpacesRequiredBefore = 1;
anatofuz
parents:
diff changeset
529
anatofuz
parents:
diff changeset
530 if (Current->SpacesRequiredBefore == 0 || !Current->Previous)
anatofuz
parents:
diff changeset
531 return false;
anatofuz
parents:
diff changeset
532
anatofuz
parents:
diff changeset
533 Current = Current->Previous;
anatofuz
parents:
diff changeset
534
anatofuz
parents:
diff changeset
535 // If token is a ")", skip over the parameter list, to the
anatofuz
parents:
diff changeset
536 // token that precedes the "("
anatofuz
parents:
diff changeset
537 if (Current->is(tok::r_paren) && Current->MatchingParen) {
anatofuz
parents:
diff changeset
538 Current = Current->MatchingParen->Previous;
anatofuz
parents:
diff changeset
539 SpacesRequiredBefore = 0;
anatofuz
parents:
diff changeset
540 }
anatofuz
parents:
diff changeset
541
anatofuz
parents:
diff changeset
542 if (!Current || !Current->is(tok::identifier))
anatofuz
parents:
diff changeset
543 return false;
anatofuz
parents:
diff changeset
544
anatofuz
parents:
diff changeset
545 if (!Current->Previous || !Current->Previous->is(tok::pp_define))
anatofuz
parents:
diff changeset
546 return false;
anatofuz
parents:
diff changeset
547
anatofuz
parents:
diff changeset
548 // For a macro function, 0 spaces are required between the
anatofuz
parents:
diff changeset
549 // identifier and the lparen that opens the parameter list.
anatofuz
parents:
diff changeset
550 // For a simple macro, 1 space is required between the
anatofuz
parents:
diff changeset
551 // identifier and the first token of the defined value.
anatofuz
parents:
diff changeset
552 return Current->Next->SpacesRequiredBefore == SpacesRequiredBefore;
anatofuz
parents:
diff changeset
553 };
anatofuz
parents:
diff changeset
554
anatofuz
parents:
diff changeset
555 unsigned MinColumn = 0;
anatofuz
parents:
diff changeset
556 unsigned MaxColumn = UINT_MAX;
anatofuz
parents:
diff changeset
557
anatofuz
parents:
diff changeset
558 // Start and end of the token sequence we're processing.
anatofuz
parents:
diff changeset
559 unsigned StartOfSequence = 0;
anatofuz
parents:
diff changeset
560 unsigned EndOfSequence = 0;
anatofuz
parents:
diff changeset
561
anatofuz
parents:
diff changeset
562 // Whether a matching token has been found on the current line.
anatofuz
parents:
diff changeset
563 bool FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
564
anatofuz
parents:
diff changeset
565 unsigned I = 0;
anatofuz
parents:
diff changeset
566 for (unsigned E = Changes.size(); I != E; ++I) {
anatofuz
parents:
diff changeset
567 if (Changes[I].NewlinesBefore != 0) {
anatofuz
parents:
diff changeset
568 EndOfSequence = I;
anatofuz
parents:
diff changeset
569 // If there is a blank line, or if the last line didn't contain any
anatofuz
parents:
diff changeset
570 // matching token, the sequence ends here.
anatofuz
parents:
diff changeset
571 if (Changes[I].NewlinesBefore > 1 || !FoundMatchOnLine)
anatofuz
parents:
diff changeset
572 AlignMacroSequence(StartOfSequence, EndOfSequence, MinColumn, MaxColumn,
anatofuz
parents:
diff changeset
573 FoundMatchOnLine, AlignMacrosMatches, Changes);
anatofuz
parents:
diff changeset
574
anatofuz
parents:
diff changeset
575 FoundMatchOnLine = false;
anatofuz
parents:
diff changeset
576 }
anatofuz
parents:
diff changeset
577
anatofuz
parents:
diff changeset
578 if (!AlignMacrosMatches(Changes[I]))
anatofuz
parents:
diff changeset
579 continue;
anatofuz
parents:
diff changeset
580
anatofuz
parents:
diff changeset
581 FoundMatchOnLine = true;
anatofuz
parents:
diff changeset
582
anatofuz
parents:
diff changeset
583 if (StartOfSequence == 0)
anatofuz
parents:
diff changeset
584 StartOfSequence = I;
anatofuz
parents:
diff changeset
585
anatofuz
parents:
diff changeset
586 unsigned ChangeMinColumn = Changes[I].StartOfTokenColumn;
anatofuz
parents:
diff changeset
587 int LineLengthAfter = -Changes[I].Spaces;
anatofuz
parents:
diff changeset
588 for (unsigned j = I; j != E && Changes[j].NewlinesBefore == 0; ++j)
anatofuz
parents:
diff changeset
589 LineLengthAfter += Changes[j].Spaces + Changes[j].TokenLength;
anatofuz
parents:
diff changeset
590 unsigned ChangeMaxColumn = Style.ColumnLimit - LineLengthAfter;
anatofuz
parents:
diff changeset
591
anatofuz
parents:
diff changeset
592 MinColumn = std::max(MinColumn, ChangeMinColumn);
anatofuz
parents:
diff changeset
593 MaxColumn = std::min(MaxColumn, ChangeMaxColumn);
anatofuz
parents:
diff changeset
594 }
anatofuz
parents:
diff changeset
595
anatofuz
parents:
diff changeset
596 EndOfSequence = I;
anatofuz
parents:
diff changeset
597 AlignMacroSequence(StartOfSequence, EndOfSequence, MinColumn, MaxColumn,
anatofuz
parents:
diff changeset
598 FoundMatchOnLine, AlignMacrosMatches, Changes);
anatofuz
parents:
diff changeset
599 }
anatofuz
parents:
diff changeset
600
anatofuz
parents:
diff changeset
601 void WhitespaceManager::alignConsecutiveAssignments() {
anatofuz
parents:
diff changeset
602 if (!Style.AlignConsecutiveAssignments)
anatofuz
parents:
diff changeset
603 return;
anatofuz
parents:
diff changeset
604
anatofuz
parents:
diff changeset
605 AlignTokens(
anatofuz
parents:
diff changeset
606 Style,
anatofuz
parents:
diff changeset
607 [&](const Change &C) {
anatofuz
parents:
diff changeset
608 // Do not align on equal signs that are first on a line.
anatofuz
parents:
diff changeset
609 if (C.NewlinesBefore > 0)
anatofuz
parents:
diff changeset
610 return false;
anatofuz
parents:
diff changeset
611
anatofuz
parents:
diff changeset
612 // Do not align on equal signs that are last on a line.
anatofuz
parents:
diff changeset
613 if (&C != &Changes.back() && (&C + 1)->NewlinesBefore > 0)
anatofuz
parents:
diff changeset
614 return false;
anatofuz
parents:
diff changeset
615
anatofuz
parents:
diff changeset
616 return C.Tok->is(tok::equal);
anatofuz
parents:
diff changeset
617 },
anatofuz
parents:
diff changeset
618 Changes, /*StartAt=*/0);
anatofuz
parents:
diff changeset
619 }
anatofuz
parents:
diff changeset
620
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
621 void WhitespaceManager::alignConsecutiveBitFields() {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
622 if (!Style.AlignConsecutiveBitFields)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
623 return;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
624
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
625 AlignTokens(
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
626 Style,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
627 [&](Change const &C) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
628 // Do not align on ':' that is first on a line.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
629 if (C.NewlinesBefore > 0)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
630 return false;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
631
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
632 // Do not align on ':' that is last on a line.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
633 if (&C != &Changes.back() && (&C + 1)->NewlinesBefore > 0)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
634 return false;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
635
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
636 return C.Tok->is(TT_BitFieldColon);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
637 },
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
638 Changes, /*StartAt=*/0);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
639 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
640
150
anatofuz
parents:
diff changeset
641 void WhitespaceManager::alignConsecutiveDeclarations() {
anatofuz
parents:
diff changeset
642 if (!Style.AlignConsecutiveDeclarations)
anatofuz
parents:
diff changeset
643 return;
anatofuz
parents:
diff changeset
644
anatofuz
parents:
diff changeset
645 // FIXME: Currently we don't handle properly the PointerAlignment: Right
anatofuz
parents:
diff changeset
646 // The * and & are not aligned and are left dangling. Something has to be done
anatofuz
parents:
diff changeset
647 // about it, but it raises the question of alignment of code like:
anatofuz
parents:
diff changeset
648 // const char* const* v1;
anatofuz
parents:
diff changeset
649 // float const* v2;
anatofuz
parents:
diff changeset
650 // SomeVeryLongType const& v3;
anatofuz
parents:
diff changeset
651 AlignTokens(
anatofuz
parents:
diff changeset
652 Style,
anatofuz
parents:
diff changeset
653 [](Change const &C) {
anatofuz
parents:
diff changeset
654 // tok::kw_operator is necessary for aligning operator overload
anatofuz
parents:
diff changeset
655 // definitions.
anatofuz
parents:
diff changeset
656 if (C.Tok->isOneOf(TT_FunctionDeclarationName, tok::kw_operator))
anatofuz
parents:
diff changeset
657 return true;
anatofuz
parents:
diff changeset
658 if (C.Tok->isNot(TT_StartOfName))
anatofuz
parents:
diff changeset
659 return false;
anatofuz
parents:
diff changeset
660 // Check if there is a subsequent name that starts the same declaration.
anatofuz
parents:
diff changeset
661 for (FormatToken *Next = C.Tok->Next; Next; Next = Next->Next) {
anatofuz
parents:
diff changeset
662 if (Next->is(tok::comment))
anatofuz
parents:
diff changeset
663 continue;
anatofuz
parents:
diff changeset
664 if (!Next->Tok.getIdentifierInfo())
anatofuz
parents:
diff changeset
665 break;
anatofuz
parents:
diff changeset
666 if (Next->isOneOf(TT_StartOfName, TT_FunctionDeclarationName,
anatofuz
parents:
diff changeset
667 tok::kw_operator))
anatofuz
parents:
diff changeset
668 return false;
anatofuz
parents:
diff changeset
669 }
anatofuz
parents:
diff changeset
670 return true;
anatofuz
parents:
diff changeset
671 },
anatofuz
parents:
diff changeset
672 Changes, /*StartAt=*/0);
anatofuz
parents:
diff changeset
673 }
anatofuz
parents:
diff changeset
674
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
675 void WhitespaceManager::alignChainedConditionals() {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
676 if (Style.BreakBeforeTernaryOperators) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
677 AlignTokens(
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
678 Style,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
679 [](Change const &C) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
680 // Align question operators and last colon
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
681 return C.Tok->is(TT_ConditionalExpr) &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
682 ((C.Tok->is(tok::question) && !C.NewlinesBefore) ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
683 (C.Tok->is(tok::colon) && C.Tok->Next &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
684 (C.Tok->Next->FakeLParens.size() == 0 ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
685 C.Tok->Next->FakeLParens.back() != prec::Conditional)));
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
686 },
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
687 Changes, /*StartAt=*/0);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
688 } else {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
689 static auto AlignWrappedOperand = [](Change const &C) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
690 auto Previous = C.Tok->getPreviousNonComment(); // Previous;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
691 return C.NewlinesBefore && Previous && Previous->is(TT_ConditionalExpr) &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
692 (Previous->is(tok::question) ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
693 (Previous->is(tok::colon) &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
694 (C.Tok->FakeLParens.size() == 0 ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
695 C.Tok->FakeLParens.back() != prec::Conditional)));
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
696 };
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
697 // Ensure we keep alignment of wrapped operands with non-wrapped operands
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
698 // Since we actually align the operators, the wrapped operands need the
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
699 // extra offset to be properly aligned.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
700 for (Change &C : Changes) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
701 if (AlignWrappedOperand(C))
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
702 C.StartOfTokenColumn -= 2;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
703 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
704 AlignTokens(
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
705 Style,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
706 [this](Change const &C) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
707 // Align question operators if next operand is not wrapped, as
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
708 // well as wrapped operands after question operator or last
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
709 // colon in conditional sequence
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
710 return (C.Tok->is(TT_ConditionalExpr) && C.Tok->is(tok::question) &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
711 &C != &Changes.back() && (&C + 1)->NewlinesBefore == 0 &&
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
712 !(&C + 1)->IsTrailingComment) ||
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
713 AlignWrappedOperand(C);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
714 },
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
715 Changes, /*StartAt=*/0);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
716 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
717 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
718
150
anatofuz
parents:
diff changeset
719 void WhitespaceManager::alignTrailingComments() {
anatofuz
parents:
diff changeset
720 unsigned MinColumn = 0;
anatofuz
parents:
diff changeset
721 unsigned MaxColumn = UINT_MAX;
anatofuz
parents:
diff changeset
722 unsigned StartOfSequence = 0;
anatofuz
parents:
diff changeset
723 bool BreakBeforeNext = false;
anatofuz
parents:
diff changeset
724 unsigned Newlines = 0;
anatofuz
parents:
diff changeset
725 for (unsigned i = 0, e = Changes.size(); i != e; ++i) {
anatofuz
parents:
diff changeset
726 if (Changes[i].StartOfBlockComment)
anatofuz
parents:
diff changeset
727 continue;
anatofuz
parents:
diff changeset
728 Newlines += Changes[i].NewlinesBefore;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
729 if (Changes[i].Tok->MustBreakAlignBefore)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
730 BreakBeforeNext = true;
150
anatofuz
parents:
diff changeset
731 if (!Changes[i].IsTrailingComment)
anatofuz
parents:
diff changeset
732 continue;
anatofuz
parents:
diff changeset
733
anatofuz
parents:
diff changeset
734 unsigned ChangeMinColumn = Changes[i].StartOfTokenColumn;
anatofuz
parents:
diff changeset
735 unsigned ChangeMaxColumn;
anatofuz
parents:
diff changeset
736
anatofuz
parents:
diff changeset
737 if (Style.ColumnLimit == 0)
anatofuz
parents:
diff changeset
738 ChangeMaxColumn = UINT_MAX;
anatofuz
parents:
diff changeset
739 else if (Style.ColumnLimit >= Changes[i].TokenLength)
anatofuz
parents:
diff changeset
740 ChangeMaxColumn = Style.ColumnLimit - Changes[i].TokenLength;
anatofuz
parents:
diff changeset
741 else
anatofuz
parents:
diff changeset
742 ChangeMaxColumn = ChangeMinColumn;
anatofuz
parents:
diff changeset
743
anatofuz
parents:
diff changeset
744 // If we don't create a replacement for this change, we have to consider
anatofuz
parents:
diff changeset
745 // it to be immovable.
anatofuz
parents:
diff changeset
746 if (!Changes[i].CreateReplacement)
anatofuz
parents:
diff changeset
747 ChangeMaxColumn = ChangeMinColumn;
anatofuz
parents:
diff changeset
748
anatofuz
parents:
diff changeset
749 if (i + 1 != e && Changes[i + 1].ContinuesPPDirective)
anatofuz
parents:
diff changeset
750 ChangeMaxColumn -= 2;
anatofuz
parents:
diff changeset
751 // If this comment follows an } in column 0, it probably documents the
anatofuz
parents:
diff changeset
752 // closing of a namespace and we don't want to align it.
anatofuz
parents:
diff changeset
753 bool FollowsRBraceInColumn0 = i > 0 && Changes[i].NewlinesBefore == 0 &&
anatofuz
parents:
diff changeset
754 Changes[i - 1].Tok->is(tok::r_brace) &&
anatofuz
parents:
diff changeset
755 Changes[i - 1].StartOfTokenColumn == 0;
anatofuz
parents:
diff changeset
756 bool WasAlignedWithStartOfNextLine = false;
anatofuz
parents:
diff changeset
757 if (Changes[i].NewlinesBefore == 1) { // A comment on its own line.
anatofuz
parents:
diff changeset
758 unsigned CommentColumn = SourceMgr.getSpellingColumnNumber(
anatofuz
parents:
diff changeset
759 Changes[i].OriginalWhitespaceRange.getEnd());
anatofuz
parents:
diff changeset
760 for (unsigned j = i + 1; j != e; ++j) {
anatofuz
parents:
diff changeset
761 if (Changes[j].Tok->is(tok::comment))
anatofuz
parents:
diff changeset
762 continue;
anatofuz
parents:
diff changeset
763
anatofuz
parents:
diff changeset
764 unsigned NextColumn = SourceMgr.getSpellingColumnNumber(
anatofuz
parents:
diff changeset
765 Changes[j].OriginalWhitespaceRange.getEnd());
anatofuz
parents:
diff changeset
766 // The start of the next token was previously aligned with the
anatofuz
parents:
diff changeset
767 // start of this comment.
anatofuz
parents:
diff changeset
768 WasAlignedWithStartOfNextLine =
anatofuz
parents:
diff changeset
769 CommentColumn == NextColumn ||
anatofuz
parents:
diff changeset
770 CommentColumn == NextColumn + Style.IndentWidth;
anatofuz
parents:
diff changeset
771 break;
anatofuz
parents:
diff changeset
772 }
anatofuz
parents:
diff changeset
773 }
anatofuz
parents:
diff changeset
774 if (!Style.AlignTrailingComments || FollowsRBraceInColumn0) {
anatofuz
parents:
diff changeset
775 alignTrailingComments(StartOfSequence, i, MinColumn);
anatofuz
parents:
diff changeset
776 MinColumn = ChangeMinColumn;
anatofuz
parents:
diff changeset
777 MaxColumn = ChangeMinColumn;
anatofuz
parents:
diff changeset
778 StartOfSequence = i;
anatofuz
parents:
diff changeset
779 } else if (BreakBeforeNext || Newlines > 1 ||
anatofuz
parents:
diff changeset
780 (ChangeMinColumn > MaxColumn || ChangeMaxColumn < MinColumn) ||
anatofuz
parents:
diff changeset
781 // Break the comment sequence if the previous line did not end
anatofuz
parents:
diff changeset
782 // in a trailing comment.
anatofuz
parents:
diff changeset
783 (Changes[i].NewlinesBefore == 1 && i > 0 &&
anatofuz
parents:
diff changeset
784 !Changes[i - 1].IsTrailingComment) ||
anatofuz
parents:
diff changeset
785 WasAlignedWithStartOfNextLine) {
anatofuz
parents:
diff changeset
786 alignTrailingComments(StartOfSequence, i, MinColumn);
anatofuz
parents:
diff changeset
787 MinColumn = ChangeMinColumn;
anatofuz
parents:
diff changeset
788 MaxColumn = ChangeMaxColumn;
anatofuz
parents:
diff changeset
789 StartOfSequence = i;
anatofuz
parents:
diff changeset
790 } else {
anatofuz
parents:
diff changeset
791 MinColumn = std::max(MinColumn, ChangeMinColumn);
anatofuz
parents:
diff changeset
792 MaxColumn = std::min(MaxColumn, ChangeMaxColumn);
anatofuz
parents:
diff changeset
793 }
anatofuz
parents:
diff changeset
794 BreakBeforeNext = (i == 0) || (Changes[i].NewlinesBefore > 1) ||
anatofuz
parents:
diff changeset
795 // Never start a sequence with a comment at the beginning
anatofuz
parents:
diff changeset
796 // of the line.
anatofuz
parents:
diff changeset
797 (Changes[i].NewlinesBefore == 1 && StartOfSequence == i);
anatofuz
parents:
diff changeset
798 Newlines = 0;
anatofuz
parents:
diff changeset
799 }
anatofuz
parents:
diff changeset
800 alignTrailingComments(StartOfSequence, Changes.size(), MinColumn);
anatofuz
parents:
diff changeset
801 }
anatofuz
parents:
diff changeset
802
anatofuz
parents:
diff changeset
803 void WhitespaceManager::alignTrailingComments(unsigned Start, unsigned End,
anatofuz
parents:
diff changeset
804 unsigned Column) {
anatofuz
parents:
diff changeset
805 for (unsigned i = Start; i != End; ++i) {
anatofuz
parents:
diff changeset
806 int Shift = 0;
anatofuz
parents:
diff changeset
807 if (Changes[i].IsTrailingComment) {
anatofuz
parents:
diff changeset
808 Shift = Column - Changes[i].StartOfTokenColumn;
anatofuz
parents:
diff changeset
809 }
anatofuz
parents:
diff changeset
810 if (Changes[i].StartOfBlockComment) {
anatofuz
parents:
diff changeset
811 Shift = Changes[i].IndentationOffset +
anatofuz
parents:
diff changeset
812 Changes[i].StartOfBlockComment->StartOfTokenColumn -
anatofuz
parents:
diff changeset
813 Changes[i].StartOfTokenColumn;
anatofuz
parents:
diff changeset
814 }
anatofuz
parents:
diff changeset
815 assert(Shift >= 0);
anatofuz
parents:
diff changeset
816 Changes[i].Spaces += Shift;
anatofuz
parents:
diff changeset
817 if (i + 1 != Changes.size())
anatofuz
parents:
diff changeset
818 Changes[i + 1].PreviousEndOfTokenColumn += Shift;
anatofuz
parents:
diff changeset
819 Changes[i].StartOfTokenColumn += Shift;
anatofuz
parents:
diff changeset
820 }
anatofuz
parents:
diff changeset
821 }
anatofuz
parents:
diff changeset
822
anatofuz
parents:
diff changeset
823 void WhitespaceManager::alignEscapedNewlines() {
anatofuz
parents:
diff changeset
824 if (Style.AlignEscapedNewlines == FormatStyle::ENAS_DontAlign)
anatofuz
parents:
diff changeset
825 return;
anatofuz
parents:
diff changeset
826
anatofuz
parents:
diff changeset
827 bool AlignLeft = Style.AlignEscapedNewlines == FormatStyle::ENAS_Left;
anatofuz
parents:
diff changeset
828 unsigned MaxEndOfLine = AlignLeft ? 0 : Style.ColumnLimit;
anatofuz
parents:
diff changeset
829 unsigned StartOfMacro = 0;
anatofuz
parents:
diff changeset
830 for (unsigned i = 1, e = Changes.size(); i < e; ++i) {
anatofuz
parents:
diff changeset
831 Change &C = Changes[i];
anatofuz
parents:
diff changeset
832 if (C.NewlinesBefore > 0) {
anatofuz
parents:
diff changeset
833 if (C.ContinuesPPDirective) {
anatofuz
parents:
diff changeset
834 MaxEndOfLine = std::max(C.PreviousEndOfTokenColumn + 2, MaxEndOfLine);
anatofuz
parents:
diff changeset
835 } else {
anatofuz
parents:
diff changeset
836 alignEscapedNewlines(StartOfMacro + 1, i, MaxEndOfLine);
anatofuz
parents:
diff changeset
837 MaxEndOfLine = AlignLeft ? 0 : Style.ColumnLimit;
anatofuz
parents:
diff changeset
838 StartOfMacro = i;
anatofuz
parents:
diff changeset
839 }
anatofuz
parents:
diff changeset
840 }
anatofuz
parents:
diff changeset
841 }
anatofuz
parents:
diff changeset
842 alignEscapedNewlines(StartOfMacro + 1, Changes.size(), MaxEndOfLine);
anatofuz
parents:
diff changeset
843 }
anatofuz
parents:
diff changeset
844
anatofuz
parents:
diff changeset
845 void WhitespaceManager::alignEscapedNewlines(unsigned Start, unsigned End,
anatofuz
parents:
diff changeset
846 unsigned Column) {
anatofuz
parents:
diff changeset
847 for (unsigned i = Start; i < End; ++i) {
anatofuz
parents:
diff changeset
848 Change &C = Changes[i];
anatofuz
parents:
diff changeset
849 if (C.NewlinesBefore > 0) {
anatofuz
parents:
diff changeset
850 assert(C.ContinuesPPDirective);
anatofuz
parents:
diff changeset
851 if (C.PreviousEndOfTokenColumn + 1 > Column)
anatofuz
parents:
diff changeset
852 C.EscapedNewlineColumn = 0;
anatofuz
parents:
diff changeset
853 else
anatofuz
parents:
diff changeset
854 C.EscapedNewlineColumn = Column;
anatofuz
parents:
diff changeset
855 }
anatofuz
parents:
diff changeset
856 }
anatofuz
parents:
diff changeset
857 }
anatofuz
parents:
diff changeset
858
anatofuz
parents:
diff changeset
859 void WhitespaceManager::generateChanges() {
anatofuz
parents:
diff changeset
860 for (unsigned i = 0, e = Changes.size(); i != e; ++i) {
anatofuz
parents:
diff changeset
861 const Change &C = Changes[i];
anatofuz
parents:
diff changeset
862 if (i > 0) {
anatofuz
parents:
diff changeset
863 assert(Changes[i - 1].OriginalWhitespaceRange.getBegin() !=
anatofuz
parents:
diff changeset
864 C.OriginalWhitespaceRange.getBegin() &&
anatofuz
parents:
diff changeset
865 "Generating two replacements for the same location");
anatofuz
parents:
diff changeset
866 }
anatofuz
parents:
diff changeset
867 if (C.CreateReplacement) {
anatofuz
parents:
diff changeset
868 std::string ReplacementText = C.PreviousLinePostfix;
anatofuz
parents:
diff changeset
869 if (C.ContinuesPPDirective)
anatofuz
parents:
diff changeset
870 appendEscapedNewlineText(ReplacementText, C.NewlinesBefore,
anatofuz
parents:
diff changeset
871 C.PreviousEndOfTokenColumn,
anatofuz
parents:
diff changeset
872 C.EscapedNewlineColumn);
anatofuz
parents:
diff changeset
873 else
anatofuz
parents:
diff changeset
874 appendNewlineText(ReplacementText, C.NewlinesBefore);
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
875 appendIndentText(
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
876 ReplacementText, C.Tok->IndentLevel, std::max(0, C.Spaces),
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
877 C.StartOfTokenColumn - std::max(0, C.Spaces), C.IsAligned);
150
anatofuz
parents:
diff changeset
878 ReplacementText.append(C.CurrentLinePrefix);
anatofuz
parents:
diff changeset
879 storeReplacement(C.OriginalWhitespaceRange, ReplacementText);
anatofuz
parents:
diff changeset
880 }
anatofuz
parents:
diff changeset
881 }
anatofuz
parents:
diff changeset
882 }
anatofuz
parents:
diff changeset
883
anatofuz
parents:
diff changeset
884 void WhitespaceManager::storeReplacement(SourceRange Range, StringRef Text) {
anatofuz
parents:
diff changeset
885 unsigned WhitespaceLength = SourceMgr.getFileOffset(Range.getEnd()) -
anatofuz
parents:
diff changeset
886 SourceMgr.getFileOffset(Range.getBegin());
anatofuz
parents:
diff changeset
887 // Don't create a replacement, if it does not change anything.
anatofuz
parents:
diff changeset
888 if (StringRef(SourceMgr.getCharacterData(Range.getBegin()),
anatofuz
parents:
diff changeset
889 WhitespaceLength) == Text)
anatofuz
parents:
diff changeset
890 return;
anatofuz
parents:
diff changeset
891 auto Err = Replaces.add(tooling::Replacement(
anatofuz
parents:
diff changeset
892 SourceMgr, CharSourceRange::getCharRange(Range), Text));
anatofuz
parents:
diff changeset
893 // FIXME: better error handling. For now, just print an error message in the
anatofuz
parents:
diff changeset
894 // release version.
anatofuz
parents:
diff changeset
895 if (Err) {
anatofuz
parents:
diff changeset
896 llvm::errs() << llvm::toString(std::move(Err)) << "\n";
anatofuz
parents:
diff changeset
897 assert(false);
anatofuz
parents:
diff changeset
898 }
anatofuz
parents:
diff changeset
899 }
anatofuz
parents:
diff changeset
900
anatofuz
parents:
diff changeset
901 void WhitespaceManager::appendNewlineText(std::string &Text,
anatofuz
parents:
diff changeset
902 unsigned Newlines) {
anatofuz
parents:
diff changeset
903 for (unsigned i = 0; i < Newlines; ++i)
anatofuz
parents:
diff changeset
904 Text.append(UseCRLF ? "\r\n" : "\n");
anatofuz
parents:
diff changeset
905 }
anatofuz
parents:
diff changeset
906
anatofuz
parents:
diff changeset
907 void WhitespaceManager::appendEscapedNewlineText(
anatofuz
parents:
diff changeset
908 std::string &Text, unsigned Newlines, unsigned PreviousEndOfTokenColumn,
anatofuz
parents:
diff changeset
909 unsigned EscapedNewlineColumn) {
anatofuz
parents:
diff changeset
910 if (Newlines > 0) {
anatofuz
parents:
diff changeset
911 unsigned Spaces =
anatofuz
parents:
diff changeset
912 std::max<int>(1, EscapedNewlineColumn - PreviousEndOfTokenColumn - 1);
anatofuz
parents:
diff changeset
913 for (unsigned i = 0; i < Newlines; ++i) {
anatofuz
parents:
diff changeset
914 Text.append(Spaces, ' ');
anatofuz
parents:
diff changeset
915 Text.append(UseCRLF ? "\\\r\n" : "\\\n");
anatofuz
parents:
diff changeset
916 Spaces = std::max<int>(0, EscapedNewlineColumn - 1);
anatofuz
parents:
diff changeset
917 }
anatofuz
parents:
diff changeset
918 }
anatofuz
parents:
diff changeset
919 }
anatofuz
parents:
diff changeset
920
anatofuz
parents:
diff changeset
921 void WhitespaceManager::appendIndentText(std::string &Text,
anatofuz
parents:
diff changeset
922 unsigned IndentLevel, unsigned Spaces,
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
923 unsigned WhitespaceStartColumn,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
924 bool IsAligned) {
150
anatofuz
parents:
diff changeset
925 switch (Style.UseTab) {
anatofuz
parents:
diff changeset
926 case FormatStyle::UT_Never:
anatofuz
parents:
diff changeset
927 Text.append(Spaces, ' ');
anatofuz
parents:
diff changeset
928 break;
anatofuz
parents:
diff changeset
929 case FormatStyle::UT_Always: {
anatofuz
parents:
diff changeset
930 if (Style.TabWidth) {
anatofuz
parents:
diff changeset
931 unsigned FirstTabWidth =
anatofuz
parents:
diff changeset
932 Style.TabWidth - WhitespaceStartColumn % Style.TabWidth;
anatofuz
parents:
diff changeset
933
anatofuz
parents:
diff changeset
934 // Insert only spaces when we want to end up before the next tab.
anatofuz
parents:
diff changeset
935 if (Spaces < FirstTabWidth || Spaces == 1) {
anatofuz
parents:
diff changeset
936 Text.append(Spaces, ' ');
anatofuz
parents:
diff changeset
937 break;
anatofuz
parents:
diff changeset
938 }
anatofuz
parents:
diff changeset
939 // Align to the next tab.
anatofuz
parents:
diff changeset
940 Spaces -= FirstTabWidth;
anatofuz
parents:
diff changeset
941 Text.append("\t");
anatofuz
parents:
diff changeset
942
anatofuz
parents:
diff changeset
943 Text.append(Spaces / Style.TabWidth, '\t');
anatofuz
parents:
diff changeset
944 Text.append(Spaces % Style.TabWidth, ' ');
anatofuz
parents:
diff changeset
945 } else if (Spaces == 1) {
anatofuz
parents:
diff changeset
946 Text.append(Spaces, ' ');
anatofuz
parents:
diff changeset
947 }
anatofuz
parents:
diff changeset
948 break;
anatofuz
parents:
diff changeset
949 }
anatofuz
parents:
diff changeset
950 case FormatStyle::UT_ForIndentation:
anatofuz
parents:
diff changeset
951 if (WhitespaceStartColumn == 0) {
anatofuz
parents:
diff changeset
952 unsigned Indentation = IndentLevel * Style.IndentWidth;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
953 Spaces = appendTabIndent(Text, Spaces, Indentation);
150
anatofuz
parents:
diff changeset
954 }
anatofuz
parents:
diff changeset
955 Text.append(Spaces, ' ');
anatofuz
parents:
diff changeset
956 break;
anatofuz
parents:
diff changeset
957 case FormatStyle::UT_ForContinuationAndIndentation:
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
958 if (WhitespaceStartColumn == 0)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
959 Spaces = appendTabIndent(Text, Spaces, Spaces);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
960 Text.append(Spaces, ' ');
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
961 break;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
962 case FormatStyle::UT_AlignWithSpaces:
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
963 if (WhitespaceStartColumn == 0) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
964 unsigned Indentation =
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
965 IsAligned ? IndentLevel * Style.IndentWidth : Spaces;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
966 Spaces = appendTabIndent(Text, Spaces, Indentation);
150
anatofuz
parents:
diff changeset
967 }
anatofuz
parents:
diff changeset
968 Text.append(Spaces, ' ');
anatofuz
parents:
diff changeset
969 break;
anatofuz
parents:
diff changeset
970 }
anatofuz
parents:
diff changeset
971 }
anatofuz
parents:
diff changeset
972
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
973 unsigned WhitespaceManager::appendTabIndent(std::string &Text, unsigned Spaces,
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
974 unsigned Indentation) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
975 // This happens, e.g. when a line in a block comment is indented less than the
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
976 // first one.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
977 if (Indentation > Spaces)
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
978 Indentation = Spaces;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
979 if (Style.TabWidth) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
980 unsigned Tabs = Indentation / Style.TabWidth;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
981 Text.append(Tabs, '\t');
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
982 Spaces -= Tabs * Style.TabWidth;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
983 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
984 return Spaces;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
985 }
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
986
150
anatofuz
parents:
diff changeset
987 } // namespace format
anatofuz
parents:
diff changeset
988 } // namespace clang