[CLJS-211] apply does not work on IFn objects in CLJS Created: 27/Apr/12 Updated: 28/Apr/12 Resolved: 28/Apr/12
|Patch:||Code and Test|
http://dev.clojure.org/jira/browse/CLJS-135 fixed this for multimethods, but the same issue applies to anyone who implements IFn.
The attached patch fixes this by modifying extend-type to emit an apply method on the extended object's prototype.
|Comment by Brandon Bloom [ 28/Apr/12 2:31 AM ]|
Updated with patch!
Worth noting: String has an implementation of IFn and manually defines prototype.apply. I contemplated deleting it, since this makes it unnecessary, however, my general purpose implementation is probably a tad slower. Maybe, maybe not. Depends on how Google's inlining does. I figured: make it work, then make it fast. So I didn't bother profiling them and left the existing implementation there.
|Comment by David Nolen [ 28/Apr/12 12:50 PM ]|