This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-06-15
Channels
- # aws-lambda (6)
- # beginners (8)
- # cider (12)
- # cljs-dev (12)
- # cljsrn (9)
- # clojars (2)
- # clojure (161)
- # clojure-berlin (9)
- # clojure-italy (10)
- # clojure-russia (18)
- # clojure-spec (20)
- # clojure-uk (18)
- # clojurescript (37)
- # cursive (13)
- # data-science (1)
- # datomic (23)
- # flambo (2)
- # hoplon (22)
- # immutant (11)
- # jobs (1)
- # klipse (11)
- # leiningen (5)
- # liberator (7)
- # luminus (5)
- # lumo (3)
- # off-topic (28)
- # om (8)
- # onyx (15)
- # parinfer (111)
- # pedestal (21)
- # re-frame (6)
- # reagent (38)
- # rum (4)
- # spacemacs (2)
- # specter (34)
- # unrepl (122)
- # untangled (3)
- # vim (1)
- # yada (8)
An interesting behaviour:
Lumo 1.4.1
ClojureScript 1.9.521
Node.js v7.9.0
Docs: (doc function-name-here)
Exit: Control+D or :cljs/quit or exit
cljs.user=> (doc doc)
-------------------------
lumo.repl/doc
([name])
Macro
Prints documentation for a var or special form given its name
nil
cljs.user=> (find-ns 'lumo.repl)
#object[cljs.core.Namespace]
cljs.user=> (doc doc)
WARNING: Use of undeclared Var cljs.user/doc at line 1
WARNING: Use of undeclared Var cljs.user/doc at line 1
TypeError: Cannot read property 'call' of undefined
This ticket (which makes recur to protocol method heads work like Clojure where you omit the target object) is fairly challenging IMHO, in terms of not breaking existing code and covering the right cases. Worth testing and/or providing feedback. https://dev.clojure.org/jira/browse/CLJS-2085
@mfikes cursory reading looks good and the patch does what I would expect, will do a deeper read later, thanks!
Ahh. No, surprisingly. It does in Planck where it should
WARNING: Ignoring target object "this" passed in recur to protocol method head at line 88 /Users/mfikes/Projects/planck/planck-cljs/src/planck/core.cljs
WARNING: Ignoring target object "this" passed in recur to protocol method head at line 77 /Users/mfikes/Projects/planck/planck-cljs/src/planck/core.cljs
Here is the Planck code the change accepts, but warns on: https://github.com/mfikes/planck/blob/master/planck-cljs/src/planck/core.cljs#L88