line wrapping breaks in weird places

[jdavisclark]

per @jackocnr via jdavisclark/JsFormat#103:

---

It breaks after negation e.g.

if ( .... lots of conditions here ... && !someVar) {

becomes

if ( .... lots of conditions here ... && !
    someVar) {

and also before property names e.g.

if ( .... lots of conditions here ... && someVar.someProperty) {

becomes

if ( .... lots of conditions here ... && someVar
    .someProperty) {

Both make the code way less readable. Perhaps it should only break on spaces? Or between logical operators e.g. && or ||?

[bitwiseman]

The negation case is definitely fixable. We can keep the ! with the following word.
The property case basically by design - what if the properties keep going?

if ( .... lots of conditions here ... && someVar.someProperty.someProperty2,.someProperty3.someProperty4) {

When should we give up and choose to break? (#200 limits our knowledge) We choose to wrap.

[bitwiseman]

Just to be clear, I made a change that should get rid of some cases where ! is left on the previous line. There might be more cases of ! or other unary operators than need addressing. Feel free to file those.

The property stuff is by design.

[clemblanco]

Can we have if (! something) {?

[bitwiseman]

@ClayMM - Please file a new issue with input, expected output, and actual output.