Remove Parantheses

[ghost]

The purpose of !angular_travel is specifically to ensure that exact zero cases
with different target and initial positions get handled at the top
rather than at the bottom where they get handled inconsistently. The
parentheses that were added to the original PR changed the logic back
to what it was initially, with exact zeros with different targets getting handled
by the switch statement.

Thus, the logic should read if (exact zero) or (near zero and target near initial position)

Also, this is my first time using github. Moving forward is it more appropriate to somehow modify the original pull request or to open a new pull request. Github didn't really offer me a way to change the commits once the changes were merged and hence why I opened a new PR.

#20314

[thinkyhead]

You cannot combine || operators and && operators like this without some parentheses. You'll need to put the parentheses where they belong before this can be accepted.

[ghost]

Oh I guess that's a good point. It could be prudent given the use of macros. No worries, in that case I just moved them over to group the && statements together.

[thinkyhead]

Thanks! It's all much clearer now….