unreadable generated JS for comma-separated statements

[makoConstruct]

Currently, when you assign the result of a conditional to something, the code in each block of the conditional will be generated as comma separated statements rather than semicolon separated statements. The problem is that these statements are not each given their own line. To illustrate:

a = if true
  do_things()
  do_more_things()
  generate_result()
else
  0

resolves to

a = true ? (do_things(), do_more_things(), generate_result()) : 0;

If the conditionals have a lot of content in the branches, this can make for some very unreadable generated code. It would be easier to debug our code if, instead, we got

a = true ? (
  do_things(),
  do_more_things(),
  generate_result()
) : (
  0
);

[jashkenas]

Great idea!

[MattKunze]

We use React, but use CoffeeScript instead of JSX for the render functions, and the JS emitted from those sections is less than friendly for this reason. This would help a lot to improve the readability and ease of debugging of these use cases.

[makoConstruct]

Note that sourcemappings provide a better solution for debugging generated
code.

On Wed, Nov 19, 2014 at 6:50 PM, Matt Kunze notifications@github.com
wrote:

We use React, but use CoffeeScript instead of JSX for the render
functions, and the JS emitted from those sections is less than friendly for
this reason. This would help a lot to improve the readability and ease of
debugging of these use cases.

—
Reply to this email directly or view it on GitHub
#3729 (comment)
.

[GeoffreyBooth]

A PR that implements this would be welcome.