#8625 closed patch
Scumm: Fix charset glitches in original Save-dialog
| Reported by: | SF/tobigun | Owned by: | Kirben |
|---|---|---|---|
| Priority: | normal | Component: | Engine: SCUMM |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
The editing area of the original Save-dialog (Alt-F5) in the savegame name list is not drawn correctly. It reveals some graphical glitches which this patch fixes.
One reason for these bugs is the trimming of strings done in ScummEngine::drawString() which also trims spaces you enter when editing a savegame name. Another cause is the buggy handling of empty strings ("") which are replaced by a string with one space (" "). Thats the reason why there is always shown a " " when you click on an empty slot.
This patch also works with the Grail Diary in Indy3 but I have not tested it with the MAC version.
Ticket imported from: #1687053. Ticket imported from: patches/730.
Attachments (1)
Change History (7)
by , 17 years ago
| Attachment: | dialog_charset_glitch.patch added |
|---|
comment:1 by , 17 years ago
| Owner: | set to |
|---|---|
| Status: | new → closed |
comment:2 by , 17 years ago
comment:3 by , 17 years ago
Kirben took your patch as motivation to recheck assembly and made some changes to string.cpp accordingly. This the patch doesn't apply anymore... Not sure whether there is anything left in this patch which is not covered by Kirben's changes... Kirben?
comment:4 by , 17 years ago
| Status: | closed → new |
|---|
comment:5 by , 17 years ago
| Status: | new → closed |
|---|
comment:6 by , 5 years ago
| Component: | → Engine: SCUMM |
|---|
I checked the disassembly of original games, and the trimming of excess spaces only applies to SCUMM4 and later games. So I adjusted the code in ScummVM to limit that section to SCUMM4 and later games, in order to fix those charset glitches.
I committed the other section of the patch to ScummVM SVN.