From 747bc75bc724f6055c4140403eb60613777e245e Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Thu, 3 Nov 2011 22:40:28 +0200
Subject: SCI: Fixed typo in colorIsFromMacClut()

---
 engines/sci/graphics/palette.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'engines/sci/graphics/palette.cpp')

diff --git a/engines/sci/graphics/palette.cpp b/engines/sci/graphics/palette.cpp
index b52af38675..a7a5dc94d1 100644
--- a/engines/sci/graphics/palette.cpp
+++ b/engines/sci/graphics/palette.cpp
@@ -966,7 +966,7 @@ void GfxPalette::loadMacIconBarPalette() {
 }
 
 bool GfxPalette::colorIsFromMacClut(byte index) {
-	return index != 0 && _macClut && (_macClut[index * 3] != 0 || _macClut[index * 3 + 1] != 0 || _macClut[index * 3 + 1] != 0);
+	return index != 0 && _macClut && (_macClut[index * 3] != 0 || _macClut[index * 3 + 1] != 0 || _macClut[index * 3 + 2] != 0);
 }
 
 #ifdef ENABLE_SCI32
-- 
cgit v1.2.3


From 5fc9831577858a15ee323b41a1b9e6d01c444118 Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Thu, 17 Nov 2011 11:00:09 +0200
Subject: SCI: Fixed bug #3439240 - "QFG1VGA:  Game Crashes While Sleeping at
 Erana's Peace"

---
 engines/sci/graphics/palette.cpp | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'engines/sci/graphics/palette.cpp')

diff --git a/engines/sci/graphics/palette.cpp b/engines/sci/graphics/palette.cpp
index a7a5dc94d1..b8a4248c93 100644
--- a/engines/sci/graphics/palette.cpp
+++ b/engines/sci/graphics/palette.cpp
@@ -709,6 +709,11 @@ bool GfxPalette::palVaryLoadTargetPalette(GuiResourceId resourceId) {
 }
 
 void GfxPalette::palVaryInstallTimer() {
+	// Remove any possible leftover palVary timer callbacks.
+	// This happens for example in QFG1VGA, when sleeping at Erana's place
+	// (bug #3439240).
+	palVaryRemoveTimer();
+
 	int16 ticks = _palVaryTicks > 0 ? _palVaryTicks : 1;
 	// Call signal increase every [ticks]
 	g_sci->getTimerManager()->installTimerProc(&palVaryCallback, 1000000 / 60 * ticks, this, "sciPalette");
-- 
cgit v1.2.3


From ae54885379a040321ac8576be8db7848078ba616 Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Thu, 17 Nov 2011 11:16:10 +0200
Subject: SCI: Updated information regarding bug #3439240

---
 engines/sci/graphics/palette.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'engines/sci/graphics/palette.cpp')

diff --git a/engines/sci/graphics/palette.cpp b/engines/sci/graphics/palette.cpp
index b8a4248c93..47d1647c6c 100644
--- a/engines/sci/graphics/palette.cpp
+++ b/engines/sci/graphics/palette.cpp
@@ -711,7 +711,9 @@ bool GfxPalette::palVaryLoadTargetPalette(GuiResourceId resourceId) {
 void GfxPalette::palVaryInstallTimer() {
 	// Remove any possible leftover palVary timer callbacks.
 	// This happens for example in QFG1VGA, when sleeping at Erana's place
-	// (bug #3439240).
+	// (bug #3439240) - the nighttime to daytime effect clashes with the
+	// scene transition effect, as we load scene images too quickly for
+	// the SCI scripts in that case (also refer to kernelPalVaryInit).
 	palVaryRemoveTimer();
 
 	int16 ticks = _palVaryTicks > 0 ? _palVaryTicks : 1;
-- 
cgit v1.2.3