Reject return statement in static block, even if allowReturnOutsideFunction is used

FIX: Fix an issue where the parser would allow `return` statements in `static` blocks when `allowReturnOutsideFunction` was enabled.

Author: ota-meshi

acorn/src/state.js

@@ -115,6 +115,12 @@ export class Parser {
     return (this.inModule && this.options.ecmaVersion >= 13) || this.options.allowAwaitOutsideFunction
   }
 
+  get allowReturn() {
+    if (this.inFunction) return true
+    if (this.options.allowReturnOutsideFunction && this.currentVarScope().flags & SCOPE_TOP) return true
+    return false
+  }
+
   get allowSuper() {
     const {flags} = this.currentThisScope()
     return (flags & SCOPE_SUPER) > 0 || this.options.allowSuperOutsideMethod

acorn/src/statement.js

@@ -356,7 +356,7 @@ pp.parseIfStatement = function(node) {
 }
 
 pp.parseReturnStatement = function(node) {
-  if (!this.inFunction && !this.options.allowReturnOutsideFunction)
+  if (!this.allowReturn)
     this.raise(this.start, "'return' outside of function")
   this.next()
 

test/tests-class-features-2022.js

@@ -6391,3 +6391,5 @@ testFail(`class C {
     return #brand in #brand in obj;
   }
 }`, 'Private identifier can only be left side of binary expression (5:21)', { ecmaVersion: 13 })
+
+testFail(`class X { static { return; } }`, "'return' outside of function (1:19)", {allowReturnOutsideFunction: true, ecmaVersion: 13});