Opened 7 years ago

Closed 7 years ago

#9674 closed defect (fixed)

SCI: QFG3: Broken day/night cycle stuck on daylight

Reported by: salty-horse Owned by: wjp
Priority: normal Component: Engine: SCI
Version: Keywords:
Cc: Game: Quest for Glory 3


At one point in the game, it shifts to day and never changes back to night.

The bug was originally mentioned in a podcast - I did not see it myself. I have since obtained savegames showing the problem (attached). It was also mentioned on the forums in 2014

I am not sure if this is a bug in the original that is exposed by ScummVM, or a ScummVM bug.

Attachments (1) (113.1 KB ) - added by salty-horse 7 years ago.
Savegames showing the problem

Download all attachments as: .zip

Change History (4)

by salty-horse, 7 years ago

Attachment: added

Savegames showing the problem

comment:1 by wjp, 7 years ago

The reason the palette is not changing in this savegame is that _palVaryPaused somehow seems to be equal to 3 (instead of 0), so the night-time palette change is stuck in a paused state.

I'm unsure how this happened, but one thing that seems fishy is that restoring doesn't reset the palVary state, unless the game that's being loaded has _palVaryResourceId != -1

comment:2 by wjp, 7 years ago

Here's one way of reproducing the problem:

When going to the bazaar during twilight, the function bazaarR::init (script 51) pauses the palvary. If you then use the ScummVM menu at Ctrl-F5 to load a game that does not have an active palvary, _palVaryPaused stays stuck at 1. This should be repeatable to get it to 3.

The QfG3 scripts seem to always try to ensure to that palvary is unpaused when saving or loading. But using the Ctrl-F5 menu to load bypasses this.

Random note, unrelated to this bug: if the twilight effect is (normally) paused while visiting the bazaar, entering and closing (scripted) dialogs (such as the game's load/save/settings menu) will un-pause the twilight effect. This happens in SSCI too. (But fixing that script issue in SSCI might not have been straightforward due to the interaction with saving/loading at twilight.)

comment:3 by csnover, 7 years ago

Owner: set to wjp
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.