-
Notifications
You must be signed in to change notification settings - Fork 157
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 pure-Haskell replacement for is_ascii.c #545
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the impacted exported functions are decodeLatin1
and isAscii
. [edit] And decodeASCIIPrefix
, which depends on asciiPrefixLength
Nothing started about |
0f386b8
to
da41a9e
Compare
Plot twist: I had some extra time today. |
Oops, bad rebase. Will fix tomorrow. |
da41a9e
to
537faed
Compare
This is ready for review now. |
537faed
to
5a2f7e5
Compare
5a2f7e5
to
c04d31a
Compare
Thanks for doing this @chreekat! |
Thanks! |
wip. Will add some comments inline.