ClojureScript function params shadow top-level namespace segments

2012-04-13 Thread Stuart Campbell
Given the following ClojureScript:

(ns foo)
(defn bar [] 42)

(ns baz
  (:require [foo :as x]))

(defn quux [foo]
  (x/bar))

baz.quux compiles to:

baz.quux = function quux(foo) {
  return foo.bar.call(null)
};

i.e. the parameter name shadows the top-level foo namespace.

Is that a bug?

Regards,
Stuart

-- 
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

Re: ClojureScript function params shadow top-level namespace segments

2012-04-13 Thread David Nolen
Yes. Please open a ticket in JIRA.

Thanks!

On Fri, Apr 13, 2012 at 3:40 AM, Stuart Campbell stu...@harto.org wrote:

 Given the following ClojureScript:

 (ns foo)
 (defn bar [] 42)

 (ns baz
   (:require [foo :as x]))

 (defn quux [foo]
   (x/bar))

 baz.quux compiles to:

 baz.quux = function quux(foo) {
   return foo.bar.call(null)
 };

 i.e. the parameter name shadows the top-level foo namespace.

 Is that a bug?

 Regards,
 Stuart

 --
 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 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

Re: ClojureScript function params shadow top-level namespace segments

2012-04-13 Thread Stuart Campbell
Done: http://dev.clojure.org/jira/browse/CLJS-180

On 13 April 2012 22:14, David Nolen dnolen.li...@gmail.com wrote:

 Yes. Please open a ticket in JIRA.

 Thanks!

 On Fri, Apr 13, 2012 at 3:40 AM, Stuart Campbell stu...@harto.org wrote:

 Given the following ClojureScript:

 (ns foo)
 (defn bar [] 42)

 (ns baz
   (:require [foo :as x]))

 (defn quux [foo]
   (x/bar))

 baz.quux compiles to:

 baz.quux = function quux(foo) {
   return foo.bar.call(null)
 };

 i.e. the parameter name shadows the top-level foo namespace.

 Is that a bug?

 Regards,
 Stuart

  --
 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 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 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