Normalize whitespace for transformed JSX code

[syranide]

It currently generates "pretty output" as we agreed upon, it respects original indenting all that, newlines are kept as-is. It also puts {} at just after/before the outer ().

There is only one case that I know of currently that produces incorrect output and that is when the attribute name and value are on different lines (see below), I could fix it, but I don't think it's worth the code complexity for something you really shouldn't do, but I could fix it.

I back-tracked on removing trailing spaces after commas, it's not super hard to do, but it's a bit of messy to sort out and I don't find it worthwhile right now. I'll leave it for another PR for now.

I haven't had time to test it extensively yet.

PS. There's a also a bunch of source code lines that got trimmed now that I'm using a better editor (I unintentionally added them with my previous whitespace PR). So that's why there's a bunch of weird "empty" changes below.

<div>
  <div    >  </   div   ><div></div>
  <    div>{0}    A    B    {1}    {2}</div>
  <div>    A    </div>
  <div>
      A
  </div>
  <div

    a

    =

    "a"

    b =
    "b"

    c
    = "c"
  />
  <div a="A" b="B" />
  <div a="A"/>
  <div   a  =     "b"   c = "d"    />
  <div
    a="A"
    b="B"

    c="C"

  />
  <div id="test"
    >


    {0}A{1}B{2}C {    D
  }E


    A


    B


    {1}


    {2}

    awdawdawd
    awdawdawd

  </div>

Produces:

React.DOM.div(null,
  React.DOM.div(null, "  "), React.DOM.div(null),
  React.DOM.div(null, 0, "    A    B    ", 1, "    ", 2),
  React.DOM.div(null, "    A    "),
  React.DOM.div(null,
      "A"
  ),
  React.DOM.div({

    a:



"a",

    b:
"b",

    c:
"c"
  }),
  React.DOM.div({a: "A", b: "B"}),
  React.DOM.div({a: "A"}),
  React.DOM.div({a: "b", c: "d"}),
  React.DOM.div({
    a: "A",
    b: "B",

    c: "C"

  }),
  React.DOM.div({id: "test"
    },


    0, "A", 1, "B", 2, "C ", D,
  "E" + ' ' +


    "A" + ' ' +


    "B",


    1,


    2,

    "awdawdawd" + ' ' +
    "awdawdawd"

  )
)

[zpao]

Now that our transforms are all synced, want to rebase this? I assume @jeffmo has no qualms with it.

[syranide]

@zpao For sure! Rebased.

I discussed quite a bit with @jeffmo before doing this, so he should be fine with the output. But best check to see if he has any feedback.

[jeffmo]

Can you rebase this on top of your other two commits that just landed?

[syranide]

@jeffmo Rebased, no conflicts.