search

bennofs / th-lift-instances

Lift instances for common haskell data types
http://hackage.haskell.org/package/th-lift-instances
Other
12 stars 11 forks source link

ByteString lift instance should use mkBytes #23

Open mpickering opened 4 years ago

mpickering commented 4 years ago

The Lift instance for ByteString is very inefficient. It should use the mkBytes combinator which was designed for use in file-embed.

https://hackage.haskell.org/package/template-haskell-2.16.0.0/docs/Language-Haskell-TH-Lib.html#v:mkBytes