Visitar URL original
No long in C-FFI by phadej · Pull Request #213 · haskell-unordered-containers/hashable · GitHub
Skip to content

No long in C-FFI - #213

Merged
phadej merged 3 commits into
masterfrom
no-long
Aug 24, 2021
Merged

phadej merged 3 commits into
masterfrom
no-long

Conversation

@phadej

@phadej phadej commented Aug 23, 2021

Copy link
Copy Markdown
Contributor

@mtolly, thanks for spotting the problem and proposing the fix. I cleaned up it a bit more, could you test that I didn't undo anything on Windows (and that tests pass, the new one in particular)

@phadej
phadej force-pushed the no-long branch 4 times, most recently from dc00264 to 239e138 Compare August 23, 2021 09:17
@phadej phadej changed the title No long No long in C-FFI Aug 23, 2021
@mtolly

mtolly commented Aug 24, 2021

Copy link
Copy Markdown
Contributor

Looks good to me 👍 Test passes and the clash I observed is still fixed.

@phadej
phadej merged commit 3e817eb into master Aug 24, 2021
@phadej
phadej deleted the no-long branch August 24, 2021 06:54
@phadej

phadej commented Aug 24, 2021

Copy link
Copy Markdown
Contributor Author

@mtolly, thanks for checking. I made a release https://hackage.haskell.org/package/hashable-1.3.3.0 with this patch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants