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

Fixed NULL character bug when writing generated TypeScript code onto files using the utf16le encoding #114

Merged
merged 4 commits into from
May 20, 2022

Conversation

Luna-Klatzer
Copy link
Member

What type of change does this PR perform?

  • Bug fix (Non-breaking change which fixes an issue)

Summary

Fixed NULL character issue when writing generated code onto files using the utf16le encoding.

Changes

  • Fixed utf16le file writing issue in @kipper/cli.

Does this PR create new warnings?

No.

Changelog

Changed

  • Fixed NULL character issue when writing generated code onto files using the utf16le encoding. From now on a buffer
    will be created using the proper encoding (also for ascii and utf8) that should be properly writable to a file.

Linked other issues or PRs

No linked issues.

@Luna-Klatzer Luna-Klatzer added bug Bug or issue in the language or API fix Fixes an issue or bug labels May 20, 2022
@Luna-Klatzer Luna-Klatzer self-assigned this May 20, 2022
@Luna-Klatzer Luna-Klatzer changed the title Fixed NULL character issue when writing generated code onto files using the utf16le encoding Fixed NULL character bug when writing generated TypeScript code onto files using the utf16le encoding May 20, 2022
@Luna-Klatzer Luna-Klatzer merged commit 0b97cd8 into dev May 20, 2022
@Luna-Klatzer Luna-Klatzer deleted the 114-fix-wrong-utf16-write branch May 20, 2022 21:16
@Luna-Klatzer Luna-Klatzer added this to the Stable Kipper Release milestone May 20, 2022
This was referenced May 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug or issue in the language or API fix Fixes an issue or bug
Projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant