Opened 7 years ago
Closed 19 months ago
#7066 closed defect (fixed)
MOHAWK: LB - Error at start of Just Grandma and Me (German)
|Reported by:||SF/mattn81||Owned by:||besentv|
ScummVM 1.7.0 & 1.8.1pre12
When starting the German windows version of "Grandma and Me" from the living books 1 bundle version 2 I get the following error:
ERROR: INIFile::loadFromStream: Invalid character 'ö' occured in the section name in line 63!
When I start the English version and choose the German language from the menu the whole book is working perfectly from start to end. It seems as if only the German menu is the problem.
Ticket imported from: bugs/7066.
Change History (6)
comment:1 by , 4 years ago
|Summary:||Error at start of Grandma and Me windows German → LB: Error at start of Just Grandma and Me (German)|
comment:2 by , 3 years ago
Adding a duplicate for sake of completeness #10949
MOHAWK: Just Grandma and Me - only English starts, German and French doesn't
OS: all Windows
ScummVM Version: all (2.1.0git-5236 and all older)
Game: Just Grandma and Me (CD Version)
Reproducibility: 100% on different Windows OS (WinXP,7,8.1/10) german/english
Starting the game in german ends with:
ERROR: INIFile::LoadFrameStream: Invalid character 'ö' occured in section name in line 63!
Starting the game in french ends with:
ERROR: INIFile::LoadFrameStream: Invalid character 'ç' occured in section name in line 63!
Line 63 of german file "JGMD.LB" is [Französisch.Play]
Line 63 of french file "JGMF.LB" is [Français.Play]
The problem seems to be caused by the special characters "ö" (german) and "ç" (french) which are used in the corresponding files.
comment:3 by , 3 years ago
|Summary:||LB: Error at start of Just Grandma and Me (German) → MOHAWK: LB - Error at start of Just Grandma and Me (German)|
comment:4 by , 2 years ago
The game works in German and French when you change 2 lines in /common/ini-file.cpp:
while (*p && (isAlnum(*p) || *p == '-' || *p == '_' || *p == '.' || *p == ' ' || *p == '\xF6' || *p == '\xE7'))
while (*p && ((_allowNonEnglishCharacters && *p != ']') || isAlnum(*p) || *p == '-' || *p == '_' || *p == '.' || *p == ' ' || *p == '\xF6' || *p == '\xE7'))
Here is an image of my custom build: (Not sure if it breaks sth else)
comment:5 by , 2 years ago
Could you add a pull request of your changes to https://github.com/scummvm/scummvm/pulls
That way devs are able to review it and give feedback.
Bug reports are great, but tend to not get the love they deserve
comment:6 by , 19 months ago
|Status:||new → closed|
This has been fixed a while ago in this commit.
Thanks for your report!
This happens because our INI-parser (
common/ini-file.cpp) currently doesn't allow non-alphanumeric characters in section names.
As an aside, the 2.0 version is still considered unstable and thus unsupported.