Use array-maps explicitly for serializer collections? (patch attached)

revelytix / carbonite

Clojure library for serializing Clojure data using Kryo

Apache License 2.0

59 stars 29 forks source link

Use array-maps explicitly for serializer collections? (patch attached) #2

Closed sritchie closed 12 years ago

sritchie commented 12 years ago

Patch here: https://gist.github.com/1434696

This patch converts the {} reader macros on clojure-primitives and java-primitives to use array-map explicitly. I think Clojure uses array-maps by default for small maps; I'm not sure, though, and if {} created a hash-map with different ordering when populating two different instances of kryo, deserialization will fail.

puredanger commented 12 years ago

Thanks, applied patch.