Menu
▾
▴
#4774 MADE: Screen effects sometimes look wrong/bad
closed-fixed
7
2011-09-26
2010-02-23
No
Current ScummVM SVN
I've verified it with the English version of Return to Zork, but I guess any supported version will do
The screen effects in the MADE engine were modified to run on low-end hardware. However, the result doesn't always look as good as the original behaviour. Here's a slow-motion comparision I made:
http://www.update.uu.se/~d91tan/ScummVM/rtz.html
It would be nice if we could both look good on "normal" hardware, while still playing nice on low-end hardware. Possibly by introducing a compile- or runtime setting.
This was discussed on scummvm-devel recently, but nothing was decided. The discussion can be read here:
Raising priority to "nice to have fixed for 1.2.0".
I'm doing this because to me the bug is pretty darn noticeable, and we already have the correct behavior implemented. It just happened to be too slow on some low-end platforms.
Patch for screen effects
I've added a patch with a possible fix. It does what the fade in/out effect already did, that is it does reduce the number of palette updates. It looks pretty close to the original but needs fewer palette updates.
I didn't commit it right away since I wanted to hear from porters of "slow" platforms first if the effects are fast enough or slow again.
Any comments are welcome.
It looks promising, but I only have a fairly fast desktop computer to test it on.
One thing I noticed is that the patch doesn't seem to do anything about vfx09(). But even using the supposedly original behaviour for it doesn't seem to match what I see in DOSBox. The colours in ScummVM seem a bit more... I don't know, random. (Also there seem to be some missing effects in ScummVM. E.g. when arriving at the lighthouse door, there's a transition both at the beginning and the end of the brief cutscene movie. In ScummVM, there's only one at the end. This may be getting off-topic though.)
This bug is nice to get fixed before the release. Raising priority for keeping the track.
Applied patch in r8c431e1
After talking with erikborbjorn, it seems that the bug is fixed now after applying johndoe's patch. Closing