Fix gensym leak at toplevel

michaelficarra / CoffeeScriptRedux

:sweat: rewrite of the CoffeeScript compiler with proper compiler design principles and a focus on robustness and extensibility

https://michaelficarra.github.com/CoffeeScriptRedux/

BSD 3-Clause "New" or "Revised" License

1.84k stars 110 forks source link

Fix gensym leak at toplevel #350

Closed ef4 closed 9 years ago

ef4 commented 9 years ago

This closes #251.

Unshifting declarations during the walk phase is too early, it needs to be done during generateSymbols. But the existing generateSymbols never visits the topmost node.

michaelficarra commented 9 years ago

Wow, thanks.

ef4 commented 9 years ago

This is all part of my grand scheme to make this into a coffeescript-to-es6 compiler. :smile: