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

Count surrogate pairs. #2090

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kikaragyozov
Copy link

@kikaragyozov kikaragyozov commented Nov 23, 2022

Fixes #2088.

Caution: I haven't tested the written code, nor have I written tests for it. I just wanted to implement something in a bare-bone fashion so that it can be inspected and commented on.

If this initial draft looks good, I can try and make some tests in the weekends. If there are any instructions/conventions I must follow for the unit tests, let me know!

@JoshClose
Copy link
Owner

I believe this is the same thing as this pull request. #2106

I'm going to wait until after the SIMD work.

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.

ByteCount fails to count surrogate characters properly.
2 participants