Merge pull request microsoft#3608 from Microsoft/dontClassifyMissingNodes

vladima · vladima · commit 5b3eba13a2ee · 2015-06-23T11:39:09.000-07:00
do not try to classify missing nodes
diff --git a/src/services/services.ts b/src/services/services.ts
@@ -6300,6 +6300,10 @@ namespace ts {
             }
 
             function classifyToken(token: Node): void {
+                if (nodeIsMissing(token)) {
+                    return;
+                }
+
                 let tokenStart = classifyLeadingTriviaAndGetTokenStart(token);
 
                 let tokenWidth = token.end - tokenStart;
diff --git a/tests/cases/fourslash/syntacticClassificationWithErrors.ts b/tests/cases/fourslash/syntacticClassificationWithErrors.ts
@@ -0,0 +1,14 @@
+﻿/// <reference path='fourslash.ts'/>
+
+////class A {
+////    a:
+////}
+////c =
+
+let c = classification
+verify.syntacticClassificationsAre(
+    c.keyword("class"), c.className("A"), c.punctuation("{"),
+    c.text("a"), c.punctuation(":"),
+    c.punctuation("}"),
+    c.text("c"), c.operator("=")
+    );

Original file line number	Diff line number	Diff line change
`@@ -6300,6 +6300,10 @@ namespace ts {`
`6300`	`6300`	`}`
`6301`	`6301`
`6302`	`6302`	`function classifyToken(token: Node): void {`
	`6303`	`+ if (nodeIsMissing(token)) {`
	`6304`	`+ return;`
	`6305`	`+ }`
	`6306`	`+`
`6303`	`6307`	`let tokenStart = classifyLeadingTriviaAndGetTokenStart(token);`
`6304`	`6308`
`6305`	`6309`	`let tokenWidth = token.end - tokenStart;`