Proposal: Add NFData instances for strict and lazy ByteStrings
|Reported by:||basvandijk||Owned by:|
|Type of failure:||None/Unknown||Test Case:|
|Related Tickets:||Differential Revisions:|
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.