Fix bug: Ensure JSDoc type range is valid (#27343)

microsoft · Oct 9, 2018 · f6ca105 · f6ca105
1 parent dd9b8ca
commit f6ca105
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -6308,7 +6308,7 @@ namespace ts {
 
             // Parses out a JSDoc type expression.
             export function parseJSDocTypeExpression(mayOmitBraces?: boolean): JSDocTypeExpression {
-                const result = <JSDocTypeExpression>createNode(SyntaxKind.JSDocTypeExpression, scanner.getTokenPos());
+                const result = <JSDocTypeExpression>createNode(SyntaxKind.JSDocTypeExpression);
 
                 const hasBrace = (mayOmitBraces ? parseOptional : parseExpected)(SyntaxKind.OpenBraceToken);
                 result.type = doInsideOfContext(NodeFlags.JSDoc, parseJSDocType);

diff --git a/tests/cases/fourslash/editJsdocType.ts b/tests/cases/fourslash/editJsdocType.ts
@@ -0,0 +1,13 @@
+/// <reference path='fourslash.ts' />
+
+// @allowJs: true
+// @noLib: true
+
+// @Filename: /a.js
+/////** @type/**/ */
+////const x = 0;
+
+goTo.marker();
+verify.quickInfoIs("");
+edit.insert(" ");
+verify.quickInfoIs("");