Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for text 2.0 #46

Closed
wants to merge 2 commits into from
Closed

Conversation

RikvanToor
Copy link
Contributor

This PR adds support for this library to be used with text versions 2.0 and up. From 2.0 on, text uses UTF-8 encoding internally, where it used to rely UTF-16. This broke the escapeText function, as this directly accesses the internal text array. I've fixed this to work for UTF-8 if GHC detects text >= 2.0.

Additionally, the version constraints for for memory and tls have been loosened a bit.

@albertov
Copy link

This PR fixes a segmentation fault in our application. Thanks!

@jappeace
Copy link
Collaborator

jappeace commented Aug 14, 2023

this has been merged here: #54
the PR was conflicting on cabal.

I'll make another release,
thank you!

@jappeace jappeace closed this Aug 14, 2023
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.

3 participants