From 85b44876ea581170e13fb978ac5935d99271eec7 Mon Sep 17 00:00:00 2001
From: Filippos Karapetis
Date: Fri, 20 Nov 2009 16:48:51 +0000
Subject: Added a FIXME for the syncTime and syncCue selectors, for games which
 don't have a default selector dictionary

svn-id: r46013
---
 engines/sci/engine/static_selectors.cpp | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

(limited to 'engines/sci')

diff --git a/engines/sci/engine/static_selectors.cpp b/engines/sci/engine/static_selectors.cpp
index 3252106e70..c047bdd3b0 100644
--- a/engines/sci/engine/static_selectors.cpp
+++ b/engines/sci/engine/static_selectors.cpp
@@ -70,7 +70,11 @@ static const SelectorRemap sci0SelectorRemap[] = {
 // Taken from Leisure Suit Larry 1 VGA (Full Game)
 static const SelectorRemap sci1SelectorRemap[] = {
 	{      "nodePtr",  44 }, {   "cantBeHere",  57 }, {    "topString", 101 },
-	{        "flags", 102 }, {     "syncTime", 247 }, {      "syncCue", 248 },
+	{        "flags", 102 },
+	// FIXME: These two selectors differ for each game. We need to find a reliable
+	// way to detect them, or apply them on a per-game basis for games which are
+	// missing them
+	{     "syncTime", 247 }, {      "syncCue", 248 },
 	{              0,   0 }
 };
 
@@ -78,7 +82,11 @@ static const SelectorRemap sci1SelectorRemap[] = {
 static const SelectorRemap sci11SelectorRemap[] = {
 	{      "nodePtr",  41 }, {   "cantBeHere",  54 }, {     "topString", 98 },
 	{        "flags",  99 }, {       "scaleX", 104 }, {       "scaleY", 105 },
-	{     "syncTime", 279 }, {      "syncCue", 280 }, {              0,   0 }
+	// FIXME: These two selectors differ for each game. We need to find a reliable
+	// way to detect them, or apply them on a per-game basis for games which are
+	// missing them
+	{     "syncTime", 279 }, {      "syncCue", 280 },
+	{              0,   0 }
 };
 
 Common::StringList Kernel::checkStaticSelectorNames() {
-- 
cgit v1.2.3