resolve-var for symbol with dot still wrong

Description

We need to recur on the first segment passing an new additional argument to resolve-var indicating that we should not try to resolve in the current namespace and instead warn.

Environment

None

Activity

Show:
Mike Fikes
February 1, 2019, 2:18 AM

FWIW, the issue Thomas points to does in fact derail self-hosted ClojureScript, so if we improved warnings around this, that would be good. See

Thomas Heller
September 5, 2019, 3:13 PM
Edited

Found another case where this is “hiding” other bugs. A single dot is accepted as a symbol and only fails at runtime when it should at least warn or throw during compilation.

This hides another problem with type inference where (if (. response -ok) ...) is trying to resolve the . here.

Mike Fikes
September 5, 2019, 3:50 PM

For the very last comment, see CLJS-3163.

David Nolen
April 10, 2020, 3:25 PM
Edited

See also CLJS-2957

 

David Nolen
April 10, 2020, 6:29 PM
Fixed

Assignee

David Nolen

Reporter

David Nolen

Labels

None

Approval

None

Patch

None

Fix versions

Affects versions

Priority

Blocker