On 1 July 2012 16:25, David Nolen <dnolen.li...@gmail.com> wrote:
> On Sun, Jul 1, 2012 at 9:04 AM, Vinzent <ru.vinz...@gmail.com> wrote:
>> Extend on nil, IIndexed and default (I realize that it's currently
>> impossible in cljs).
>
> IIndexed is the protocol that maps to nth. We already extend IIndexed
> to nil. Extending default to IIndexed leads to horrible things like
> the following:
>
> (indexed? 1) ; => true

Ah right, I missed nil in the above. extend-type default was removed,
in any case, for IIndexed, see CLJS-234 and commit fda0500ecda.

Since my phrasing in my next-to-last message now seems unfortunate to
me and I never delivered on the "see below" promise of the third
sentence: the perf improvements around that time had to do with
satisfies? (which got much faster thanks to a great idea of David's);
that in turn made it possible to back out of some unfortunate default
implementations of protocols while improving the performance of some
of the basic seq / coll functions quite significantly.

M.

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

Reply via email to