search

unisonweb / unison

A friendly programming language from the future
https://unison-lang.org
Other
5.81k stars 271 forks source link

the term parser doesn't think `lower.Upper` is a constructor #5320

Closed mitchellwrosen closed 2 months ago

mitchellwrosen commented 2 months ago

There's logic in the term parser that tries to distinguish vars from constructors by checking whether the first letter is uppercase. It's incorrectly checking the first letter of the entire name, rather than the first letter of the last segment.

aryairani commented 2 months ago

We should not accept multi-segment identifiers as vars anyway; I dunno how many of those are out in the wild.