From 5d0d67e5742d9a65ad9c944a86ff3223ff967042 Mon Sep 17 00:00:00 2001
From: Johannes Schickel
Date: Thu, 9 Jul 2009 16:23:22 +0000
Subject: Add fixme about SCI32 specific code.

svn-id: r42305
---
 engines/sci/resource.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'engines')

diff --git a/engines/sci/resource.cpp b/engines/sci/resource.cpp
index b248729503..6d33c77179 100644
--- a/engines/sci/resource.cpp
+++ b/engines/sci/resource.cpp
@@ -766,6 +766,12 @@ int ResourceManager::detectMapVersion() {
 #ifdef ENABLE_SCI32
 	// late SCI1.1 and SCI32 maps have last directory entry set to 0xFF
 	// offset set to filesize and 4 more bytes
+
+	// TODO/FIXME: This code was not updated in r43000, which changed the behavior of this
+	// function a lot. To make it compile again "off" was changed to the newly introduced
+	// "lastDirectoryOffset". This is probably not the correct fix, since before r43000
+	// the loop above could not prematurely terminate and thus this would always check the
+	// last directory entry instead of the last checked directory entry.
 	file.seek(lastDirectoryOffset - 7, SEEK_SET);
 	if (file.readByte() == 0xFF && file.readUint16LE() == file.size())
 		return SCI_VERSION_32; // TODO : check if there is a difference between these maps
-- 
cgit v1.2.3