-
Notifications
You must be signed in to change notification settings - Fork 4
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
ScummTR: V1EN games use a \x85 character that's only available in Windows-1252 #5
Comments
Code pages actually start to differ after 127 ;) Looks like I wrote that as if my local was universal, most probably out of ignorance. This array is used in the If you replace You could make scummtr work with unicode, but it is a little more work. |
Oh, OK, thank you! (EDIT: my original comment got a lot of things wrong 😅) The problem with Windows-1252 is that it's incompatible with ISO-8859-1 for some codepoints: The ellipsis is nice, but ISO-8859-1 doesn't have one. I think that, nowadays, restricting ourselves to the ISO-8859-1/Windows-1252 compatible subset is probably the safest, most interoperable choice. I'll test with a V1-EN game, since this array is only for V1-EN games, anyway. Maybe I could borrow the As for Unicode, I could maybe use |
It's an incompatibility between Windows-1252 and ISO-8859-1, and it looks like no V1-EN game uses it anyway, to the best of my knowledge. It's still possible to use the associated escape sequence, if necessary. Issue #5.
There's a
\x85
character in theText::CT_V1EN
array of ScummTR.scummtr/src/ScummTr/text.cpp
Line 50 in 5d30bfc
This file contains escape sequences for some ISO-8859-1 characters, such as
\xa3
for a©
symbol (original file was in Windows-1252, but I used escape sequences while converting it to UTF-8; it's also better for compiler portability).The problem is that the
\x85
value only makes sense in ISO-8859-1, not in Windows-1252. And nowadays, it's probably better to limit ourself to the identical subset between ISO-8859-1/Windows-1252 (this character isn't part of it).The text was updated successfully, but these errors were encountered: