Fork me on GitHub
#cljs-dev
<
2016-03-02
>
mfikes13:03:11

I’m curious if supporting IPrintWithWriter for functions simply involves an additional test for satisfies? at the scope where we know it is a function, or if there are more subtle issues at play (in which case, since it is a very low priority, won’t burden anyone with it). This appears to work fine: https://github.com/mfikes/clojurescript/commit/035c0418569eadf7adb4930e721f6ea1a6af4e07

mfikes14:03:31

I don’t appreciate the subtleties. For example, this second check and the lack of ^not-native is something I couldn’t explain and justify if asked: https://github.com/clojure/clojurescript/blob/f58dcdf4dc37ef52d4eb1e2b7c994282bf6351f5/src/main/cljs/cljs/core.cljs#L9096

thheller16:03:25

looks like a mistake the IPrintWithWriter is checked twice