Double warning for undeclared Var (REPL only)

Description

A regression occurred where an undeclared Var in a {{require}}d file causes two diagnostics:

Environment

None

Activity

Show:
David Nolen
July 8, 2017, 6:23 PM

Can confirm this is still a problem thanks to bump from António, however I do not understand the patch. It doesn't seem safe to suppress warnings here like that.

David Nolen
July 8, 2017, 4:34 PM

pretty sure this was resolved when Mike Fikes fixed how we process fn bodies.

Thomas Heller
November 14, 2016, 3:51 PM

FWIW I don't think this is related to the REPL at all.

I have been seeing doubled warnings for a while now in shadow-build but never bothered to find you why.

Will always warn twice about "xzy" but only once for "abc", doesn't matter if a REPL is involved or not.

António Monteiro
November 14, 2016, 3:29 PM

I just now realized that it will probably affect those cases as well, although the `add-dependencies` function seems to (currently) only be used in `cljs.repl`. What other approach should I try? Restrict the cases where we

at the REPL?

Completed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

David Nolen

Reporter

Mike Fikes

Labels

Patch

Code