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

[py] Fix #8197 #8340

Merged
merged 1 commit into from May 28, 2019
Merged

[py] Fix #8197 #8340

merged 1 commit into from May 28, 2019

Conversation

Aurel300
Copy link
Member

@Aurel300 Aurel300 commented May 27, 2019

Prior to this PR, when using Sys.stdout().writeString(...) with a Unicode string:

  • string is encoded into a UTF-8 byte array
  • the byte array is iterated byte by byte
  • each byte is written separately via String.fromCharCode
  • any byte over 127 is re-encoded by Python into UTF-8 (so 0x80 turns into the incorrect 0xC2 0x80)

Test for this is part of #8135 .

@Aurel300 Aurel300 requested a review from frabbit May 27, 2019 16:56
@RealyUniqueName RealyUniqueName added this to the Release 4.0 milestone May 27, 2019
@skial skial mentioned this pull request May 28, 2019
1 task
@Simn Simn merged commit 96cacb8 into HaxeFoundation:development May 28, 2019
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

3 participants