Skip to content

Commit ff0961b

Browse files
committed
Remove trailing space from emitLeadingComment
This will prevent emiting an extraneous trailing space on comments to match https://eslint.org/docs/rules/no-trailing-spaces This also removes a space that may have been originally there after a comment but isn't necessary syntactically.
1 parent 2e02778 commit ff0961b

File tree

45 files changed

+53
-56
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

45 files changed

+53
-56
lines changed

src/compiler/comments.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -274,9 +274,6 @@ namespace ts {
274274
if (hasTrailingNewLine) {
275275
writer.writeLine();
276276
}
277-
else {
278-
writer.write(" ");
279-
}
280277
}
281278

282279
function emitLeadingCommentsOfPosition(pos: number) {

tests/baselines/reference/ambientDeclarationsExternal.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ var n: number;
2424

2525

2626
//// [decls.js]
27-
// Ambient external import declaration referencing ambient external module using top level module name
27+
// Ambient external import declaration referencing ambient external module using top level module name
2828
//// [consumer.js]
2929
"use strict";
3030
exports.__esModule = true;

tests/baselines/reference/anyAssignableToEveryType.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,4 +87,4 @@ function foo(x, y, z) {
8787
// x = a;
8888
// y = a;
8989
// z = a;
90-
//}
90+
//}

tests/baselines/reference/augmentedTypesClass3.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,4 +46,4 @@ var c5c = /** @class */ (function () {
4646
c5c.prototype.foo = function () { };
4747
return c5c;
4848
}());
49-
//import c5c = require('');
49+
//import c5c = require('');

tests/baselines/reference/augmentedTypesEnum.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,4 +100,4 @@ var e6b;
100100
})(e6b || (e6b = {})); // should be error
101101
// enum then import, messes with error reporting
102102
//enum e7 { One }
103-
//import e7 = require(''); // should be error
103+
//import e7 = require(''); // should be error

tests/baselines/reference/augmentedTypesEnum2.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,4 +41,4 @@ var e2 = /** @class */ (function () {
4141
return e2;
4242
}());
4343
//enum then enum - covered
44-
//enum then import - covered
44+
//enum then import - covered

tests/baselines/reference/augmentedTypesFunction.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,4 +79,4 @@ function y5b() { }
7979
function y5c() { }
8080
// function then import, messes with other errors
8181
//function y6() { }
82-
//import y6 = require('');
82+
//import y6 = require('');

tests/baselines/reference/augmentedTypesInterface.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,4 +48,4 @@ var i3;
4848
i3[i3["One"] = 0] = "One";
4949
})(i3 || (i3 = {}));
5050
; // error
51-
//import i4 = require(''); // error
51+
//import i4 = require(''); // error

tests/baselines/reference/baseIndexSignatureResolution.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,4 +59,4 @@ interface B<TBase extends Base> extends A {
5959
}
6060
var b: B<Derived> = null;
6161
var z: Derived = b.foo();
62-
*/
62+
*/

tests/baselines/reference/commentEmitAtEndOfFile1.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,4 +18,4 @@ var foo;
1818
(function (foo) {
1919
function bar() { }
2020
})(foo || (foo = {}));
21-
// test #4
21+
// test #4

0 commit comments

Comments
 (0)