From 9af803f575aa63cd30e82c596e9e6cbac82613bb Mon Sep 17 00:00:00 2001 From: Martin Kiewitz Date: Tue, 26 Oct 2010 16:21:00 +0000 Subject: SCI: kAnimate bit 2 is hoyle 4 exclusive adjusted comments about that and also renamed the bit, also added additional TODO, because hoyle 4 has special code that gets into action, when bit 0 is not set, but bit 2 is svn-id: r53853 --- engines/sci/graphics/animate.cpp | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) (limited to 'engines/sci/graphics/animate.cpp') diff --git a/engines/sci/graphics/animate.cpp b/engines/sci/graphics/animate.cpp index 30329fa688..8aa6f5284f 100644 --- a/engines/sci/graphics/animate.cpp +++ b/engines/sci/graphics/animate.cpp @@ -286,16 +286,12 @@ void GfxAnimate::fill(byte &old_picNotValid) { view->getCelRect(it->loopNo, it->celNo, it->x, it->y, it->z, it->celRect); } - // This statement must be here for Hoyle4, otherwise cards are unclickable. - // This is probably one of the experimental features that were occasionally - // added to SCI interpreters; the corresponding check is absent in many SSCI - // versions. m_kiewitz knew about this flag before I (lskovlun) implemented it, - // so it is possible that more test cases are known. Also, some presently open - // SCI1.1 bugs may be fixed by this and should be re-tested with this patch generalized. - - // NOTE: *this* breaks at least eco quest 2. One should go through interpreters and check - // the code before enabling it for more games (TODO) - if ((g_sci->getGameId() == GID_HOYLE4) && (it->scaleSignal & kScaleSignalDontSetNsrect)) + // Checking for this bit must be here for Hoyle4, otherwise cards are unclickable. + // This feature is not included in the other SCI1.1 interpreters and MUST NOT be + // checked in those cases, otherwise we will break games (e.g. EcoQuest 2, room 200) + // TODO: hoyle 4 has different code inside kAnimate and even special code, when bit 0 is not + // set, but bit 2 is + if ((g_sci->getGameId() == GID_HOYLE4) && (it->scaleSignal & kScaleSignalHoyle4DontSetNsrect)) setNsRect = false; if (setNsRect) { -- cgit v1.2.3