Fix spacing in binary expression when right is a binary expression and has a unary on the left

[amasad]

Fixes https://phabricator.babeljs.io/T6736

cc @loganfsmyth @spicyj @sebmck

[loganfsmyth]

Seems like we should get some - cases in here too.

[amasad]

ok

[codecov-io]

Current coverage is 84.86%

Merging #3121 into master will increase coverage by +0.01% as of afa9174

@@            master   #3121   diff @@
======================================
  Files          214     214       
  Stmts        15547   15551     +4
  Branches      3314    3315     +1
  Methods          0       0       
======================================
+ Hit          13193   13197     +4
  Partial        676     676       
  Missed        1678    1678       

Review entire Coverage Diff as of afa9174

Powered by Codecov. Updated on successful CI builds.

[sophiebits]

👍 looks good.

[phantom10111]

@amasad This fixes the issue, but for example input a+(+b<<2) produces output a+ (+b<<2), with an unnecessary space. Not sure if you care about saving that one byte, but just letting you know.

[amasad]

I do care. Thanks for pointing that out. I'll look into taking prenthesized
expressions into account.

On Tuesday, December 1, 2015, phantom10111 [email protected] wrote:

@amasad https://github.com/amasad This fixes the issue, but for example
input a+(+b<<2) produces output a+ (+b<<2), with an unnecessary space.
Not sure if you care about saving that one byte, but just letting you know.

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

[mariusk]

@amasad Yes, I can confirm it works as expected now. Thanks!

[hzoo]

Can add && !(node.right.extra && node.right.extra.parenthesized) for the parenthesized check - can do a pr later if you don't. Nvm it's pretty simple #3122