Break and return false in forced_brace_bounds? when Parent is a Binary

[weizheheng]

Fixes #122

[weizheheng]

@kddnewton Sorry for opening another PR, here is the original PR. I messed up the original PR while trying to fix the idempotency_test. Unfortunately, the tests are still failing and I appreciate it if you could give me some guidance on fixing it.

Tried running the CI=true bundle exec rake and it gave me more errors as compared to the Github Action run. The error I on Github Action run are:

• I am not sure why the rdoc/parser/ruby.rb and c.rb is raising syntax error.

[kddnewton]

No worries @weizheheng I'll take a look.

[kddnewton]

I did some more digging into this issue. The issue is this kind of expression:

1 while line.gsub!(/\t+/) do
    ' ' * (tab_width*$&.length - $`.length % tab_width)
  end  && $~

In this case, it will get flipped into the non-modifier form, as in:

while line.gsub!(/\t+/) do
        " " * (tab_width * $&.length - $`.length % tab_width)
      end && $~
  1
end

but with your PR it's going to use do..end. This means the do keyword is going to get confused with the optional do keyword of the while loop. Which means we need to do more than to just check if it's a Binary node.

[kddnewton]

I'm going to close this for now since it seems like we won't be able to solve this.