Don't recognize directives when ECMAVersion < 5

marijnh · marijnh · commit 9c2cad5c9b51 · 2022-04-27T13:02:47.000+02:00
FIX: Fix an issue where the tree contained `directive` properties in when parsing with a language version that doesn't support them. Closes #1126
diff --git a/acorn/src/statement.js b/acorn/src/statement.js
@@ -1051,6 +1051,7 @@ pp.adaptDirectivePrologue = function(statements) {
 }
 pp.isDirectiveCandidate = function(statement) {
   return (
+    this.options.ecmaVersion >= 5 &&
     statement.type === "ExpressionStatement" &&
     statement.expression.type === "Literal" &&
     typeof statement.expression.value === "string" &&

Original file line number	Diff line number	Diff line change
`@@ -1051,6 +1051,7 @@ pp.adaptDirectivePrologue = function(statements) {`
`1051`	`1051`	`}`
`1052`	`1052`	`pp.isDirectiveCandidate = function(statement) {`
`1053`	`1053`	`return (`
	`1054`	`+ this.options.ecmaVersion >= 5 &&`
`1054`	`1055`	`statement.type === "ExpressionStatement" &&`
`1055`	`1056`	`statement.expression.type === "Literal" &&`
`1056`	`1057`	`typeof statement.expression.value === "string" &&`