ClojureScript

undeclared ns warning needs to check for existing required namespaces

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

with-out-str currently produces a warning even though goog.string is loaded by core.clj.

Activity

Hide
David Nolen added a comment - - edited

This means we need to always know the entire set of namespaces that are currently required. This will eliminate a long outstanding bug with respect to resolving symbols generated by macros that are in namespaces not explicitly required by users as they are implicit when loading the main library namespace. This is of course known for ClojureScript libraries, but we need special casing for Google Closure libs, Closure compatible libs, and foreign libs that the user provides a namespace for.

Show
David Nolen added a comment - - edited This means we need to always know the entire set of namespaces that are currently required. This will eliminate a long outstanding bug with respect to resolving symbols generated by macros that are in namespaces not explicitly required by users as they are implicit when loading the main library namespace. This is of course known for ClojureScript libraries, but we need special casing for Google Closure libs, Closure compatible libs, and foreign libs that the user provides a namespace for.
Hide
David Nolen added a comment -
Show
David Nolen added a comment - See http://dev.clojure.org/jira/browse/CLJS-615
Hide
David Nolen added a comment -

Resolved by CLJS-615

Show
David Nolen added a comment - Resolved by CLJS-615

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: