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

Revert SmingLocale.h encoding #2862

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

mikee47
Copy link
Contributor

@mikee47 mikee47 commented Jul 15, 2024

The french and german strings defined in SmingLocale.h appear to have been corrupted.

Taking the é in février as an example:

PR #1549 Initial commit as ascii (probably Windows) e9
PR #1715 changed to UTF-8 'replacement character' ef bf bd.

The UTF-8 code would be ce a9 but as that's two bytes it could break existing code.
Best to stick with original encoding.

SmingHub#1549 Initial commit as ascii (probably Windows) e9
SmingHub#1715 replaced with UTF-8 'replacement character' ef bf bd. UTF-8 code is ce a9

Best to stick with original encoding, UTF8 is likely to cause problems with existing code
@mikee47 mikee47 changed the title Revert SmingLocale.h coding Revert SmingLocale.h encoding Jul 16, 2024
@slaff slaff added this to the 5.2.0 milestone Jul 16, 2024
@slaff slaff merged commit f56189f into SmingHub:develop Jul 16, 2024
46 checks passed
@mikee47 mikee47 deleted the fix/sminglocale-encoding branch July 17, 2024 11:23
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.

2 participants