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

Reply via email to