On Feb 3, 2009, at 8:16 PM, Stephen C. Gilardi wrote: > > On Feb 3, 2009, at 10:44 PM, Jason Wolfe wrote: > >> user> (contains? (.keySet {:a :b}) :a) >> false >> >> It seems that all that's required to make "contains?" work on general >> Sets is to replace "IPersistentSet" with "Set" on lines 648 and 649 >> of >> RT.java. I can make a patch if desired. > > Along those lines, would it be a good idea for the object returned > by clojure.core/keys to implement java.util.Set and/or > clojure.lang.IPersistentSet? > > --Steve >
Yeah, that would be great. (Or, at least, a separate "keyset" function). I would ask for IPersistentSet, since java.util.Sets don't play as well with Clojure (e.g., you can't call "into" on them, and they aren't "set?"s). Thanks, Jason --~--~---------~--~----~------------~-------~--~----~ 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 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 -~----------~----~----~----~------~----~------~--~---