Fix containers - Githubissues

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

Fix containers #29

Closed treeowl closed 2 years ago

treeowl commented 3 years ago

The containers version check was shifted off, preventing the internals-based code from ever compiling. Fix that.
Make the internals-based Data.Sequence.Seq instance compile.
Add instances for Data.Sequence.ViewL and Data.Sequence.ViewR.
Change the older-containers instances for Set, Map, and IntMap to use fromDistinctAscList to increase speed.

Closes #24

treeowl commented 3 years ago

A bit of experimentation suggests I'm probably right about the coerce implementation. The number of terms doesn't change, but the number of types and the number of coercions go down quite a bit. So that should be a win.