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

fix broken tables caused by incorrectly calculated unicode string length #56

Closed
wants to merge 1 commit into from

Conversation

tyrasd
Copy link

@tyrasd tyrasd commented Dec 29, 2014

Counting the number of characters in unicode strings is hard. This should fix problems with astral symbols and combining marks (e.g. diacritics).

@GaryJones
Copy link

Needs a rebase to fix the merge conflicts.

@tyrasd
Copy link
Author

tyrasd commented Sep 20, 2018

I believe #97 addressed the same issue as well, doesn't it? ~~@joshbetz – feel free to close this PR if that is the case. ~~ //edit: just checked it, and yes, this fix is now superfluous

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.

None yet

2 participants