Revert "Only issue matching token errors on non-dupe locations (micro…

…soft#43460)" This reverts commit 76a2ae3.
jessetrinity · May 19, 2021 · bcef31c · bcef31c
1 parent e67da8a
commit bcef31c
Show file tree

Hide file tree

Showing 40 changed files with 334 additions and 25 deletions.
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -1336,27 +1336,24 @@ namespace ts {
             return inContext(NodeFlags.AwaitContext);
         }
 
-        function parseErrorAtCurrentToken(message: DiagnosticMessage, arg0?: any): DiagnosticWithDetachedLocation | undefined {
-            return parseErrorAt(scanner.getTokenPos(), scanner.getTextPos(), message, arg0);
+        function parseErrorAtCurrentToken(message: DiagnosticMessage, arg0?: any): void {
+            parseErrorAt(scanner.getTokenPos(), scanner.getTextPos(), message, arg0);
         }
 
-        function parseErrorAtPosition(start: number, length: number, message: DiagnosticMessage, arg0?: any): DiagnosticWithDetachedLocation | undefined {
+        function parseErrorAtPosition(start: number, length: number, message: DiagnosticMessage, arg0?: any): void {
             // Don't report another error if it would just be at the same position as the last error.
             const lastError = lastOrUndefined(parseDiagnostics);
-            let result: DiagnosticWithDetachedLocation | undefined;
             if (!lastError || start !== lastError.start) {
-                result = createDetachedDiagnostic(fileName, start, length, message, arg0);
-                parseDiagnostics.push(result);
+                parseDiagnostics.push(createDetachedDiagnostic(fileName, start, length, message, arg0));
             }
 
             // Mark that we've encountered an error.  We'll set an appropriate bit on the next
             // node we finish so that it can't be reused incrementally.
             parseErrorBeforeNextFinishedNode = true;
-            return result;
         }
 
-        function parseErrorAt(start: number, end: number, message: DiagnosticMessage, arg0?: any): DiagnosticWithDetachedLocation | undefined {
-            return parseErrorAtPosition(start, end - start, message, arg0);
+        function parseErrorAt(start: number, end: number, message: DiagnosticMessage, arg0?: any): void {
+            parseErrorAtPosition(start, end - start, message, arg0);
         }
 
         function parseErrorAtRange(range: TextRange, message: DiagnosticMessage, arg0?: any): void {
@@ -1547,17 +1544,17 @@ namespace ts {
         }
 
         function parseExpectedMatchingBrackets(openKind: SyntaxKind, closeKind: SyntaxKind, openPosition: number) {
-            if (token() === closeKind) {
-                nextToken();
-                return;
-            }
-            const lastError = parseErrorAtCurrentToken(Diagnostics._0_expected, tokenToString(closeKind));
-            if (lastError) {
-                addRelatedInfo(
-                    lastError,
-                    createDetachedDiagnostic(fileName, openPosition, 1, Diagnostics.The_parser_expected_to_find_a_1_to_match_the_0_token_here, tokenToString(openKind), tokenToString(closeKind))
-                );
+            if (!parseExpected(closeKind)) {
+                const lastError = lastOrUndefined(parseDiagnostics);
+                if (lastError && lastError.code === Diagnostics._0_expected.code) {
+                    addRelatedInfo(
+                        lastError,
+                        createDetachedDiagnostic(fileName, openPosition, 1, Diagnostics.The_parser_expected_to_find_a_1_to_match_the_0_token_here, tokenToString(openKind), tokenToString(closeKind))
+                    );
+                }
+                return false;
             }
+            return true;
         }
 
         function parseOptional(t: SyntaxKind): boolean {
@@ -5523,7 +5520,15 @@ namespace ts {
             parseExpected(SyntaxKind.OpenBraceToken);
             const multiLine = scanner.hasPrecedingLineBreak();
             const properties = parseDelimitedList(ParsingContext.ObjectLiteralMembers, parseObjectLiteralElement, /*considerSemicolonAsDelimiter*/ true);
-            parseExpectedMatchingBrackets(SyntaxKind.OpenBraceToken, SyntaxKind.CloseBraceToken, openBracePosition);
+            if (!parseExpected(SyntaxKind.CloseBraceToken)) {
+                const lastError = lastOrUndefined(parseDiagnostics);
+                if (lastError && lastError.code === Diagnostics._0_expected.code) {
+                    addRelatedInfo(
+                        lastError,
+                        createDetachedDiagnostic(fileName, openBracePosition, 1, Diagnostics.The_parser_expected_to_find_a_1_to_match_the_0_token_here, tokenToString(SyntaxKind.OpenBraceToken), tokenToString(SyntaxKind.CloseBraceToken))
+                    );
+                }
+            }
             return finishNode(factory.createObjectLiteralExpression(properties, multiLine), pos);
         }
 
@@ -8010,9 +8015,13 @@ namespace ts {
                             hasChildren = true;
                             if (child.kind === SyntaxKind.JSDocTypeTag) {
                                 if (childTypeTag) {
-                                    const lastError = parseErrorAtCurrentToken(Diagnostics.A_JSDoc_typedef_comment_may_not_contain_multiple_type_tags);
+                                    parseErrorAtCurrentToken(Diagnostics.A_JSDoc_typedef_comment_may_not_contain_multiple_type_tags);
+                                    const lastError = lastOrUndefined(parseDiagnostics);
                                     if (lastError) {
-                                        addRelatedInfo(lastError, createDetachedDiagnostic(fileName, 0, 0, Diagnostics.The_tag_was_first_specified_here));
+                                        addRelatedInfo(
+                                            lastError,
+                                            createDetachedDiagnostic(fileName, 0, 0, Diagnostics.The_tag_was_first_specified_here)
+                                        );
                                     }
                                     break;
                                 }

diff --git a/tests/baselines/reference/constructorWithIncompleteTypeAnnotation.errors.txt b/tests/baselines/reference/constructorWithIncompleteTypeAnnotation.errors.txt
@@ -505,6 +505,7 @@ tests/cases/compiler/constructorWithIncompleteTypeAnnotation.ts(261,1): error TS
 !!! error TS1135: Argument expression expected.
                                     ~
 !!! error TS1005: '(' expected.
+!!! related TS1007 tests/cases/compiler/constructorWithIncompleteTypeAnnotation.ts:257:33: The parser expected to find a ')' to match the '(' token here.
                                       ~~~~~~
 !!! error TS2693: 'string' only refers to a type, but is being used as a value here.
                                               ~~~

diff --git a/tests/baselines/reference/destructuringParameterDeclaration2.errors.txt b/tests/baselines/reference/destructuringParameterDeclaration2.errors.txt
@@ -39,6 +39,7 @@ tests/cases/conformance/es6/destructuring/destructuringParameterDeclaration2.ts(
 !!! error TS2322: Type 'string' is not assignable to type 'number'.
                                 ~
 !!! error TS1005: ',' expected.
+!!! related TS1007 tests/cases/conformance/es6/destructuring/destructuringParameterDeclaration2.ts:7:4: The parser expected to find a ']' to match the '[' token here.
     a0([1, 2, [["world"]], "string"]);  // Error
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 !!! error TS2345: Argument of type '[number, number, string[][], string]' is not assignable to parameter of type '[number, number, string[][]]'.

diff --git a/tests/baselines/reference/errorRecoveryWithDotFollowedByNamespaceKeyword.errors.txt b/tests/baselines/reference/errorRecoveryWithDotFollowedByNamespaceKeyword.errors.txt
@@ -16,4 +16,5 @@ tests/cases/compiler/errorRecoveryWithDotFollowedByNamespaceKeyword.ts(9,2): err
     }
 
 !!! error TS1005: '}' expected.
-!!! related TS1007 tests/cases/compiler/errorRecoveryWithDotFollowedByNamespaceKeyword.ts:3:19: The parser expected to find a '}' to match the '{' token here.
+!!! related TS1007 tests/cases/compiler/errorRecoveryWithDotFollowedByNamespaceKeyword.ts:3:19: The parser expected to find a '}' to match the '{' token here.
+!!! related TS1007 tests/cases/compiler/errorRecoveryWithDotFollowedByNamespaceKeyword.ts:2:20: The parser expected to find a '}' to match the '{' token here.
diff --git a/tests/baselines/reference/nestedClassDeclaration.errors.txt b/tests/baselines/reference/nestedClassDeclaration.errors.txt
@@ -32,6 +32,7 @@ tests/cases/conformance/classes/nestedClassDeclaration.ts(17,1): error TS1128: D
 !!! error TS2304: Cannot find name 'C4'.
                  ~
 !!! error TS1005: ',' expected.
+!!! related TS1007 tests/cases/conformance/classes/nestedClassDeclaration.ts:14:9: The parser expected to find a '}' to match the '{' token here.
         }
     }
     ~

diff --git a/tests/baselines/reference/objectLiteralWithSemicolons4.errors.txt b/tests/baselines/reference/objectLiteralWithSemicolons4.errors.txt
@@ -9,4 +9,5 @@ tests/cases/compiler/objectLiteralWithSemicolons4.ts(3,1): error TS1005: ',' exp
 !!! error TS18004: No value exists in scope for the shorthand property 'a'. Either declare one or provide an initializer.
     ;
     ~
-!!! error TS1005: ',' expected.
+!!! error TS1005: ',' expected.
+!!! related TS1007 tests/cases/compiler/objectLiteralWithSemicolons4.ts:1:9: The parser expected to find a '}' to match the '{' token here.
diff --git a/tests/baselines/reference/objectSpreadNegativeParse.errors.txt b/tests/baselines/reference/objectSpreadNegativeParse.errors.txt
@@ -28,6 +28,7 @@ tests/cases/conformance/types/spread/objectSpreadNegativeParse.ts(4,20): error T
 !!! error TS2304: Cannot find name 'matchMedia'.
                                ~
 !!! error TS1005: ',' expected.
+!!! related TS1007 tests/cases/conformance/types/spread/objectSpreadNegativeParse.ts:3:10: The parser expected to find a '}' to match the '{' token here.
                                   ~
 !!! error TS1128: Declaration or statement expected.
     let o10 = { ...get x() { return 12; }};

diff --git a/tests/baselines/reference/parseErrorIncorrectReturnToken.errors.txt b/tests/baselines/reference/parseErrorIncorrectReturnToken.errors.txt
@@ -25,6 +25,7 @@ tests/cases/compiler/parseErrorIncorrectReturnToken.ts(12,1): error TS1128: Decl
         m(n: number) => string {
                      ~~
 !!! error TS1005: '{' expected.
+!!! related TS1007 tests/cases/compiler/parseErrorIncorrectReturnToken.ts:8:9: The parser expected to find a '}' to match the '{' token here.
                         ~~~~~~
 !!! error TS2693: 'string' only refers to a type, but is being used as a value here.
                                ~

diff --git a/tests/baselines/reference/parserErrorRecoveryArrayLiteralExpression3.errors.txt b/tests/baselines/reference/parserErrorRecoveryArrayLiteralExpression3.errors.txt
@@ -8,6 +8,7 @@ tests/cases/conformance/parser/ecmascript5/ErrorRecovery/ArrayLiteralExpressions
     var texCoords = [2, 2, 0.5000001192092895, 0.8749999 ; 403953552, 0.5000001192092895, 0.8749999403953552];
                                                          ~
 !!! error TS1005: ',' expected.
+!!! related TS1007 tests/cases/conformance/parser/ecmascript5/ErrorRecovery/ArrayLiteralExpressions/parserErrorRecoveryArrayLiteralExpression3.ts:1:17: The parser expected to find a ']' to match the '[' token here.
                                                            ~~~~~~~~~
 !!! error TS2695: Left side of comma operator is unused and has no side effects.
                                                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

diff --git a/tests/baselines/reference/parserFuzz1.errors.txt b/tests/baselines/reference/parserFuzz1.errors.txt
@@ -20,4 +20,5 @@ tests/cases/conformance/parser/ecmascript5/ErrorRecovery/parserFuzz1.ts(2,15): e
         ~~~~~~
 !!! error TS1005: ';' expected.
 
-!!! error TS1005: '{' expected.
+!!! error TS1005: '{' expected.
+!!! related TS1007 tests/cases/conformance/parser/ecmascript5/ErrorRecovery/parserFuzz1.ts:1:9: The parser expected to find a '}' to match the '{' token here.
diff --git a/tests/baselines/reference/reservedWords2.errors.txt b/tests/baselines/reference/reservedWords2.errors.txt
@@ -105,6 +105,7 @@ tests/cases/compiler/reservedWords2.ts(12,17): error TS1138: Parameter declarati
 !!! error TS1005: ';' expected.
                      ~
 !!! error TS1005: '(' expected.
+!!! related TS1007 tests/cases/compiler/reservedWords2.ts:9:18: The parser expected to find a ')' to match the '(' token here.
                        ~
 !!! error TS1128: Declaration or statement expected.
     enum void {}

diff --git a/...baselines/reference/tsbuild/noEmitOnError/initial-build/syntax-errors-with-incremental.js b/...baselines/reference/tsbuild/noEmitOnError/initial-build/syntax-errors-with-incremental.js
@@ -47,6 +47,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/src/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 
 Found 1 error.
 
@@ -76,6 +81,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/src/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 
 Found 1 error.
 

diff --git a/tests/baselines/reference/tsbuild/noEmitOnError/initial-build/syntax-errors.js b/tests/baselines/reference/tsbuild/noEmitOnError/initial-build/syntax-errors.js
@@ -47,6 +47,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/src/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 
 Found 1 error.
 
@@ -76,6 +81,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/src/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 
 Found 1 error.
 

diff --git a/...ence/tsbuild/watchMode/noEmitOnError/does-not-emit-any-files-on-error-with-incremental.js b/...ence/tsbuild/watchMode/noEmitOnError/does-not-emit-any-files-on-error-with-incremental.js
@@ -56,6 +56,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:35 AM[0m] Found 1 error. Watching for file changes.
 
 
@@ -108,6 +113,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:42 AM[0m] Found 1 error. Watching for file changes.
 
 

diff --git a/...s/baselines/reference/tsbuild/watchMode/noEmitOnError/does-not-emit-any-files-on-error.js b/...s/baselines/reference/tsbuild/watchMode/noEmitOnError/does-not-emit-any-files-on-error.js
@@ -56,6 +56,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:35 AM[0m] Found 1 error. Watching for file changes.
 
 
@@ -108,6 +113,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:42 AM[0m] Found 1 error. Watching for file changes.
 
 

diff --git a/tests/baselines/reference/tsc/incremental/initial-build/with-noEmitOnError-syntax-errors.js b/tests/baselines/reference/tsc/incremental/initial-build/with-noEmitOnError-syntax-errors.js
@@ -47,6 +47,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/src/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 
 Found 1 error.
 
@@ -155,6 +160,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/src/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 
 Found 1 error.
 

diff --git a/...rUpdates/assumeChangesOnlyAffectDirectDependencies/with-noEmitOnError-with-incremental.js b/...rUpdates/assumeChangesOnlyAffectDirectDependencies/with-noEmitOnError-with-incremental.js
@@ -43,6 +43,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:37 AM[0m] Found 1 error. Watching for file changes.
 
 
@@ -175,6 +180,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:44 AM[0m] Found 1 error. Watching for file changes.
 
 

diff --git a/...Watch/emitAndErrorUpdates/assumeChangesOnlyAffectDirectDependencies/with-noEmitOnError.js b/...Watch/emitAndErrorUpdates/assumeChangesOnlyAffectDirectDependencies/with-noEmitOnError.js
@@ -43,6 +43,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:32 AM[0m] Found 1 error. Watching for file changes.
 
 
@@ -99,6 +104,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:37 AM[0m] Found 1 error. Watching for file changes.
 
 

diff --git a/...ates/assumeChangesOnlyAffectDirectDependenciesAndD/with-noEmitOnError-with-incremental.js b/...ates/assumeChangesOnlyAffectDirectDependenciesAndD/with-noEmitOnError-with-incremental.js
@@ -43,6 +43,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:37 AM[0m] Found 1 error. Watching for file changes.
 
 
@@ -176,6 +181,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:44 AM[0m] Found 1 error. Watching for file changes.
 
 

diff --git a/...h/emitAndErrorUpdates/assumeChangesOnlyAffectDirectDependenciesAndD/with-noEmitOnError.js b/...h/emitAndErrorUpdates/assumeChangesOnlyAffectDirectDependenciesAndD/with-noEmitOnError.js
@@ -43,6 +43,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:32 AM[0m] Found 1 error. Watching for file changes.
 
 
@@ -99,6 +104,11 @@ Output::
 [7m4[0m ;
 [7m [0m [91m~[0m
 
+  [96msrc/main.ts[0m:[93m2[0m:[93m11[0m
+    [7m2[0m const a = {
+    [7m [0m [96m          ~[0m
+    The parser expected to find a '}' to match the '{' token here.
+
 [[90m12:00:37 AM[0m] Found 1 error. Watching for file changes.