Fix microsoft#10758 Add compiler option to parse in strict mode

slawomir · slawomir · commit 8210634e8dbc · 2016-10-10T23:03:07.000+02:00
* fix comment
 * optimize loop
diff --git a/src/compiler/factory.ts b/src/compiler/factory.ts
@@ -2242,19 +2242,16 @@ namespace ts {
      */
     export function ensureUseStrict(node: SourceFile): SourceFile {
         let foundUseStrict = false;
-        let statementOffset = 0;
-        const numStatements = node.statements.length;
-        while (statementOffset < numStatements) {
-            const statement = node.statements[statementOffset];
+        for (const statement of node.statements) {
             if (isPrologueDirective(statement)) {
                 if (isUseStrictPrologue(statement as ExpressionStatement)) {
                     foundUseStrict = true;
+                    break;
                 }
             }
             else {
                 break;
             }
-            statementOffset++;
         }
         if (!foundUseStrict) {
             const statements: Statement[] = [];
diff --git a/src/compiler/transformers/ts.ts b/src/compiler/transformers/ts.ts
@@ -436,7 +436,7 @@ namespace ts {
         function visitSourceFile(node: SourceFile) {
             currentSourceFile = node;
 
-            // ensure "use strict"" is emitted in all scenarios in alwaysStrict mode
+            // ensure "use strict" is emitted in all scenarios in alwaysStrict mode
             if (compilerOptions.alwaysStrict) {
                 node = ensureUseStrict(node);
             }

Original file line number	Diff line number	Diff line change
`@@ -2242,19 +2242,16 @@ namespace ts {`
`2242`	`2242`	`*/`
`2243`	`2243`	`export function ensureUseStrict(node: SourceFile): SourceFile {`
`2244`	`2244`	`let foundUseStrict = false;`
`2245`		`- let statementOffset = 0;`
`2246`		`- const numStatements = node.statements.length;`
`2247`		`- while (statementOffset < numStatements) {`
`2248`		`- const statement = node.statements[statementOffset];`
	`2245`	`+ for (const statement of node.statements) {`
`2249`	`2246`	`if (isPrologueDirective(statement)) {`
`2250`	`2247`	`if (isUseStrictPrologue(statement as ExpressionStatement)) {`
`2251`	`2248`	`foundUseStrict = true;`
	`2249`	`+ break;`
`2252`	`2250`	`}`
`2253`	`2251`	`}`
`2254`	`2252`	`else {`
`2255`	`2253`	`break;`
`2256`	`2254`	`}`
`2257`		`- statementOffset++;`
`2258`	`2255`	`}`
`2259`	`2256`	`if (!foundUseStrict) {`
`2260`	`2257`	`const statements: Statement[] = [];`
Original file line number	Diff line number	Diff line change
`@@ -436,7 +436,7 @@ namespace ts {`
`436`	`436`	`function visitSourceFile(node: SourceFile) {`
`437`	`437`	`currentSourceFile = node;`
`438`	`438`
`439`		`- // ensure "use strict"" is emitted in all scenarios in alwaysStrict mode`
	`439`	`+ // ensure "use strict" is emitted in all scenarios in alwaysStrict mode`
`440`	`440`	`if (compilerOptions.alwaysStrict) {`
`441`	`441`	`node = ensureUseStrict(node);`
`442`	`442`	`}`