coffeeClasses improved compatibility: private static class fields via =, bound methods via => , constructor shouldn't return

[edemaine]

Improved compatibility with CoffeeScript under coffeeClasses mode:

• Private static classic fields via x = y within a class body, by wrapping body in an IIFE and adding some local variables. Probably only works with autoVar enabled (like various other coffeeCompat features). Fixes Coffeescript runtime bug: private static class fields #457
• Bound methods via f: => within a class body, by adding to the constructor (after super call if there is one). Also rewrite into method, similar to f: ->. Fixes coffeeCompat thicc arrow function in classes #55.
• Fix constructor getting an automatically inserted return in most cases. (Recent regression perhaps? This would make coffeeClasses pretty unusable.)

Compilations should match CoffeeScript's compilations.

[STRd6]

Thanks!

[greghuc]

@edemaine many thanks for all this. The generated code even looks nicer than Coffeescript!

This will unblock me on my major CS->Civet migration. I'll aim to make another attempt after Christmas. Fingers crossed on no more big bugs..

Thanks again.

[edemaine]

Great, thanks! Let us know how it goes. (Feel free to join us on Discord too.) Have a good Christmas!