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

(find-ns 'cljs.core) causes all further macro resolution (for cljs.core) to fail

https://dev.clojure.org/jira/browse/CLJS-2088

@cgrand thanks for the patch, applying now

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!

Thanks. That change is a bit "scarier" than the usual, so no need to rush it.

@mfikes out of curiosity does this trigger any warnings in ClojureScript itself?

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

k

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

@mfikes your fix looked fine to me, merged