microsoft/TypeScript
1
0
Fork 0
mirror of https://github.com/microsoft/TypeScript.git synced 2026-05-15 12:51:30 -05:00
Code Issues Packages Projects Releases 212 Wiki Activity

Fixed trimming comments on the remaining range (#45807)

Browse Source 
* Fixed trimming comments on the remaining range

* Added test
This commit is contained in:
Armando Aguirre
2021-09-17 14:00:04 -07:00
committed by GitHub
parent 3c27c3a5ca
commit 8346143450
2 changed files with 37 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
src/services/formatting/formatting.ts
View File

@@ -432,13 +432,12 @@ namespace ts.formatting {
if (leadingTrivia) {
indentTriviaItems(leadingTrivia, indentation, /*indentNextTokenOrTrivia*/ false,
item => processRange(item, sourceFile.getLineAndCharacterOfPosition(item.pos), enclosingNode, enclosingNode, /*dynamicIndentation*/ undefined!));
if (options.trimTrailingWhitespace !== false) {
trimTrailingWhitespacesForRemainingRange(leadingTrivia);
}
}
}
if (options.trimTrailingWhitespace !== false) {
trimTrailingWhitespacesForRemainingRange();
}
return edits;
// local functions
@@ -1142,13 +1141,29 @@ namespace ts.formatting {
}
/**
* Trimming will be done for lines after the previous range
* Trimming will be done for lines after the previous range.
* Exclude comments as they had been previously processed.
*/
function trimTrailingWhitespacesForRemainingRange() {
const startPosition = previousRange ? previousRange.end : originalRange.pos;
function trimTrailingWhitespacesForRemainingRange(trivias: TextRangeWithKind<SyntaxKind>[]) {
let startPos = previousRange ? previousRange.end : originalRange.pos;
for (const trivia of trivias) {
if (isComment(trivia.kind)) {
if (startPos < trivia.pos) {
trimTrailingWitespacesForPositions(startPos, trivia.pos - 1, previousRange);
}
const startLine = sourceFile.getLineAndCharacterOfPosition(startPosition).line;
const endLine = sourceFile.getLineAndCharacterOfPosition(originalRange.end).line;
startPos = trivia.end + 1;
}
}
if (startPos < originalRange.end) {
trimTrailingWitespacesForPositions(startPos, originalRange.end, previousRange);
}
}
function trimTrailingWitespacesForPositions(startPos: number, endPos: number, previousRange: TextRangeWithKind) {
const startLine = sourceFile.getLineAndCharacterOfPosition(startPos).line;
const endLine = sourceFile.getLineAndCharacterOfPosition(endPos).line;
trimTrailingWhitespacesForLines(startLine, endLine + 1, previousRange);
}