Hey Herwig,

I'm currently on [org.clojure/clojurescript "0.0-2261"], and I'm seeing no munging:

paddleguru.api.register.validation> (defn console [])
#<function console(){return null;
}>
nil
paddleguru.api.register.validation> (defn console [s] (.log js/console s))
#<function console(s){return console.log(s);
}>
nil
paddleguru.api.register.validation> (console "HI!")
"Error evaluating:" (console "HI!") :as "paddleguru.api.register.validation.console.call(null,\"HI!\");\n"
#<TypeError: undefined is not a function>
TypeError: undefined is not a function
at console (eval at <anonymous> (https://local.paddleguru.com/cljs/dev/generated.js:83183:294), <anonymous>:1:146) at eval (eval at <anonymous> (https://local.paddleguru.com/cljs/dev/generated.js:83183:294), <anonymous>:1:108) at eval (eval at <anonymous> (https://local.paddleguru.com/cljs/dev/generated.js:83183:294), <anonymous>:5:3)
    at https://local.paddleguru.com/cljs/dev/generated.js:83183:289
    at https://local.paddleguru.com/cljs/dev/generated.js:83197:4
at G__30641__2 (https://local.paddleguru.com/cljs/dev/generated.js:23732:22) at G__30641 [as call] (https://local.paddleguru.com/cljs/dev/generated.js:23977:28) at null.<anonymous> (https://local.paddleguru.com/cljs/dev/generated.js:83238:80) at goog.events.EventTarget.fireListeners (https://local.paddleguru.com/cljs/dev/generated.js:42772:23) at Function.goog.events.EventTarget.dispatchEventInternal_ (https://local.paddleguru.com/cljs/dev/generated.js:42817:26)
nil

Herwig Hochleitner <mailto:hhochleit...@gmail.com>
August 4, 2014 at 11:43 AM
Thomas, in my test, (defn console []) is munged aswell, so I'm guessing that Sam is using a version from before https://github.com/clojure/clojurescript/commit/f371c04d95a00cdda79c63f89f35088d62de8e73
Sam, is that correct?

The observation that eval should be in js-globals is irrelevant, because as I detailed a couple of messages ago, js-globals is wrong and should be removed.
(fn x []) should never cast a shadow on js/x for any x.
So far, I believe David to agree. I commented on CLJS-833 to reflect this point of view.

Waiting on a statement towards fully removing gensyms from the compiler, i.e. CLJS-401 revisited ...
--
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
---
You received this message because you are subscribed to the Google Groups "Clojure" group. To unsubscribe from this group and stop receiving emails from it, send an email to clojure+unsubscr...@googlegroups.com <mailto:clojure+unsubscr...@googlegroups.com>.
For more options, visit https://groups.google.com/d/optout.
Thomas Heller <mailto:th.hel...@gmail.com>
August 4, 2014 at 10:40 AM
https://gist.github.com/thheller/4731f682665d38b1053c

On Monday, August 4, 2014 3:34:33 PM UTC+2, Nicola Mometto wrote: --
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
---
You received this message because you are subscribed to the Google Groups "Clojure" group. To unsubscribe from this group and stop receiving emails from it, send an email to clojure+unsubscr...@googlegroups.com <mailto:clojure+unsubscr...@googlegroups.com>.
For more options, visit https://groups.google.com/d/optout.
Nicola Mometto <mailto:brobro...@gmail.com>
August 4, 2014 at 7:34 AM
Try with (fn document [] js/document)


Thomas Heller <mailto:th.hel...@gmail.com>
August 4, 2014 at 7:30 AM
I was not able to reproduce Sam's initial problem.

"eval" as I said does not appear in the :js-globals which seems like a mistake.
--
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
---
You received this message because you are subscribed to the Google Groups "Clojure" group. To unsubscribe from this group and stop receiving emails from it, send an email to clojure+unsubscr...@googlegroups.com <mailto:clojure+unsubscr...@googlegroups.com>.
For more options, visit https://groups.google.com/d/optout.
Herwig Hochleitner <mailto:hhochleit...@gmail.com>
August 4, 2014 at 7:11 AM
I dug a bit deeper to see where :js-globals came from and found the old ticket for this exact issue: http://dev.clojure.org/jira/browse/CLJS-680 I propose that we remove it, because it's unnessecary when we gensym fn names the same way as let bindings + blacklists are never a great solution, but in the case of an open set of names (such as globals that can differ from runtime to runtime), they are particularly awful.

Ad reproducing the issue: Can the shadowing mechanism be influenced by compiler flags such as :advanced?



--
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
---
You received this message because you are subscribed to the Google Groups "Clojure" group. To unsubscribe from this group and stop receiving emails from it, send an email to clojure+unsubscr...@googlegroups.com <mailto:clojure+unsubscr...@googlegroups.com>.
For more options, visit https://groups.google.com/d/optout.

--
Sam Ritchie (@sritchie)
Paddleguru Co-Founder
703.863.8561
www.paddleguru.com <http://www.paddleguru.com/>
Twitter <http://twitter.com/paddleguru>// Facebook <http://facebook.com/paddleguru>

--
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
--- You received this message because you are subscribed to the Google Groups "Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to