Spacing of range operators

[lucaswerkmeister]

// hand-written, doesn’t work because I can’t influence spacing inside l/r expressions
complex+expression-foo..complex+expression-bar
// current formatted result – confusing because precedence different than spacing
complex + expression - foo..complex + expression - bar
// better?
complex + expression - foo .. complex + expression - bar

Same for : and ....

[lucaswerkmeister]

How about this: No spaces around the operators if l+r expression are

• Literal (1..2)
• BaseMemberExpression (foo)
• QualifiedMemberExpression (foo.bar)

otherwise, spaces.

[gavinking]

Or

• parenthesized

[lucaswerkmeister]

Oh, right. So, Expression with mainToken exists.

[lucaswerkmeister]

No, Expression with (mainToken?.text else "") == "(". a - b is an expression with a mainToken as well.

[lucaswerkmeister]

Okay, the type hierarchy had me confused for a bit… most XYZExpression classes don’t extend Expression. The condition is now term is Atom|Primary || (expr.mainToken?.text else "") == "(". I’ll test a few ranges, and if it looks good, I’ll commit it.

[lucaswerkmeister]

No, that doesn’t make sense, an Expression is an Atom.

[lucaswerkmeister]

I guess this also applies to EntryOp (1->2).