Fix #5378, stack overflow on large files (#5380)

jashkenas · Oct 2, 2021 · d572d74 · d572d74
1 parent e82de9c
commit d572d74
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 21 deletions.
diff --git a/docs/v2/browser-compiler-legacy/coffeescript.js b/docs/v2/browser-compiler-legacy/coffeescript.js
diff --git a/docs/v2/browser-compiler-modern/coffeescript.js b/docs/v2/browser-compiler-modern/coffeescript.js
diff --git a/lib/coffeescript-browser-compiler-legacy/coffeescript.js b/lib/coffeescript-browser-compiler-legacy/coffeescript.js
diff --git a/lib/coffeescript-browser-compiler-modern/coffeescript.js b/lib/coffeescript-browser-compiler-modern/coffeescript.js
diff --git a/lib/coffeescript/nodes.js b/lib/coffeescript/nodes.js
diff --git a/src/nodes.coffee b/src/nodes.coffee
@@ -528,13 +528,8 @@ exports.Root = class Root extends Base
     @initializeScope o
     fragments = @body.compileRoot o
     return fragments if o.bare
-    parts = []
-    parts.push @makeCode '('
-    parts.push @makeCode 'async ' if @isAsync
-    parts.push @makeCode 'function() {\n'
-    parts.push ...fragments
-    parts.push @makeCode '\n}).call(this);\n'
-    [].concat ...parts
+    functionKeyword = "#{if @isAsync then 'async ' else ''}function"
+    [].concat @makeCode("(#{functionKeyword}() {\n"), fragments, @makeCode("\n}).call(this);\n")
 
   initializeScope: (o) ->
     o.scope = new Scope null, @body, null, o.referencedVars ? []