data.fressian

Add c.l.PersistentHashSet ReadHandler

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    data.fressian 0.2.0

Description

clojure.data.fressian doesn't have a ReadHandler defined for "set". This means that when you serialise and deserialise a Clojure PersistentHashSet, it returns as a java.util.HashMap. Is there a reason why we wouldn't define a ReadHandler for "set" in clojure-read-handlers?

Activity

Hide
Daniel Compton added a comment - - edited

@stu is it intentional that c.d.f doesn't have a ReadHandler for reading fressian sets into persistent sets, or vectors into persistent vectors?

Show
Daniel Compton added a comment - - edited @stu is it intentional that c.d.f doesn't have a ReadHandler for reading fressian sets into persistent sets, or vectors into persistent vectors?
Hide
Stuart Halloway added a comment -

It is intentional, but not necessarily a great idea. Just burned me today.

Show
Stuart Halloway added a comment - It is intentional, but not necessarily a great idea. Just burned me today.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: