search

brandonbloom / dispatch-map

Persistent map with Clojure's multimethod dispatch semantics
29 stars 3 forks source link

ClojureScript port #5

Open tslocke opened 9 years ago

tslocke commented 9 years ago

Thanks for dispatch-map -- I needed it in a cljs project so I ported it, including the tests. The underlying implementation works unchanged, but the deftype is quite different.

I've not included the multimethod macros, although they may well just work unchanged.

I didn't make any effort to create a combined clojure/clojurescript version (.cljc), so not sure if a pull request makes sense. Anyway it's here:

https://gist.github.com/tablatom/793ebad6fb4d5f64add9

Feel free to do what you will with it, if anything.

This one test is failing - not an issue for me so I've not investigated why.

brandonbloom commented 9 years ago

Glad it's useful for you!

Would be happy to consider a PR if you split out the code in to clj/cljc/cljs files & got the broken test figured out. Otherwise, I'll just leave this issue here in case any other enterprising individual could benefit from your port. Cheers!