Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 1 | //===--- BreakableToken.cpp - Format C++ code -----------------------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | /// |
| 10 | /// \file |
Adrian Prantl | 9fc8faf | 2018-05-09 01:00:01 +0000 | [diff] [blame] | 11 | /// Contains implementation of BreakableToken class and classes derived |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 12 | /// from it. |
| 13 | /// |
| 14 | //===----------------------------------------------------------------------===// |
| 15 | |
| 16 | #include "BreakableToken.h" |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 17 | #include "ContinuationIndenter.h" |
Alexander Kornienko | 555efc3 | 2013-06-11 16:01:49 +0000 | [diff] [blame] | 18 | #include "clang/Basic/CharInfo.h" |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 19 | #include "clang/Format/Format.h" |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 20 | #include "llvm/ADT/STLExtras.h" |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 21 | #include "llvm/Support/Debug.h" |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 22 | #include <algorithm> |
| 23 | |
Chandler Carruth | 1034666 | 2014-04-22 03:17:02 +0000 | [diff] [blame] | 24 | #define DEBUG_TYPE "format-token-breaker" |
| 25 | |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 26 | namespace clang { |
| 27 | namespace format { |
| 28 | |
Daniel Jasper | 580da27 | 2013-10-30 07:36:40 +0000 | [diff] [blame] | 29 | static const char *const Blanks = " \t\v\f\r"; |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 30 | static bool IsBlank(char C) { |
| 31 | switch (C) { |
Daniel Jasper | 3ac9b9e | 2013-07-08 14:34:09 +0000 | [diff] [blame] | 32 | case ' ': |
| 33 | case '\t': |
| 34 | case '\v': |
| 35 | case '\f': |
Daniel Jasper | 580da27 | 2013-10-30 07:36:40 +0000 | [diff] [blame] | 36 | case '\r': |
Daniel Jasper | 3ac9b9e | 2013-07-08 14:34:09 +0000 | [diff] [blame] | 37 | return true; |
| 38 | default: |
| 39 | return false; |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 40 | } |
| 41 | } |
| 42 | |
Krasimir Georgiev | 410ed24 | 2017-11-10 12:50:09 +0000 | [diff] [blame] | 43 | static StringRef getLineCommentIndentPrefix(StringRef Comment, |
| 44 | const FormatStyle &Style) { |
| 45 | static const char *const KnownCStylePrefixes[] = {"///<", "//!<", "///", "//", |
| 46 | "//!"}; |
Krasimir Georgiev | 45dde41 | 2018-06-07 09:46:24 +0000 | [diff] [blame] | 47 | static const char *const KnownTextProtoPrefixes[] = {"//", "#", "##", "###", |
| 48 | "####"}; |
Krasimir Georgiev | 410ed24 | 2017-11-10 12:50:09 +0000 | [diff] [blame] | 49 | ArrayRef<const char *> KnownPrefixes(KnownCStylePrefixes); |
| 50 | if (Style.Language == FormatStyle::LK_TextProto) |
| 51 | KnownPrefixes = KnownTextProtoPrefixes; |
| 52 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 53 | StringRef LongestPrefix; |
| 54 | for (StringRef KnownPrefix : KnownPrefixes) { |
| 55 | if (Comment.startswith(KnownPrefix)) { |
| 56 | size_t PrefixLength = KnownPrefix.size(); |
| 57 | while (PrefixLength < Comment.size() && Comment[PrefixLength] == ' ') |
| 58 | ++PrefixLength; |
| 59 | if (PrefixLength > LongestPrefix.size()) |
| 60 | LongestPrefix = Comment.substr(0, PrefixLength); |
| 61 | } |
| 62 | } |
| 63 | return LongestPrefix; |
| 64 | } |
| 65 | |
Craig Topper | bfb5c40 | 2013-07-01 03:38:29 +0000 | [diff] [blame] | 66 | static BreakableToken::Split getCommentSplit(StringRef Text, |
| 67 | unsigned ContentStartColumn, |
| 68 | unsigned ColumnLimit, |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 69 | unsigned TabWidth, |
Martin Probst | 9d71781 | 2018-08-02 11:52:08 +0000 | [diff] [blame] | 70 | encoding::Encoding Encoding, |
| 71 | const FormatStyle &Style) { |
| 72 | LLVM_DEBUG(llvm::dbgs() << "Comment split: \"" << Text |
| 73 | << "\", Column limit: " << ColumnLimit |
| 74 | << ", Content start: " << ContentStartColumn << "\n"); |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 75 | if (ColumnLimit <= ContentStartColumn + 1) |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 76 | return BreakableToken::Split(StringRef::npos, 0); |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 77 | |
| 78 | unsigned MaxSplit = ColumnLimit - ContentStartColumn + 1; |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 79 | unsigned MaxSplitBytes = 0; |
| 80 | |
| 81 | for (unsigned NumChars = 0; |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 82 | NumChars < MaxSplit && MaxSplitBytes < Text.size();) { |
| 83 | unsigned BytesInChar = |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 84 | encoding::getCodePointNumBytes(Text[MaxSplitBytes], Encoding); |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 85 | NumChars += |
| 86 | encoding::columnWidthWithTabs(Text.substr(MaxSplitBytes, BytesInChar), |
| 87 | ContentStartColumn, TabWidth, Encoding); |
| 88 | MaxSplitBytes += BytesInChar; |
| 89 | } |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 90 | |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 91 | StringRef::size_type SpaceOffset = Text.find_last_of(Blanks, MaxSplitBytes); |
Francois Ferrand | a881be8 | 2017-05-22 14:47:17 +0000 | [diff] [blame] | 92 | |
Benjamin Kramer | f76861c | 2018-03-20 21:52:19 +0000 | [diff] [blame] | 93 | static auto *const kNumberedListRegexp = new llvm::Regex("^[1-9][0-9]?\\."); |
Martin Probst | f326b6b | 2018-08-03 13:58:33 +0000 | [diff] [blame^] | 94 | while (SpaceOffset != StringRef::npos) { |
| 95 | // Do not split before a number followed by a dot: this would be interpreted |
| 96 | // as a numbered list, which would prevent re-flowing in subsequent passes. |
| 97 | if (kNumberedListRegexp->match(Text.substr(SpaceOffset).ltrim(Blanks))) |
| 98 | SpaceOffset = Text.find_last_of(Blanks, SpaceOffset); |
| 99 | // In JavaScript, some @tags can be followed by {, and machinery that parses |
| 100 | // these comments will fail to understand the comment if followed by a line |
| 101 | // break. So avoid ever breaking before a {. |
| 102 | else if (Style.Language == FormatStyle::LK_JavaScript && |
| 103 | SpaceOffset + 1 < Text.size() && Text[SpaceOffset + 1] == '{') |
| 104 | SpaceOffset = Text.find_last_of(Blanks, SpaceOffset); |
| 105 | else |
| 106 | break; |
| 107 | } |
Francois Ferrand | a881be8 | 2017-05-22 14:47:17 +0000 | [diff] [blame] | 108 | |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 109 | if (SpaceOffset == StringRef::npos || |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 110 | // Don't break at leading whitespace. |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 111 | Text.find_last_not_of(Blanks, SpaceOffset) == StringRef::npos) { |
Manuel Klimek | ae1fbfb | 2013-05-29 22:06:18 +0000 | [diff] [blame] | 112 | // Make sure that we don't break at leading whitespace that |
| 113 | // reaches past MaxSplit. |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 114 | StringRef::size_type FirstNonWhitespace = Text.find_first_not_of(Blanks); |
Manuel Klimek | ae1fbfb | 2013-05-29 22:06:18 +0000 | [diff] [blame] | 115 | if (FirstNonWhitespace == StringRef::npos) |
| 116 | // If the comment is only whitespace, we cannot split. |
| 117 | return BreakableToken::Split(StringRef::npos, 0); |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 118 | SpaceOffset = Text.find_first_of( |
| 119 | Blanks, std::max<unsigned>(MaxSplitBytes, FirstNonWhitespace)); |
Manuel Klimek | ae1fbfb | 2013-05-29 22:06:18 +0000 | [diff] [blame] | 120 | } |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 121 | if (SpaceOffset != StringRef::npos && SpaceOffset != 0) { |
Martin Probst | 9d71781 | 2018-08-02 11:52:08 +0000 | [diff] [blame] | 122 | // adaptStartOfLine will break after lines starting with /** if the comment |
| 123 | // is broken anywhere. Avoid emitting this break twice here. |
| 124 | // Example: in /** longtextcomesherethatbreaks */ (with ColumnLimit 20) will |
| 125 | // insert a break after /**, so this code must not insert the same break. |
| 126 | if (SpaceOffset == 1 && Text[SpaceOffset - 1] == '*') |
| 127 | return BreakableToken::Split(StringRef::npos, 0); |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 128 | StringRef BeforeCut = Text.substr(0, SpaceOffset).rtrim(Blanks); |
| 129 | StringRef AfterCut = Text.substr(SpaceOffset).ltrim(Blanks); |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 130 | return BreakableToken::Split(BeforeCut.size(), |
| 131 | AfterCut.begin() - BeforeCut.end()); |
| 132 | } |
| 133 | return BreakableToken::Split(StringRef::npos, 0); |
| 134 | } |
| 135 | |
Daniel Jasper | b05a81d | 2014-05-09 13:11:16 +0000 | [diff] [blame] | 136 | static BreakableToken::Split |
| 137 | getStringSplit(StringRef Text, unsigned UsedColumns, unsigned ColumnLimit, |
| 138 | unsigned TabWidth, encoding::Encoding Encoding) { |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 139 | // FIXME: Reduce unit test case. |
| 140 | if (Text.empty()) |
| 141 | return BreakableToken::Split(StringRef::npos, 0); |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 142 | if (ColumnLimit <= UsedColumns) |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 143 | return BreakableToken::Split(StringRef::npos, 0); |
Alexander Kornienko | 71d95d6 | 2013-11-26 10:38:53 +0000 | [diff] [blame] | 144 | unsigned MaxSplit = ColumnLimit - UsedColumns; |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 145 | StringRef::size_type SpaceOffset = 0; |
| 146 | StringRef::size_type SlashOffset = 0; |
Alexander Kornienko | 7285207 | 2013-06-19 14:22:47 +0000 | [diff] [blame] | 147 | StringRef::size_type WordStartOffset = 0; |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 148 | StringRef::size_type SplitPoint = 0; |
| 149 | for (unsigned Chars = 0;;) { |
| 150 | unsigned Advance; |
| 151 | if (Text[0] == '\\') { |
| 152 | Advance = encoding::getEscapeSequenceLength(Text); |
| 153 | Chars += Advance; |
| 154 | } else { |
| 155 | Advance = encoding::getCodePointNumBytes(Text[0], Encoding); |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 156 | Chars += encoding::columnWidthWithTabs( |
| 157 | Text.substr(0, Advance), UsedColumns + Chars, TabWidth, Encoding); |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 158 | } |
| 159 | |
Daniel Jasper | e4b48c6 | 2015-01-21 19:50:35 +0000 | [diff] [blame] | 160 | if (Chars > MaxSplit || Text.size() <= Advance) |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 161 | break; |
| 162 | |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 163 | if (IsBlank(Text[0])) |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 164 | SpaceOffset = SplitPoint; |
| 165 | if (Text[0] == '/') |
| 166 | SlashOffset = SplitPoint; |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 167 | if (Advance == 1 && !isAlphanumeric(Text[0])) |
Alexander Kornienko | 7285207 | 2013-06-19 14:22:47 +0000 | [diff] [blame] | 168 | WordStartOffset = SplitPoint; |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 169 | |
| 170 | SplitPoint += Advance; |
| 171 | Text = Text.substr(Advance); |
| 172 | } |
| 173 | |
| 174 | if (SpaceOffset != 0) |
| 175 | return BreakableToken::Split(SpaceOffset + 1, 0); |
| 176 | if (SlashOffset != 0) |
| 177 | return BreakableToken::Split(SlashOffset + 1, 0); |
Alexander Kornienko | 7285207 | 2013-06-19 14:22:47 +0000 | [diff] [blame] | 178 | if (WordStartOffset != 0) |
| 179 | return BreakableToken::Split(WordStartOffset + 1, 0); |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 180 | if (SplitPoint != 0) |
| 181 | return BreakableToken::Split(SplitPoint, 0); |
| 182 | return BreakableToken::Split(StringRef::npos, 0); |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 183 | } |
| 184 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 185 | bool switchesFormatting(const FormatToken &Token) { |
| 186 | assert((Token.is(TT_BlockComment) || Token.is(TT_LineComment)) && |
| 187 | "formatting regions are switched by comment tokens"); |
| 188 | StringRef Content = Token.TokenText.substr(2).ltrim(); |
| 189 | return Content.startswith("clang-format on") || |
| 190 | Content.startswith("clang-format off"); |
| 191 | } |
| 192 | |
| 193 | unsigned |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 194 | BreakableToken::getLengthAfterCompression(unsigned RemainingTokenColumns, |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 195 | Split Split) const { |
| 196 | // Example: consider the content |
| 197 | // lala lala |
| 198 | // - RemainingTokenColumns is the original number of columns, 10; |
| 199 | // - Split is (4, 2), denoting the two spaces between the two words; |
| 200 | // |
| 201 | // We compute the number of columns when the split is compressed into a single |
| 202 | // space, like: |
| 203 | // lala lala |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 204 | // |
| 205 | // FIXME: Correctly measure the length of whitespace in Split.second so it |
| 206 | // works with tabs. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 207 | return RemainingTokenColumns + 1 - Split.second; |
| 208 | } |
| 209 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 210 | unsigned BreakableStringLiteral::getLineCount() const { return 1; } |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 211 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 212 | unsigned BreakableStringLiteral::getRangeLength(unsigned LineIndex, |
| 213 | unsigned Offset, |
| 214 | StringRef::size_type Length, |
| 215 | unsigned StartColumn) const { |
Manuel Klimek | 477f369 | 2017-11-29 15:09:12 +0000 | [diff] [blame] | 216 | llvm_unreachable("Getting the length of a part of the string literal " |
| 217 | "indicates that the code tries to reflow it."); |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 218 | } |
| 219 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 220 | unsigned |
| 221 | BreakableStringLiteral::getRemainingLength(unsigned LineIndex, unsigned Offset, |
| 222 | unsigned StartColumn) const { |
| 223 | return UnbreakableTailLength + Postfix.size() + |
| 224 | encoding::columnWidthWithTabs(Line.substr(Offset, StringRef::npos), |
| 225 | StartColumn, Style.TabWidth, Encoding); |
| 226 | } |
| 227 | |
| 228 | unsigned BreakableStringLiteral::getContentStartColumn(unsigned LineIndex, |
| 229 | bool Break) const { |
| 230 | return StartColumn + Prefix.size(); |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 231 | } |
| 232 | |
Alexander Kornienko | 81e3294 | 2013-09-16 20:20:49 +0000 | [diff] [blame] | 233 | BreakableStringLiteral::BreakableStringLiteral( |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 234 | const FormatToken &Tok, unsigned StartColumn, StringRef Prefix, |
Krasimir Georgiev | 55c23a1 | 2018-01-23 11:26:19 +0000 | [diff] [blame] | 235 | StringRef Postfix, unsigned UnbreakableTailLength, bool InPPDirective, |
| 236 | encoding::Encoding Encoding, const FormatStyle &Style) |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 237 | : BreakableToken(Tok, InPPDirective, Encoding, Style), |
| 238 | StartColumn(StartColumn), Prefix(Prefix), Postfix(Postfix), |
Krasimir Georgiev | 55c23a1 | 2018-01-23 11:26:19 +0000 | [diff] [blame] | 239 | UnbreakableTailLength(UnbreakableTailLength) { |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 240 | assert(Tok.TokenText.startswith(Prefix) && Tok.TokenText.endswith(Postfix)); |
| 241 | Line = Tok.TokenText.substr( |
| 242 | Prefix.size(), Tok.TokenText.size() - Prefix.size() - Postfix.size()); |
| 243 | } |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 244 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 245 | BreakableToken::Split BreakableStringLiteral::getSplit( |
| 246 | unsigned LineIndex, unsigned TailOffset, unsigned ColumnLimit, |
| 247 | unsigned ContentStartColumn, llvm::Regex &CommentPragmasRegex) const { |
| 248 | return getStringSplit(Line.substr(TailOffset), ContentStartColumn, |
| 249 | ColumnLimit - Postfix.size(), Style.TabWidth, Encoding); |
Alexander Kornienko | 9e90b62 | 2013-04-17 17:34:05 +0000 | [diff] [blame] | 250 | } |
| 251 | |
Alexander Kornienko | 555efc3 | 2013-06-11 16:01:49 +0000 | [diff] [blame] | 252 | void BreakableStringLiteral::insertBreak(unsigned LineIndex, |
| 253 | unsigned TailOffset, Split Split, |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 254 | unsigned ContentIndent, |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 255 | WhitespaceManager &Whitespaces) const { |
Alexander Kornienko | 555efc3 | 2013-06-11 16:01:49 +0000 | [diff] [blame] | 256 | Whitespaces.replaceWhitespaceInToken( |
| 257 | Tok, Prefix.size() + TailOffset + Split.first, Split.second, Postfix, |
Alexander Kornienko | d4fa2e6 | 2017-04-11 09:55:00 +0000 | [diff] [blame] | 258 | Prefix, InPPDirective, 1, StartColumn); |
Alexander Kornienko | 555efc3 | 2013-06-11 16:01:49 +0000 | [diff] [blame] | 259 | } |
| 260 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 261 | BreakableComment::BreakableComment(const FormatToken &Token, |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 262 | unsigned StartColumn, bool InPPDirective, |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 263 | encoding::Encoding Encoding, |
| 264 | const FormatStyle &Style) |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 265 | : BreakableToken(Token, InPPDirective, Encoding, Style), |
Krasimir Georgiev | 4b15922 | 2017-02-21 10:54:50 +0000 | [diff] [blame] | 266 | StartColumn(StartColumn) {} |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 267 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 268 | unsigned BreakableComment::getLineCount() const { return Lines.size(); } |
| 269 | |
Krasimir Georgiev | 17725d8 | 2017-03-08 08:55:12 +0000 | [diff] [blame] | 270 | BreakableToken::Split |
| 271 | BreakableComment::getSplit(unsigned LineIndex, unsigned TailOffset, |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 272 | unsigned ColumnLimit, unsigned ContentStartColumn, |
Krasimir Georgiev | 17725d8 | 2017-03-08 08:55:12 +0000 | [diff] [blame] | 273 | llvm::Regex &CommentPragmasRegex) const { |
| 274 | // Don't break lines matching the comment pragmas regex. |
| 275 | if (CommentPragmasRegex.match(Content[LineIndex])) |
| 276 | return Split(StringRef::npos, 0); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 277 | return getCommentSplit(Content[LineIndex].substr(TailOffset), |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 278 | ContentStartColumn, ColumnLimit, Style.TabWidth, |
Martin Probst | 9d71781 | 2018-08-02 11:52:08 +0000 | [diff] [blame] | 279 | Encoding, Style); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 280 | } |
| 281 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 282 | void BreakableComment::compressWhitespace( |
| 283 | unsigned LineIndex, unsigned TailOffset, Split Split, |
| 284 | WhitespaceManager &Whitespaces) const { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 285 | StringRef Text = Content[LineIndex].substr(TailOffset); |
| 286 | // Text is relative to the content line, but Whitespaces operates relative to |
| 287 | // the start of the corresponding token, so compute the start of the Split |
| 288 | // that needs to be compressed into a single space relative to the start of |
| 289 | // its token. |
| 290 | unsigned BreakOffsetInToken = |
| 291 | Text.data() - tokenAt(LineIndex).TokenText.data() + Split.first; |
| 292 | unsigned CharsToRemove = Split.second; |
Alexander Kornienko | 555efc3 | 2013-06-11 16:01:49 +0000 | [diff] [blame] | 293 | Whitespaces.replaceWhitespaceInToken( |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 294 | tokenAt(LineIndex), BreakOffsetInToken, CharsToRemove, "", "", |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 295 | /*InPPDirective=*/false, /*Newlines=*/0, /*Spaces=*/1); |
Alexander Kornienko | 875395f | 2013-11-12 17:50:13 +0000 | [diff] [blame] | 296 | } |
| 297 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 298 | const FormatToken &BreakableComment::tokenAt(unsigned LineIndex) const { |
| 299 | return Tokens[LineIndex] ? *Tokens[LineIndex] : Tok; |
| 300 | } |
| 301 | |
| 302 | static bool mayReflowContent(StringRef Content) { |
| 303 | Content = Content.trim(Blanks); |
Krasimir Georgiev | 28912c0 | 2017-02-02 10:52:08 +0000 | [diff] [blame] | 304 | // Lines starting with '@' commonly have special meaning. |
Francois Ferrand | a881be8 | 2017-05-22 14:47:17 +0000 | [diff] [blame] | 305 | // Lines starting with '-', '-#', '+' or '*' are bulleted/numbered lists. |
Krasimir Georgiev | 28912c0 | 2017-02-02 10:52:08 +0000 | [diff] [blame] | 306 | bool hasSpecialMeaningPrefix = false; |
Benjamin Kramer | f76861c | 2018-03-20 21:52:19 +0000 | [diff] [blame] | 307 | for (StringRef Prefix : |
| 308 | {"@", "TODO", "FIXME", "XXX", "-# ", "- ", "+ ", "* "}) { |
Krasimir Georgiev | 28912c0 | 2017-02-02 10:52:08 +0000 | [diff] [blame] | 309 | if (Content.startswith(Prefix)) { |
| 310 | hasSpecialMeaningPrefix = true; |
| 311 | break; |
| 312 | } |
| 313 | } |
Francois Ferrand | a881be8 | 2017-05-22 14:47:17 +0000 | [diff] [blame] | 314 | |
| 315 | // Numbered lists may also start with a number followed by '.' |
| 316 | // To avoid issues if a line starts with a number which is actually the end |
| 317 | // of a previous line, we only consider numbers with up to 2 digits. |
Benjamin Kramer | f76861c | 2018-03-20 21:52:19 +0000 | [diff] [blame] | 318 | static auto *const kNumberedListRegexp = new llvm::Regex("^[1-9][0-9]?\\. "); |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 319 | hasSpecialMeaningPrefix = |
Benjamin Kramer | f76861c | 2018-03-20 21:52:19 +0000 | [diff] [blame] | 320 | hasSpecialMeaningPrefix || kNumberedListRegexp->match(Content); |
Francois Ferrand | a881be8 | 2017-05-22 14:47:17 +0000 | [diff] [blame] | 321 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 322 | // Simple heuristic for what to reflow: content should contain at least two |
| 323 | // characters and either the first or second character must be |
| 324 | // non-punctuation. |
Krasimir Georgiev | 28912c0 | 2017-02-02 10:52:08 +0000 | [diff] [blame] | 325 | return Content.size() >= 2 && !hasSpecialMeaningPrefix && |
| 326 | !Content.endswith("\\") && |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 327 | // Note that this is UTF-8 safe, since if isPunctuation(Content[0]) is |
| 328 | // true, then the first code point must be 1 byte long. |
| 329 | (!isPunctuation(Content[0]) || !isPunctuation(Content[1])); |
| 330 | } |
| 331 | |
Alexander Kornienko | ffcc010 | 2013-06-05 14:09:10 +0000 | [diff] [blame] | 332 | BreakableBlockComment::BreakableBlockComment( |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 333 | const FormatToken &Token, unsigned StartColumn, |
Alexander Kornienko | be63390 | 2013-06-14 11:46:10 +0000 | [diff] [blame] | 334 | unsigned OriginalStartColumn, bool FirstInLine, bool InPPDirective, |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 335 | encoding::Encoding Encoding, const FormatStyle &Style) |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 336 | : BreakableComment(Token, StartColumn, InPPDirective, Encoding, Style), |
Manuel Klimek | 48c930c | 2017-12-04 08:53:16 +0000 | [diff] [blame] | 337 | DelimitersOnNewline(false), |
| 338 | UnbreakableTailLength(Token.UnbreakableTailLength) { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 339 | assert(Tok.is(TT_BlockComment) && |
| 340 | "block comment section must start with a block comment"); |
| 341 | |
| 342 | StringRef TokenText(Tok.TokenText); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 343 | assert(TokenText.startswith("/*") && TokenText.endswith("*/")); |
| 344 | TokenText.substr(2, TokenText.size() - 4).split(Lines, "\n"); |
| 345 | |
| 346 | int IndentDelta = StartColumn - OriginalStartColumn; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 347 | Content.resize(Lines.size()); |
| 348 | Content[0] = Lines[0]; |
| 349 | ContentColumn.resize(Lines.size()); |
| 350 | // Account for the initial '/*'. |
| 351 | ContentColumn[0] = StartColumn + 2; |
| 352 | Tokens.resize(Lines.size()); |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 353 | for (size_t i = 1; i < Lines.size(); ++i) |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 354 | adjustWhitespace(i, IndentDelta); |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 355 | |
Krasimir Georgiev | bb99a36 | 2017-02-16 12:39:31 +0000 | [diff] [blame] | 356 | // Align decorations with the column of the star on the first line, |
| 357 | // that is one column after the start "/*". |
| 358 | DecorationColumn = StartColumn + 1; |
| 359 | |
| 360 | // Account for comment decoration patterns like this: |
| 361 | // |
| 362 | // /* |
| 363 | // ** blah blah blah |
| 364 | // */ |
| 365 | if (Lines.size() >= 2 && Content[1].startswith("**") && |
| 366 | static_cast<unsigned>(ContentColumn[1]) == StartColumn) { |
| 367 | DecorationColumn = StartColumn; |
| 368 | } |
| 369 | |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 370 | Decoration = "* "; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 371 | if (Lines.size() == 1 && !FirstInLine) { |
| 372 | // Comments for which FirstInLine is false can start on arbitrary column, |
| 373 | // and available horizontal space can be too small to align consecutive |
| 374 | // lines with the first one. |
| 375 | // FIXME: We could, probably, align them to current indentation level, but |
| 376 | // now we just wrap them without stars. |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 377 | Decoration = ""; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 378 | } |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 379 | for (size_t i = 1, e = Lines.size(); i < e && !Decoration.empty(); ++i) { |
| 380 | // If the last line is empty, the closing "*/" will have a star. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 381 | if (i + 1 == e && Content[i].empty()) |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 382 | break; |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 383 | if (!Content[i].empty() && i + 1 != e && Decoration.startswith(Content[i])) |
Daniel Jasper | 6d9b88d | 2015-05-06 07:17:22 +0000 | [diff] [blame] | 384 | continue; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 385 | while (!Content[i].startswith(Decoration)) |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 386 | Decoration = Decoration.substr(0, Decoration.size() - 1); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 387 | } |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 388 | |
| 389 | LastLineNeedsDecoration = true; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 390 | IndentAtLineBreak = ContentColumn[0] + 1; |
| 391 | for (size_t i = 1, e = Lines.size(); i < e; ++i) { |
| 392 | if (Content[i].empty()) { |
| 393 | if (i + 1 == e) { |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 394 | // Empty last line means that we already have a star as a part of the |
| 395 | // trailing */. We also need to preserve whitespace, so that */ is |
| 396 | // correctly indented. |
| 397 | LastLineNeedsDecoration = false; |
Krasimir Georgiev | bb99a36 | 2017-02-16 12:39:31 +0000 | [diff] [blame] | 398 | // Align the star in the last '*/' with the stars on the previous lines. |
| 399 | if (e >= 2 && !Decoration.empty()) { |
| 400 | ContentColumn[i] = DecorationColumn; |
| 401 | } |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 402 | } else if (Decoration.empty()) { |
| 403 | // For all other lines, set the start column to 0 if they're empty, so |
| 404 | // we do not insert trailing whitespace anywhere. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 405 | ContentColumn[i] = 0; |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 406 | } |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 407 | continue; |
| 408 | } |
Daniel Jasper | 6d9b88d | 2015-05-06 07:17:22 +0000 | [diff] [blame] | 409 | |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 410 | // The first line already excludes the star. |
Krasimir Georgiev | bb99a36 | 2017-02-16 12:39:31 +0000 | [diff] [blame] | 411 | // The last line excludes the star if LastLineNeedsDecoration is false. |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 412 | // For all other lines, adjust the line to exclude the star and |
| 413 | // (optionally) the first whitespace. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 414 | unsigned DecorationSize = Decoration.startswith(Content[i]) |
| 415 | ? Content[i].size() |
| 416 | : Decoration.size(); |
Krasimir Georgiev | bb99a36 | 2017-02-16 12:39:31 +0000 | [diff] [blame] | 417 | if (DecorationSize) { |
| 418 | ContentColumn[i] = DecorationColumn + DecorationSize; |
| 419 | } |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 420 | Content[i] = Content[i].substr(DecorationSize); |
| 421 | if (!Decoration.startswith(Content[i])) |
Daniel Jasper | 6d9b88d | 2015-05-06 07:17:22 +0000 | [diff] [blame] | 422 | IndentAtLineBreak = |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 423 | std::min<int>(IndentAtLineBreak, std::max(0, ContentColumn[i])); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 424 | } |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 425 | IndentAtLineBreak = std::max<unsigned>(IndentAtLineBreak, Decoration.size()); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 426 | |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 427 | // Detect a multiline jsdoc comment and set DelimitersOnNewline in that case. |
| 428 | if (Style.Language == FormatStyle::LK_JavaScript || |
| 429 | Style.Language == FormatStyle::LK_Java) { |
| 430 | if ((Lines[0] == "*" || Lines[0].startswith("* ")) && Lines.size() > 1) { |
| 431 | // This is a multiline jsdoc comment. |
| 432 | DelimitersOnNewline = true; |
| 433 | } else if (Lines[0].startswith("* ") && Lines.size() == 1) { |
| 434 | // Detect a long single-line comment, like: |
| 435 | // /** long long long */ |
| 436 | // Below, '2' is the width of '*/'. |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 437 | unsigned EndColumn = |
| 438 | ContentColumn[0] + |
| 439 | encoding::columnWidthWithTabs(Lines[0], ContentColumn[0], |
| 440 | Style.TabWidth, Encoding) + |
| 441 | 2; |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 442 | DelimitersOnNewline = EndColumn > Style.ColumnLimit; |
| 443 | } |
| 444 | } |
| 445 | |
Nicola Zaghen | 3538b39 | 2018-05-15 13:30:56 +0000 | [diff] [blame] | 446 | LLVM_DEBUG({ |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 447 | llvm::dbgs() << "IndentAtLineBreak " << IndentAtLineBreak << "\n"; |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 448 | llvm::dbgs() << "DelimitersOnNewline " << DelimitersOnNewline << "\n"; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 449 | for (size_t i = 0; i < Lines.size(); ++i) { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 450 | llvm::dbgs() << i << " |" << Content[i] << "| " |
Krasimir Georgiev | bb99a36 | 2017-02-16 12:39:31 +0000 | [diff] [blame] | 451 | << "CC=" << ContentColumn[i] << "| " |
| 452 | << "IN=" << (Content[i].data() - Lines[i].data()) << "\n"; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 453 | } |
| 454 | }); |
| 455 | } |
| 456 | |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 457 | void BreakableBlockComment::adjustWhitespace(unsigned LineIndex, |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 458 | int IndentDelta) { |
Alexander Kornienko | be63390 | 2013-06-14 11:46:10 +0000 | [diff] [blame] | 459 | // When in a preprocessor directive, the trailing backslash in a block comment |
| 460 | // is not needed, but can serve a purpose of uniformity with necessary escaped |
| 461 | // newlines outside the comment. In this case we remove it here before |
| 462 | // trimming the trailing whitespace. The backslash will be re-added later when |
| 463 | // inserting a line break. |
| 464 | size_t EndOfPreviousLine = Lines[LineIndex - 1].size(); |
| 465 | if (InPPDirective && Lines[LineIndex - 1].endswith("\\")) |
| 466 | --EndOfPreviousLine; |
| 467 | |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 468 | // Calculate the end of the non-whitespace text in the previous line. |
Alexander Kornienko | be63390 | 2013-06-14 11:46:10 +0000 | [diff] [blame] | 469 | EndOfPreviousLine = |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 470 | Lines[LineIndex - 1].find_last_not_of(Blanks, EndOfPreviousLine); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 471 | if (EndOfPreviousLine == StringRef::npos) |
| 472 | EndOfPreviousLine = 0; |
| 473 | else |
| 474 | ++EndOfPreviousLine; |
| 475 | // Calculate the start of the non-whitespace text in the current line. |
Alexander Kornienko | b93062e | 2013-06-20 13:58:37 +0000 | [diff] [blame] | 476 | size_t StartOfLine = Lines[LineIndex].find_first_not_of(Blanks); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 477 | if (StartOfLine == StringRef::npos) |
Daniel Jasper | d6e6188 | 2015-06-17 12:23:15 +0000 | [diff] [blame] | 478 | StartOfLine = Lines[LineIndex].rtrim("\r\n").size(); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 479 | |
Alexander Kornienko | ebb43ca | 2013-09-05 14:08:34 +0000 | [diff] [blame] | 480 | StringRef Whitespace = Lines[LineIndex].substr(0, StartOfLine); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 481 | // Adjust Lines to only contain relevant text. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 482 | size_t PreviousContentOffset = |
| 483 | Content[LineIndex - 1].data() - Lines[LineIndex - 1].data(); |
| 484 | Content[LineIndex - 1] = Lines[LineIndex - 1].substr( |
| 485 | PreviousContentOffset, EndOfPreviousLine - PreviousContentOffset); |
| 486 | Content[LineIndex] = Lines[LineIndex].substr(StartOfLine); |
Manuel Klimek | 34d1515 | 2013-05-28 10:01:59 +0000 | [diff] [blame] | 487 | |
Alp Toker | f6a24ce | 2013-12-05 16:25:25 +0000 | [diff] [blame] | 488 | // Adjust the start column uniformly across all lines. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 489 | ContentColumn[LineIndex] = |
Alexander Kornienko | 39856b7 | 2013-09-10 09:38:25 +0000 | [diff] [blame] | 490 | encoding::columnWidthWithTabs(Whitespace, 0, Style.TabWidth, Encoding) + |
Alexander Kornienko | 67d9c8c | 2014-04-17 16:12:46 +0000 | [diff] [blame] | 491 | IndentDelta; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 492 | } |
| 493 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 494 | unsigned BreakableBlockComment::getRangeLength(unsigned LineIndex, |
| 495 | unsigned Offset, |
| 496 | StringRef::size_type Length, |
| 497 | unsigned StartColumn) const { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 498 | unsigned LineLength = |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 499 | encoding::columnWidthWithTabs(Content[LineIndex].substr(Offset, Length), |
| 500 | StartColumn, Style.TabWidth, Encoding); |
| 501 | // FIXME: This should go into getRemainingLength instead, but we currently |
| 502 | // break tests when putting it there. Investigate how to fix those tests. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 503 | // The last line gets a "*/" postfix. |
| 504 | if (LineIndex + 1 == Lines.size()) { |
| 505 | LineLength += 2; |
| 506 | // We never need a decoration when breaking just the trailing "*/" postfix. |
| 507 | // Note that checking that Length == 0 is not enough, since Length could |
| 508 | // also be StringRef::npos. |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 509 | if (Content[LineIndex].substr(Offset, StringRef::npos).empty()) { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 510 | LineLength -= Decoration.size(); |
| 511 | } |
| 512 | } |
| 513 | return LineLength; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 514 | } |
| 515 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 516 | unsigned BreakableBlockComment::getRemainingLength(unsigned LineIndex, |
| 517 | unsigned Offset, |
| 518 | unsigned StartColumn) const { |
Manuel Klimek | 48c930c | 2017-12-04 08:53:16 +0000 | [diff] [blame] | 519 | return UnbreakableTailLength + |
| 520 | getRangeLength(LineIndex, Offset, StringRef::npos, StartColumn); |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 521 | } |
| 522 | |
| 523 | unsigned BreakableBlockComment::getContentStartColumn(unsigned LineIndex, |
| 524 | bool Break) const { |
| 525 | if (Break) |
| 526 | return IndentAtLineBreak; |
| 527 | return std::max(0, ContentColumn[LineIndex]); |
| 528 | } |
| 529 | |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 530 | const llvm::StringSet<> |
| 531 | BreakableBlockComment::ContentIndentingJavadocAnnotations = { |
| 532 | "@param", "@return", "@returns", "@throws", "@type", "@template", |
Krasimir Georgiev | 186478d | 2018-08-01 11:48:04 +0000 | [diff] [blame] | 533 | "@see", "@deprecated", "@define", "@exports", "@mods", "@private", |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 534 | }; |
| 535 | |
| 536 | unsigned BreakableBlockComment::getContentIndent(unsigned LineIndex) const { |
| 537 | if (Style.Language != FormatStyle::LK_Java && |
| 538 | Style.Language != FormatStyle::LK_JavaScript) |
| 539 | return 0; |
| 540 | // The content at LineIndex 0 of a comment like: |
| 541 | // /** line 0 */ |
| 542 | // is "* line 0", so we need to skip over the decoration in that case. |
| 543 | StringRef ContentWithNoDecoration = Content[LineIndex]; |
| 544 | if (LineIndex == 0 && ContentWithNoDecoration.startswith("*")) { |
| 545 | ContentWithNoDecoration = ContentWithNoDecoration.substr(1).ltrim(Blanks); |
| 546 | } |
| 547 | StringRef FirstWord = ContentWithNoDecoration.substr( |
| 548 | 0, ContentWithNoDecoration.find_first_of(Blanks)); |
| 549 | if (ContentIndentingJavadocAnnotations.find(FirstWord) != |
| 550 | ContentIndentingJavadocAnnotations.end()) |
| 551 | return Style.ContinuationIndentWidth; |
| 552 | return 0; |
| 553 | } |
| 554 | |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 555 | void BreakableBlockComment::insertBreak(unsigned LineIndex, unsigned TailOffset, |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 556 | Split Split, unsigned ContentIndent, |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 557 | WhitespaceManager &Whitespaces) const { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 558 | StringRef Text = Content[LineIndex].substr(TailOffset); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 559 | StringRef Prefix = Decoration; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 560 | // We need this to account for the case when we have a decoration "* " for all |
| 561 | // the lines except for the last one, where the star in "*/" acts as a |
| 562 | // decoration. |
| 563 | unsigned LocalIndentAtLineBreak = IndentAtLineBreak; |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 564 | if (LineIndex + 1 == Lines.size() && |
| 565 | Text.size() == Split.first + Split.second) { |
| 566 | // For the last line we need to break before "*/", but not to add "* ". |
| 567 | Prefix = ""; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 568 | if (LocalIndentAtLineBreak >= 2) |
| 569 | LocalIndentAtLineBreak -= 2; |
| 570 | } |
| 571 | // The split offset is from the beginning of the line. Convert it to an offset |
| 572 | // from the beginning of the token text. |
| 573 | unsigned BreakOffsetInToken = |
| 574 | Text.data() - tokenAt(LineIndex).TokenText.data() + Split.first; |
| 575 | unsigned CharsToRemove = Split.second; |
| 576 | assert(LocalIndentAtLineBreak >= Prefix.size()); |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 577 | std::string PrefixWithTrailingIndent = Prefix; |
| 578 | for (unsigned I = 0; I < ContentIndent; ++I) |
| 579 | PrefixWithTrailingIndent += " "; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 580 | Whitespaces.replaceWhitespaceInToken( |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 581 | tokenAt(LineIndex), BreakOffsetInToken, CharsToRemove, "", |
| 582 | PrefixWithTrailingIndent, InPPDirective, /*Newlines=*/1, |
| 583 | /*Spaces=*/LocalIndentAtLineBreak + ContentIndent - |
| 584 | PrefixWithTrailingIndent.size()); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 585 | } |
| 586 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 587 | BreakableToken::Split |
| 588 | BreakableBlockComment::getReflowSplit(unsigned LineIndex, |
| 589 | llvm::Regex &CommentPragmasRegex) const { |
Krasimir Georgiev | 00c5c72 | 2017-02-02 15:32:19 +0000 | [diff] [blame] | 590 | if (!mayReflow(LineIndex, CommentPragmasRegex)) |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 591 | return Split(StringRef::npos, 0); |
Fangrui Song | 6907ce2 | 2018-07-30 19:24:48 +0000 | [diff] [blame] | 592 | |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 593 | // If we're reflowing into a line with content indent, only reflow the next |
| 594 | // line if its starting whitespace matches the content indent. |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 595 | size_t Trimmed = Content[LineIndex].find_first_not_of(Blanks); |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 596 | if (LineIndex) { |
| 597 | unsigned PreviousContentIndent = getContentIndent(LineIndex - 1); |
| 598 | if (PreviousContentIndent && Trimmed != StringRef::npos && |
| 599 | Trimmed != PreviousContentIndent) |
| 600 | return Split(StringRef::npos, 0); |
| 601 | } |
| 602 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 603 | return Split(0, Trimmed != StringRef::npos ? Trimmed : 0); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 604 | } |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 605 | |
Manuel Klimek | 7786614 | 2017-11-17 11:17:15 +0000 | [diff] [blame] | 606 | bool BreakableBlockComment::introducesBreakBeforeToken() const { |
Krasimir Georgiev | 35599fd | 2017-10-16 09:08:53 +0000 | [diff] [blame] | 607 | // A break is introduced when we want delimiters on newline. |
Manuel Klimek | 7786614 | 2017-11-17 11:17:15 +0000 | [diff] [blame] | 608 | return DelimitersOnNewline && |
Krasimir Georgiev | 35599fd | 2017-10-16 09:08:53 +0000 | [diff] [blame] | 609 | Lines[0].substr(1).find_first_not_of(Blanks) != StringRef::npos; |
| 610 | } |
| 611 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 612 | void BreakableBlockComment::reflow(unsigned LineIndex, |
| 613 | WhitespaceManager &Whitespaces) const { |
| 614 | StringRef TrimmedContent = Content[LineIndex].ltrim(Blanks); |
| 615 | // Here we need to reflow. |
| 616 | assert(Tokens[LineIndex - 1] == Tokens[LineIndex] && |
| 617 | "Reflowing whitespace within a token"); |
| 618 | // This is the offset of the end of the last line relative to the start of |
| 619 | // the token text in the token. |
| 620 | unsigned WhitespaceOffsetInToken = Content[LineIndex - 1].data() + |
| 621 | Content[LineIndex - 1].size() - |
| 622 | tokenAt(LineIndex).TokenText.data(); |
| 623 | unsigned WhitespaceLength = TrimmedContent.data() - |
| 624 | tokenAt(LineIndex).TokenText.data() - |
| 625 | WhitespaceOffsetInToken; |
| 626 | Whitespaces.replaceWhitespaceInToken( |
| 627 | tokenAt(LineIndex), WhitespaceOffsetInToken, |
| 628 | /*ReplaceChars=*/WhitespaceLength, /*PreviousPostfix=*/"", |
| 629 | /*CurrentPrefix=*/ReflowPrefix, InPPDirective, /*Newlines=*/0, |
| 630 | /*Spaces=*/0); |
| 631 | } |
| 632 | |
| 633 | void BreakableBlockComment::adaptStartOfLine( |
| 634 | unsigned LineIndex, WhitespaceManager &Whitespaces) const { |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 635 | if (LineIndex == 0) { |
| 636 | if (DelimitersOnNewline) { |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 637 | // Since we're breaking at index 1 below, the break position and the |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 638 | // break length are the same. |
Martin Probst | 9d71781 | 2018-08-02 11:52:08 +0000 | [diff] [blame] | 639 | // Note: this works because getCommentSplit is careful never to split at |
| 640 | // the beginning of a line. |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 641 | size_t BreakLength = Lines[0].substr(1).find_first_not_of(Blanks); |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 642 | if (BreakLength != StringRef::npos) |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 643 | insertBreak(LineIndex, 0, Split(1, BreakLength), /*ContentIndent=*/0, |
| 644 | Whitespaces); |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 645 | } |
| 646 | return; |
| 647 | } |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 648 | // Here no reflow with the previous line will happen. |
| 649 | // Fix the decoration of the line at LineIndex. |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 650 | StringRef Prefix = Decoration; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 651 | if (Content[LineIndex].empty()) { |
Manuel Klimek | 281dcbe | 2013-05-28 08:55:01 +0000 | [diff] [blame] | 652 | if (LineIndex + 1 == Lines.size()) { |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 653 | if (!LastLineNeedsDecoration) { |
| 654 | // If the last line was empty, we don't need a prefix, as the */ will |
| 655 | // line up with the decoration (if it exists). |
| 656 | Prefix = ""; |
| 657 | } |
Manuel Klimek | 281dcbe | 2013-05-28 08:55:01 +0000 | [diff] [blame] | 658 | } else if (!Decoration.empty()) { |
| 659 | // For other empty lines, if we do have a decoration, adapt it to not |
| 660 | // contain a trailing whitespace. |
| 661 | Prefix = Prefix.substr(0, 1); |
| 662 | } |
Daniel Jasper | 51fb2b2 | 2013-05-30 06:40:07 +0000 | [diff] [blame] | 663 | } else { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 664 | if (ContentColumn[LineIndex] == 1) { |
Alexander Kornienko | 614d96a | 2013-07-08 14:12:07 +0000 | [diff] [blame] | 665 | // This line starts immediately after the decorating *. |
Daniel Jasper | 51fb2b2 | 2013-05-30 06:40:07 +0000 | [diff] [blame] | 666 | Prefix = Prefix.substr(0, 1); |
| 667 | } |
Manuel Klimek | 281dcbe | 2013-05-28 08:55:01 +0000 | [diff] [blame] | 668 | } |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 669 | // This is the offset of the end of the last line relative to the start of the |
| 670 | // token text in the token. |
| 671 | unsigned WhitespaceOffsetInToken = Content[LineIndex - 1].data() + |
| 672 | Content[LineIndex - 1].size() - |
| 673 | tokenAt(LineIndex).TokenText.data(); |
| 674 | unsigned WhitespaceLength = Content[LineIndex].data() - |
| 675 | tokenAt(LineIndex).TokenText.data() - |
| 676 | WhitespaceOffsetInToken; |
Alexander Kornienko | 555efc3 | 2013-06-11 16:01:49 +0000 | [diff] [blame] | 677 | Whitespaces.replaceWhitespaceInToken( |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 678 | tokenAt(LineIndex), WhitespaceOffsetInToken, WhitespaceLength, "", Prefix, |
| 679 | InPPDirective, /*Newlines=*/1, ContentColumn[LineIndex] - Prefix.size()); |
Manuel Klimek | 9043c74 | 2013-05-27 15:23:34 +0000 | [diff] [blame] | 680 | } |
| 681 | |
Krasimir Georgiev | 3b86534 | 2017-08-09 09:42:32 +0000 | [diff] [blame] | 682 | BreakableToken::Split |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 683 | BreakableBlockComment::getSplitAfterLastLine(unsigned TailOffset) const { |
Krasimir Georgiev | 3b86534 | 2017-08-09 09:42:32 +0000 | [diff] [blame] | 684 | if (DelimitersOnNewline) { |
| 685 | // Replace the trailing whitespace of the last line with a newline. |
| 686 | // In case the last line is empty, the ending '*/' is already on its own |
| 687 | // line. |
| 688 | StringRef Line = Content.back().substr(TailOffset); |
| 689 | StringRef TrimmedLine = Line.rtrim(Blanks); |
| 690 | if (!TrimmedLine.empty()) |
| 691 | return Split(TrimmedLine.size(), Line.size() - TrimmedLine.size()); |
| 692 | } |
Krasimir Georgiev | 22d7e6b | 2017-07-20 22:29:39 +0000 | [diff] [blame] | 693 | return Split(StringRef::npos, 0); |
| 694 | } |
| 695 | |
Krasimir Georgiev | 00c5c72 | 2017-02-02 15:32:19 +0000 | [diff] [blame] | 696 | bool BreakableBlockComment::mayReflow(unsigned LineIndex, |
| 697 | llvm::Regex &CommentPragmasRegex) const { |
| 698 | // Content[LineIndex] may exclude the indent after the '*' decoration. In that |
| 699 | // case, we compute the start of the comment pragma manually. |
| 700 | StringRef IndentContent = Content[LineIndex]; |
| 701 | if (Lines[LineIndex].ltrim(Blanks).startswith("*")) { |
| 702 | IndentContent = Lines[LineIndex].ltrim(Blanks).substr(1); |
| 703 | } |
| 704 | return LineIndex > 0 && !CommentPragmasRegex.match(IndentContent) && |
| 705 | mayReflowContent(Content[LineIndex]) && !Tok.Finalized && |
| 706 | !switchesFormatting(tokenAt(LineIndex)); |
| 707 | } |
| 708 | |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 709 | BreakableLineCommentSection::BreakableLineCommentSection( |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 710 | const FormatToken &Token, unsigned StartColumn, |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 711 | unsigned OriginalStartColumn, bool FirstInLine, bool InPPDirective, |
| 712 | encoding::Encoding Encoding, const FormatStyle &Style) |
Krasimir Georgiev | 4b15922 | 2017-02-21 10:54:50 +0000 | [diff] [blame] | 713 | : BreakableComment(Token, StartColumn, InPPDirective, Encoding, Style) { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 714 | assert(Tok.is(TT_LineComment) && |
| 715 | "line comment section must start with a line comment"); |
| 716 | FormatToken *LineTok = nullptr; |
| 717 | for (const FormatToken *CurrentTok = &Tok; |
| 718 | CurrentTok && CurrentTok->is(TT_LineComment); |
| 719 | CurrentTok = CurrentTok->Next) { |
| 720 | LastLineTok = LineTok; |
| 721 | StringRef TokenText(CurrentTok->TokenText); |
Krasimir Georgiev | 410ed24 | 2017-11-10 12:50:09 +0000 | [diff] [blame] | 722 | assert((TokenText.startswith("//") || TokenText.startswith("#")) && |
| 723 | "unsupported line comment prefix, '//' and '#' are supported"); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 724 | size_t FirstLineIndex = Lines.size(); |
| 725 | TokenText.split(Lines, "\n"); |
| 726 | Content.resize(Lines.size()); |
| 727 | ContentColumn.resize(Lines.size()); |
| 728 | OriginalContentColumn.resize(Lines.size()); |
| 729 | Tokens.resize(Lines.size()); |
| 730 | Prefix.resize(Lines.size()); |
| 731 | OriginalPrefix.resize(Lines.size()); |
| 732 | for (size_t i = FirstLineIndex, e = Lines.size(); i < e; ++i) { |
Manuel Klimek | 45ab559 | 2017-11-14 09:19:53 +0000 | [diff] [blame] | 733 | Lines[i] = Lines[i].ltrim(Blanks); |
Krasimir Georgiev | e518e0b | 2017-01-30 21:00:01 +0000 | [diff] [blame] | 734 | // We need to trim the blanks in case this is not the first line in a |
| 735 | // multiline comment. Then the indent is included in Lines[i]. |
| 736 | StringRef IndentPrefix = |
Krasimir Georgiev | 410ed24 | 2017-11-10 12:50:09 +0000 | [diff] [blame] | 737 | getLineCommentIndentPrefix(Lines[i].ltrim(Blanks), Style); |
| 738 | assert((TokenText.startswith("//") || TokenText.startswith("#")) && |
| 739 | "unsupported line comment prefix, '//' and '#' are supported"); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 740 | OriginalPrefix[i] = Prefix[i] = IndentPrefix; |
| 741 | if (Lines[i].size() > Prefix[i].size() && |
| 742 | isAlphanumeric(Lines[i][Prefix[i].size()])) { |
| 743 | if (Prefix[i] == "//") |
| 744 | Prefix[i] = "// "; |
| 745 | else if (Prefix[i] == "///") |
| 746 | Prefix[i] = "/// "; |
| 747 | else if (Prefix[i] == "//!") |
| 748 | Prefix[i] = "//! "; |
Krasimir Georgiev | ba6b315 | 2017-05-18 07:36:21 +0000 | [diff] [blame] | 749 | else if (Prefix[i] == "///<") |
| 750 | Prefix[i] = "///< "; |
| 751 | else if (Prefix[i] == "//!<") |
| 752 | Prefix[i] = "//!< "; |
Krasimir Georgiev | 410ed24 | 2017-11-10 12:50:09 +0000 | [diff] [blame] | 753 | else if (Prefix[i] == "#" && |
| 754 | Style.Language == FormatStyle::LK_TextProto) |
| 755 | Prefix[i] = "# "; |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 756 | } |
| 757 | |
| 758 | Tokens[i] = LineTok; |
| 759 | Content[i] = Lines[i].substr(IndentPrefix.size()); |
| 760 | OriginalContentColumn[i] = |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 761 | StartColumn + encoding::columnWidthWithTabs(OriginalPrefix[i], |
| 762 | StartColumn, |
| 763 | Style.TabWidth, Encoding); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 764 | ContentColumn[i] = |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 765 | StartColumn + encoding::columnWidthWithTabs(Prefix[i], StartColumn, |
| 766 | Style.TabWidth, Encoding); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 767 | |
| 768 | // Calculate the end of the non-whitespace text in this line. |
| 769 | size_t EndOfLine = Content[i].find_last_not_of(Blanks); |
| 770 | if (EndOfLine == StringRef::npos) |
| 771 | EndOfLine = Content[i].size(); |
| 772 | else |
| 773 | ++EndOfLine; |
| 774 | Content[i] = Content[i].substr(0, EndOfLine); |
| 775 | } |
| 776 | LineTok = CurrentTok->Next; |
Krasimir Georgiev | b6ccd38 | 2017-02-02 14:36:50 +0000 | [diff] [blame] | 777 | if (CurrentTok->Next && !CurrentTok->Next->ContinuesLineCommentSection) { |
Krasimir Georgiev | 753625b | 2017-01-31 13:32:38 +0000 | [diff] [blame] | 778 | // A line comment section needs to broken by a line comment that is |
| 779 | // preceded by at least two newlines. Note that we put this break here |
| 780 | // instead of breaking at a previous stage during parsing, since that |
| 781 | // would split the contents of the enum into two unwrapped lines in this |
| 782 | // example, which is undesirable: |
| 783 | // enum A { |
| 784 | // a, // comment about a |
| 785 | // |
| 786 | // // comment about b |
| 787 | // b |
| 788 | // }; |
| 789 | // |
| 790 | // FIXME: Consider putting separate line comment sections as children to |
| 791 | // the unwrapped line instead. |
| 792 | break; |
| 793 | } |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 794 | } |
| 795 | } |
| 796 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 797 | unsigned |
| 798 | BreakableLineCommentSection::getRangeLength(unsigned LineIndex, unsigned Offset, |
| 799 | StringRef::size_type Length, |
| 800 | unsigned StartColumn) const { |
| 801 | return encoding::columnWidthWithTabs( |
| 802 | Content[LineIndex].substr(Offset, Length), StartColumn, Style.TabWidth, |
| 803 | Encoding); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 804 | } |
| 805 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 806 | unsigned BreakableLineCommentSection::getContentStartColumn(unsigned LineIndex, |
| 807 | bool Break) const { |
| 808 | if (Break) |
| 809 | return OriginalContentColumn[LineIndex]; |
| 810 | return ContentColumn[LineIndex]; |
| 811 | } |
| 812 | |
| 813 | void BreakableLineCommentSection::insertBreak( |
| 814 | unsigned LineIndex, unsigned TailOffset, Split Split, |
Krasimir Georgiev | 6a5c95b | 2018-07-30 08:45:45 +0000 | [diff] [blame] | 815 | unsigned ContentIndent, WhitespaceManager &Whitespaces) const { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 816 | StringRef Text = Content[LineIndex].substr(TailOffset); |
| 817 | // Compute the offset of the split relative to the beginning of the token |
| 818 | // text. |
| 819 | unsigned BreakOffsetInToken = |
| 820 | Text.data() - tokenAt(LineIndex).TokenText.data() + Split.first; |
| 821 | unsigned CharsToRemove = Split.second; |
| 822 | // Compute the size of the new indent, including the size of the new prefix of |
| 823 | // the newly broken line. |
| 824 | unsigned IndentAtLineBreak = OriginalContentColumn[LineIndex] + |
| 825 | Prefix[LineIndex].size() - |
| 826 | OriginalPrefix[LineIndex].size(); |
| 827 | assert(IndentAtLineBreak >= Prefix[LineIndex].size()); |
| 828 | Whitespaces.replaceWhitespaceInToken( |
| 829 | tokenAt(LineIndex), BreakOffsetInToken, CharsToRemove, "", |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 830 | Prefix[LineIndex], InPPDirective, /*Newlines=*/1, |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 831 | /*Spaces=*/IndentAtLineBreak - Prefix[LineIndex].size()); |
| 832 | } |
| 833 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 834 | BreakableComment::Split BreakableLineCommentSection::getReflowSplit( |
| 835 | unsigned LineIndex, llvm::Regex &CommentPragmasRegex) const { |
Krasimir Georgiev | 00c5c72 | 2017-02-02 15:32:19 +0000 | [diff] [blame] | 836 | if (!mayReflow(LineIndex, CommentPragmasRegex)) |
| 837 | return Split(StringRef::npos, 0); |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 838 | |
| 839 | size_t Trimmed = Content[LineIndex].find_first_not_of(Blanks); |
| 840 | |
| 841 | // In a line comment section each line is a separate token; thus, after a |
| 842 | // split we replace all whitespace before the current line comment token |
| 843 | // (which does not need to be included in the split), plus the start of the |
| 844 | // line up to where the content starts. |
| 845 | return Split(0, Trimmed != StringRef::npos ? Trimmed : 0); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 846 | } |
| 847 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 848 | void BreakableLineCommentSection::reflow(unsigned LineIndex, |
| 849 | WhitespaceManager &Whitespaces) const { |
Krasimir Georgiev | 4fc55a7 | 2018-06-12 19:33:15 +0000 | [diff] [blame] | 850 | if (LineIndex > 0 && Tokens[LineIndex] != Tokens[LineIndex - 1]) { |
| 851 | // Reflow happens between tokens. Replace the whitespace between the |
| 852 | // tokens by the empty string. |
| 853 | Whitespaces.replaceWhitespace( |
| 854 | *Tokens[LineIndex], /*Newlines=*/0, /*Spaces=*/0, |
| 855 | /*StartOfTokenColumn=*/StartColumn, /*InPPDirective=*/false); |
| 856 | } else if (LineIndex > 0) { |
| 857 | // In case we're reflowing after the '\' in: |
| 858 | // |
| 859 | // // line comment \ |
| 860 | // // line 2 |
| 861 | // |
| 862 | // the reflow happens inside the single comment token (it is a single line |
| 863 | // comment with an unescaped newline). |
| 864 | // Replace the whitespace between the '\' and '//' with the empty string. |
| 865 | // |
| 866 | // Offset points to after the '\' relative to start of the token. |
| 867 | unsigned Offset = Lines[LineIndex - 1].data() + |
| 868 | Lines[LineIndex - 1].size() - |
| 869 | tokenAt(LineIndex - 1).TokenText.data(); |
| 870 | // WhitespaceLength is the number of chars between the '\' and the '//' on |
| 871 | // the next line. |
| 872 | unsigned WhitespaceLength = |
| 873 | Lines[LineIndex].data() - tokenAt(LineIndex).TokenText.data() - Offset; |
| 874 | Whitespaces.replaceWhitespaceInToken(*Tokens[LineIndex], |
| 875 | Offset, |
| 876 | /*ReplaceChars=*/WhitespaceLength, |
| 877 | /*PreviousPostfix=*/"", |
| 878 | /*CurrentPrefix=*/"", |
| 879 | /*InPPDirective=*/false, |
| 880 | /*Newlines=*/0, |
| 881 | /*Spaces=*/0); |
| 882 | |
| 883 | } |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 884 | // Replace the indent and prefix of the token with the reflow prefix. |
Krasimir Georgiev | 4fc55a7 | 2018-06-12 19:33:15 +0000 | [diff] [blame] | 885 | unsigned Offset = |
| 886 | Lines[LineIndex].data() - tokenAt(LineIndex).TokenText.data(); |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 887 | unsigned WhitespaceLength = |
Krasimir Georgiev | 4fc55a7 | 2018-06-12 19:33:15 +0000 | [diff] [blame] | 888 | Content[LineIndex].data() - Lines[LineIndex].data(); |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 889 | Whitespaces.replaceWhitespaceInToken(*Tokens[LineIndex], |
Krasimir Georgiev | 4fc55a7 | 2018-06-12 19:33:15 +0000 | [diff] [blame] | 890 | Offset, |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 891 | /*ReplaceChars=*/WhitespaceLength, |
| 892 | /*PreviousPostfix=*/"", |
| 893 | /*CurrentPrefix=*/ReflowPrefix, |
| 894 | /*InPPDirective=*/false, |
| 895 | /*Newlines=*/0, |
| 896 | /*Spaces=*/0); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 897 | } |
| 898 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 899 | void BreakableLineCommentSection::adaptStartOfLine( |
| 900 | unsigned LineIndex, WhitespaceManager &Whitespaces) const { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 901 | // If this is the first line of a token, we need to inform Whitespace Manager |
| 902 | // about it: either adapt the whitespace range preceding it, or mark it as an |
| 903 | // untouchable token. |
| 904 | // This happens for instance here: |
| 905 | // // line 1 \ |
| 906 | // // line 2 |
| 907 | if (LineIndex > 0 && Tokens[LineIndex] != Tokens[LineIndex - 1]) { |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 908 | // This is the first line for the current token, but no reflow with the |
| 909 | // previous token is necessary. However, we still may need to adjust the |
| 910 | // start column. Note that ContentColumn[LineIndex] is the expected |
| 911 | // content column after a possible update to the prefix, hence the prefix |
| 912 | // length change is included. |
| 913 | unsigned LineColumn = |
| 914 | ContentColumn[LineIndex] - |
| 915 | (Content[LineIndex].data() - Lines[LineIndex].data()) + |
| 916 | (OriginalPrefix[LineIndex].size() - Prefix[LineIndex].size()); |
Krasimir Georgiev | 13dbaa0 | 2017-02-01 10:10:04 +0000 | [diff] [blame] | 917 | |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 918 | // We always want to create a replacement instead of adding an untouchable |
| 919 | // token, even if LineColumn is the same as the original column of the |
| 920 | // token. This is because WhitespaceManager doesn't align trailing |
| 921 | // comments if they are untouchable. |
| 922 | Whitespaces.replaceWhitespace(*Tokens[LineIndex], |
| 923 | /*Newlines=*/1, |
| 924 | /*Spaces=*/LineColumn, |
| 925 | /*StartOfTokenColumn=*/LineColumn, |
| 926 | /*InPPDirective=*/false); |
Krasimir Georgiev | b796ceb | 2017-01-31 15:40:15 +0000 | [diff] [blame] | 927 | } |
| 928 | if (OriginalPrefix[LineIndex] != Prefix[LineIndex]) { |
| 929 | // Adjust the prefix if necessary. |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 930 | |
| 931 | // Take care of the space possibly introduced after a decoration. |
| 932 | assert(Prefix[LineIndex] == (OriginalPrefix[LineIndex] + " ").str() && |
Krasimir Georgiev | b796ceb | 2017-01-31 15:40:15 +0000 | [diff] [blame] | 933 | "Expecting a line comment prefix to differ from original by at most " |
| 934 | "a space"); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 935 | Whitespaces.replaceWhitespaceInToken( |
| 936 | tokenAt(LineIndex), OriginalPrefix[LineIndex].size(), 0, "", "", |
Daniel Jasper | 7d42f3f | 2017-01-31 11:25:01 +0000 | [diff] [blame] | 937 | /*InPPDirective=*/false, /*Newlines=*/0, /*Spaces=*/1); |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 938 | } |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 939 | } |
| 940 | |
Manuel Klimek | 89628f6 | 2017-09-20 09:51:03 +0000 | [diff] [blame] | 941 | void BreakableLineCommentSection::updateNextToken(LineState &State) const { |
Krasimir Georgiev | 9183422 | 2017-01-25 13:58:58 +0000 | [diff] [blame] | 942 | if (LastLineTok) { |
| 943 | State.NextToken = LastLineTok->Next; |
| 944 | } |
| 945 | } |
| 946 | |
Krasimir Georgiev | 00c5c72 | 2017-02-02 15:32:19 +0000 | [diff] [blame] | 947 | bool BreakableLineCommentSection::mayReflow( |
| 948 | unsigned LineIndex, llvm::Regex &CommentPragmasRegex) const { |
| 949 | // Line comments have the indent as part of the prefix, so we need to |
| 950 | // recompute the start of the line. |
| 951 | StringRef IndentContent = Content[LineIndex]; |
| 952 | if (Lines[LineIndex].startswith("//")) { |
| 953 | IndentContent = Lines[LineIndex].substr(2); |
| 954 | } |
Manuel Klimek | 93699f4 | 2017-11-29 14:29:43 +0000 | [diff] [blame] | 955 | // FIXME: Decide whether we want to reflow non-regular indents: |
| 956 | // Currently, we only reflow when the OriginalPrefix[LineIndex] matches the |
| 957 | // OriginalPrefix[LineIndex-1]. That means we don't reflow |
| 958 | // // text that protrudes |
| 959 | // // into text with different indent |
| 960 | // We do reflow in that case in block comments. |
Krasimir Georgiev | 00c5c72 | 2017-02-02 15:32:19 +0000 | [diff] [blame] | 961 | return LineIndex > 0 && !CommentPragmasRegex.match(IndentContent) && |
| 962 | mayReflowContent(Content[LineIndex]) && !Tok.Finalized && |
| 963 | !switchesFormatting(tokenAt(LineIndex)) && |
| 964 | OriginalPrefix[LineIndex] == OriginalPrefix[LineIndex - 1]; |
| 965 | } |
| 966 | |
Alexander Kornienko | cb45bc1 | 2013-04-15 14:28:00 +0000 | [diff] [blame] | 967 | } // namespace format |
| 968 | } // namespace clang |