Skip to content
This repository has been archived by the owner on Jul 28, 2022. It is now read-only.

GHC 7.10 fixes #9

Closed
wants to merge 2 commits into from
Closed

GHC 7.10 fixes #9

wants to merge 2 commits into from

Conversation

bgamari
Copy link
Contributor

@bgamari bgamari commented Feb 17, 2015

No description provided.

Under the Foldable-Traversable Proposal elem is now polymorphic over the
container. Disambiguate a few uses.
@bgamari
Copy link
Contributor Author

bgamari commented Feb 17, 2015

It appears that this is already fixed in master but has yet to be pushed to Hackage. This might be a good time to do this.

@bgamari bgamari closed this Feb 17, 2015
@jgm
Copy link
Owner

jgm commented Feb 17, 2015

Will do.

+++ Ben Gamari [Feb 16 15 20:08 ]:

It appears that this is already fixed in master but has yet to be pushed to Hackage. This might be a good time to do this.


Reply to this email directly or view it on GitHub:
#9 (comment)

@bgamari
Copy link
Contributor Author

bgamari commented Feb 17, 2015

@jgm Sure, it would probably be best to wait until the FTP is actually agreed upon (although from what I've heard it seems fairly likely that this will be the case).

The issue here is the use of OverloadedStrings. I agree that this isn't great but in my experience the pain of importing Foldable et al is felt more often than OverloadedStrings is used, so I believe that the trade-off made by FTP is the right one.

What does bother me about the FTP is the fact that the monomorphic forms offered by Data.List vanish, but it seems that little can be done about this.

@jgm
Copy link
Owner

jgm commented Feb 17, 2015

+++ Ben Gamari [Feb 17 15 06:45 ]:

@jgm Sure, it would probably be best to wait until the FTP is actually agreed upon (although from what I've heard it seems fairly likely that this will be the case).

The issue here is the use of OverloadedStrings. I agree that this isn't great but in my experience the pain of importing Foldable et al is felt more often than OverloadedStrings is used, so I believe that the trade-off made by FTP is the right one.

Ah yes, I'd forgotten I was using that. Well, good. Using an explicit
list of Char ['X', 'Y', 'Z'] would be another alternative, then.

Anyway...you indicated that the issue is already resolved by an
existing commit. Let me know if that's not the case. I released
the latest on Hackage last night.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants