Opened 12 years ago
Closed 12 years ago
#5800 closed defect (fixed)
SCI: QFG1EGA inventory items truncated
|Reported by:||SF/blazingstix||Owned by:||wjp|
|Cc:||Game:||Quest for Glory 1|
in scummvm version 1.4.0git1074-g5a1d0ab, (win32 daily build jul 16, 2011), when playing quest for glory 1 ega v1.200 (the one included in the collection/anthology cd) inventory items with "of" in their name get truncated on the inventory display screen. for example: Piece of Paper String of Pearls Flask of Water all show as: Piece String Flask
Ticket imported from: #3368821. Ticket imported from: bugs/5800.
Change History (12)
comment:1 by , 12 years ago
|Summary:||QFG1EGA inventory items truncated → SCI: QFG1EGA inventory items truncated|
comment:2 by , 12 years ago
by , 12 years ago
saved game with problem
comment:3 by , 12 years ago
the problem also exists in the latest stable release (v1.3.1).
I've uploaded a saved game. once it's loaded, you can press TAB to see the inventory screen.
comment:4 by , 12 years ago
Thanks you for this help.
I'll try and see whether this is a regression from an earlier release i.e. v1.2.0, v1.2.1, v1.3.0 and then bisect with Git to locate.
Though this was not indicated during http://wiki.scummvm.org/index.php/SCI/Testing as of SVN r51733, so this is likely..
comment:5 by , 12 years ago
Hmm.. v1.3.0 shows this issue with the attached savegame, but the v1.2.* can't load this as the savegame format has changed.
blazingstix: Do you have / Could you use, v1.2.1 to make another savegame demonstrating this issue, since this would help.
comment:6 by , 12 years ago
Tried ./scummvm --debugflags=OnStartup, Strings to look at this. This shows that the string in question is "Formatting "Flask%c of Water", but is being displayed as Flask. This code is in scummvm/engines/sci/engine/kstring.cpp within the kFormat function. It is likely that the expected parameter is being read as NULL, resulting in the string being terminated.
One of the SCI devs will need to confirm what exactly is going on here. One further piece of work would be to check the original interpreter behaviour under DOSBox.
comment:7 by , 12 years ago
Behaviour can be quickly reproduced in game. After starting a new game, immediately head to the tavern (1 screen up, 1 screen left, enter tavern) and "pick up paper" in front of middle stool.
I can also confirm that original interpreter does not truncate items with "of" in them.
comment:8 by , 12 years ago
blazingstix: Thanks. I can confirm as well that this bug is present in v1.2.0 and v1.2.1 i.e. since introduction of SCI engine.
It looks like the parameter to kFormat is 0/NULL so the string is truncated. It will need a SCI engine developer to disassemble the script code and work out why we have a difference here, but we have sufficient information to progress this bug. Please standby and thanks.
comment:9 by , 12 years ago
comment:10 by , 12 years ago
Fixed in 85a056f382. Thanks for the report!
comment:11 by , 12 years ago
|Status:||new → closed|
blazingstix: Please can you attach a savegame which demonstrates this issue? Also, can you confirm if this occurs in the current stable release i.e. v1.3.1 ?