Proposal: Add NFData instances for strict and lazy ByteStrings
I noticed strict and lazy
ByteStrings don't have
NFData instances. I proposed 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.
We had 8 +1s and no objections.