support no space before operator and left paren

[ghost]

NOT(text:go)

[coveralls]

Coverage decreased (-1.1%) to 78.582% when pulling 99c3761 on uxnow:feature-nospace-between-operator-and-left-paren into c51f4e0 on bripkens:master.

[coveralls]

Coverage decreased (-1.1%) to 78.582% when pulling 99c3761 on uxnow:feature-nospace-between-operator-and-left-paren into c51f4e0 on bripkens:master.

[bripkens]

Your tests are not testing what you are describing in the message of your PR. Also, could you please also take care that the AST can be turned into the same string again via toString?

[ghost]

ok, I will look into the tests

[coveralls]

Coverage increased (+0.04%) to 79.764% when pulling 932e792 on uxnow:feature-nospace-between-operator-and-left-paren into c51f4e0 on bripkens:master.

[ghost]

Is It better to convert NOT(text:go) to NOT (text:go)?

[bripkens]

it should preferably be exactly (or very closely) what the user typed in. Meaning in this case, it there is a space, then we should retain the space character.

[ghost]

I see, thank you for your explanation which reminds me some good ideas. I will close this pull request first, thank you again.