Proposal: Add NFData instances for strict and lazy ByteStrings

26 Oct 2011

      While I was hunting for a space leak in the new hackage-server (still
haven't found it) I noticed strict and lazy ByteStrings don't have
NFData instances. I propose to add them because it enables users to
force ByteStrings in the principled way (using deepseq) so they don't
need to use tricks like: B.length b `seq` ...

instance NFData Strict.ByteString

instance NFData Lazy.ByteString where
    rnf Empty       = ()
    rnf (Chunk _ b) = rnf b

Note that the containers dependency of deepseq was recently dropped so
depending on that package is not a big issue anymore.

Discussion deadline: 2 weeks from now (9 November)

Bas

Bas van Dijk

Bas van Dijk

Johan Tibell

Felipe Almeida Lessa

Ivan Lazar Miljenovic

Edward Kmett

Gregory Collins

Edward Z. Yang

Vincent Hanquez

Herbert Valerio Riedel

Bas van Dijk

tags

participants (9)