MeAmAnUsername / pie

Pipelines for Interactive Environments
Apache License 2.0
0 stars 0 forks source link

Replace EmptyList with list of bottom type #282

Open MeAmAnUsername opened 2 years ago

MeAmAnUsername commented 2 years ago

Summary Replace the type EmptyListType() with ListType(BottomType()). Depends on #170.

Todo

Reason It is inelegant to have an edge case for empty lists. This is currently required because lists are not subtypes of each other, but that would be resolved by #170

Implementation Once subtyping for lists is allowed, replacing EmptyListType() with ListType(BottomType()) should just work. Explicit checks for empty lists to generate warnings should still work after this replacement.

Related issues