Opened 13 years ago
Closed 8 years ago
#5674 closed defect (fixed)
AGI: SQ2: Gfx glitch at intro logo "Chapter 2"
| Reported by: | SF/mokeiichi | Owned by: | m-kiewitz |
|---|---|---|---|
| Priority: | high | Component: | Engine: AGI |
| Version: | Keywords: | ||
| Cc: | Game: | Space Quest 2 |
Description
Sorry if this is a duplicate, I remember having seen this bug reported before, but couldn't find it.
In the intro of the game "Chapter II" looks wrong. See attached screenshots. When those long lines aren't there yet, they also shouldn't be visible behind the text. When they are displayed, they also should behind the text. Right now it's (almost) the opposite.
Ticket imported from: #3292787. Ticket imported from: bugs/5674.
Attachments (2)
Change History (12)
by , 13 years ago
| Attachment: | sq2-scummvm01.png added |
|---|
comment:2 by , 13 years ago
Replicated this with the latest Git Master on Linux x86_32 i.e. ScummVM 1.4.0git2461-ga6b7968-dirty (Sep 25 2011 13:22:57) Features compiled in: Vorbis FLAC MP3 ALSA SEQ TiMidity RGB zLib FluidSynth Theora AAC
Checked with original interpreter in DOSBox and confirmed as per submitter report i.e. The original interpreter behaviour is different to ScummVM AGI
comment:3 by , 13 years ago
This bug is nice to get fixed before the release. Raising priority for keeping the track.
comment:4 by , 13 years ago
| Priority: | normal → high |
|---|
comment:5 by , 11 years ago
NAGI seems to handle this correctly, but I don't know enough about AGI to figure out what they do differently.
comment:6 by , 11 years ago
At both points (lines appearing too soon and later disappearing too soon) it seems that the text sprite being drawn blits a newly rendered background picture too soon.
As far as I can tell this happens due to a scheduleUpdate being called on the text sprites while pictureShown is true, but the corresponding doUpdate only being called after pictureShown has been set to false by rendering the new pic.
This all happens within the cmdDrawPic sequence:
state->_vm->_sprites->eraseBoth(); state->_vm->_picture->decodePicture(_v[p0], true); state->_vm->_sprites->blitBoth(); state->_vm->_sprites->commitBoth(); state->pictureShown = 0;
Unsure how to fix. Maybe set pictureShown to false immediately after decodePicture here in cmdDrawPic?
comment:7 by , 8 years ago
| Owner: | set to |
|---|
comment:8 by , 8 years ago
this bug seems to be solved in my current work-in-progress graphics rewrite. Marking for later.
comment:10 by , 8 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
GFX Glitch in SQ2 - 01