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

feat(scummtr): Fix text extraction resource for German releases of Indy3 (#58) #59

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

dwatteau
Copy link
Owner

This should hopefully fix the bug reported in #58

Tested against an original DOS VGA German release.

…dy3 (#58)

This is an original bug in the official German releases, some German Eszett
characters were misencoded as \255 instead of \225, triggering the "special
espace sequence" code.

The following script lines are impacted:

[023:SC#0108]^irgendwo hier im Schlo\255.
[028:SC#0071]Nicht so schnell!  Woher weiß ich, da\255 Sie kein Spion sind?

Just ignore the \255 and replace it with the inteded \225, on the fly.
This unbreaks exporting/importing the text with ScummTR.

See also the related original ScummVM bugs:

    https://bugs.scummvm.org/ticket/1675
    https://bugs.scummvm.org/ticket/2715

Thanks to @gabberhead for reporting the issue.
@dwatteau dwatteau merged commit 5f40112 into main Jul 12, 2024
4 checks passed
@dwatteau dwatteau deleted the fix/fix-indy3-original-misencoded-german-eszett branch July 12, 2024 10:38
@dwatteau
Copy link
Owner Author

Commit 977ad42 was necessary too in order to fix all instances of the bug.

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.

1 participant