To facilitate inlining, the patch calls out to a separate larger method which handles a group of cases.
+ if(o.getClass().getName().startsWith("java.util.")) + return doalienhasheq(o); + return o.hashCode(); I was wondering whether an efficient improvement is possible that would support things like Guava ImmutableList. In particular, I was wonder which "default" cases are currently handled by the return o.hashCode() above. Replacing the three lines above with + return doalienhasheq(o); would allow the patch to also handle non-java.util collection implementations, but push the "default" cases down into the bottom of that method. On Tuesday, May 13, 2014 12:38:54 AM UTC-4, Michał Marczyk wrote: > > I've posted a patch that makes java.util.{List,Map,Map.Entry,Set} > hashes consistent with those of appropriate Clojure collection types > on the ticket. > -- 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 --- You received this message because you are subscribed to the Google Groups "Clojure" group. To unsubscribe from this group and stop receiving emails from it, send an email to clojure+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.