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

Encode strings as UTF8 when sending them to Lua, to match the string decoding. #202

Conversation

DaleStan
Copy link
Collaborator

Lua uses UTF8 strings, not Ansi. This usually doesn't matter, except that Schall Alien Loot has setting values containing the character × (U+00D7), and this gets garbled when loaded from mod-settings.dat.

In addition to Schall, I checked loading with AngelBobMadClown, ExoticIndustries, IR3, Nullius, Pyanodon, and SE. The only error I encountered was #105.

@DaleStan DaleStan force-pushed the 200-schallalienloot-has-a-richness-error-when-loaded branch from 59f14b0 to 1e2e463 Compare July 23, 2024 03:03
@DaleStan DaleStan linked an issue Jul 23, 2024 that may be closed by this pull request
Copy link
Owner

@shpaass shpaass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The VS inspection previously suggested to change ANSI to Unicode, which broke the app. Thank you for fixing it properly.

@shpaass shpaass merged commit c0e4ce5 into shpaass:master Jul 23, 2024
1 check passed
@DaleStan DaleStan deleted the 200-schallalienloot-has-a-richness-error-when-loaded branch July 23, 2024 07:17
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.

SchallAlienLoot has a richness error when loaded
2 participants