flix icon indicating copy to clipboard operation
flix copied to clipboard

Introduce `CaseSym`?

Open mlutze opened this issue 2 years ago • 5 comments

Cases are currently simply Idents. It would be more consistent with our other declarations to give them proper symbols.

mlutze avatar Jul 30 '22 21:07 mlutze

What about fields and predicates? I think they get a name but not a symbol.

magnus-madsen avatar Jul 31 '22 09:07 magnus-madsen

But those are for structural types. You can't point to a field and say "this is where it is declared"

mlutze avatar Jul 31 '22 09:07 mlutze

But those are for structural types. You can't point to a field and say "this is where it is declared"

Fair point. In that case I am sold.

magnus-madsen avatar Jul 31 '22 09:07 magnus-madsen

This becomes a little scary as the EnumSym + Name.Tag pattern goes all the way to the backend.

mlutze avatar Aug 01 '22 19:08 mlutze

I think its many changes, but all of them trivial. We don't mess with tags.

magnus-madsen avatar Aug 01 '22 21:08 magnus-madsen