SAGA: Rewrote the MacBinary resource loading code

This simplifies the overall code and makes it easier to understand. Also, a bug with the
speech in the MacBinary packed version has been corrected, so there are no more clicking
sounds before each sample.
The Common::MacResMan code isn't really useful here, since it doesn't expose the offsets
and sizes of the actual files, which is what is needed in SAGA.

svn-id: r55674

1 files changed, 31 insertions, 111 deletions

diff --git a/engines/saga/resource_rsc.cpp b/engines/saga/resource_rsc.cpp
index b07741436b..6d9ef70bfe 100644
--- a/engines/saga/resource_rsc.cpp
+++ b/engines/saga/resource_rsc.cpp
@@ -26,142 +26,62 @@
 // RSC Resource file management module (SAGA 1, used in ITE)
 
 #include "saga/saga.h"
-
-#include "saga/actor.h"
-#include "saga/animation.h"
-#include "saga/interface.h"
-#include "saga/music.h"
 #include "saga/resource.h"
-#include "saga/scene.h"
-#include "saga/sndres.h"
-
-#include "engines/advancedDetector.h"
 
 namespace Saga {
 
-struct MacResource {
-	int16 id;
-	int32 dataOffset;
-	MacResource() : id(0), dataOffset(0) {}
-};
-
-class MacResourceArray : public Common::Array<MacResource> {
-};
-
-struct MacResType {
-	uint32 id;
-	int16 maxItemId;
-	int16 offset;
-	MacResourceArray resources;
-	MacResType() : id(0), maxItemId(0), offset(0) {
-	}
-};
-
-class MacResTypeArray : public Common::Array<MacResType> {
-};
-
 #define ID_MIDI     MKID_BE('Midi')
 
-bool ResourceContext_RSC::loadMac() {
-	int32 macDataSize, macResSizePad, macResOffset;
-	uint32 macMapOffset, macDataOffset;
-
-	MacResTypeArray macResTypes;
-
-	bool notSagaContext = false;
-
+bool ResourceContext_RSC::loadMacMIDI() {
 	// Sanity check
 	if (_fileSize < RSC_MIN_FILESIZE + MAC_BINARY_HEADER_SIZE)
 		return false;
 
-	_file.seek(82);
-	if (_file.readByte() != 0)
-		return false;
-
-	macDataSize = _file.readSint32BE();
-	macResOffset = MAC_BINARY_HEADER_SIZE + (((macDataSize + 127) >> 7) << 7);
+	_file.seek(83);
+	int macDataSize = _file.readSint32BE();
+	int macResOffset = MAC_BINARY_HEADER_SIZE + (((macDataSize + 127) >> 7) << 7);
 
-	macResSizePad = (((_file.readSint32BE() + 127) >> 7) << 7);
 	_file.seek(macResOffset);
-
-	macDataOffset = _file.readUint32BE() + macResOffset;
-	macMapOffset = _file.readUint32BE() + macResOffset;
-	// Used for sanity checks
-	uint32 macDataLength = _file.readUint32BE();
-	uint32 macMapLength = _file.readUint32BE();
-
-	if (macDataOffset >= (uint)_fileSize || macMapOffset >= (uint)_fileSize ||
-		macDataLength + macMapLength > (uint)_fileSize)
-			return false;
+	uint32 macDataOffset = _file.readUint32BE() + macResOffset;
+	uint32 macMapOffset = _file.readUint32BE() + macResOffset;
 
 	_file.seek(macMapOffset + 22);
-
 	_file.readUint16BE();	// resAttr
 	int16 typeOffset = _file.readUint16BE();
 	_file.readUint16BE();	// nameOffset
-	int16 numTypes = _file.readUint16BE() + 1;
-	macResTypes.resize(numTypes);
+	uint16 numTypes = _file.readUint16BE() + 1;
 
 	_file.seek(macMapOffset + typeOffset + 2);
 
-	for (MacResTypeArray::iterator k = macResTypes.begin(); k != macResTypes.end(); ++k) {
-		k->id = _file.readUint32BE();
-		int16 items = _file.readUint16BE() + 1;
-		k->resources.resize(items);
-		k->offset = _file.readUint16BE();
-	}
-
-	for (MacResTypeArray::iterator k = macResTypes.begin(); k != macResTypes.end(); ++k) {
-		_file.seek(k->offset + macMapOffset + typeOffset);
-
-		for (MacResourceArray::iterator j = k->resources.begin(); j != k->resources.end(); ++j) {
-			j->id = _file.readUint16BE();
-			_file.readUint16BE();	// nameOffset
-			j->dataOffset = _file.readUint32BE();
-			_file.readUint32BE();	// macResSize
-
-			j->dataOffset &= 0xFFFFFF;
-			if (j->id > k->maxItemId)
-				k->maxItemId = j->id;
-		}
-	}
-
-	//
-	for (MacResTypeArray::iterator k = macResTypes.begin(); k != macResTypes.end(); ++k) {
-		//getting offsets & sizes of midi
-		if (((_fileType & GAME_MUSICFILE_GM) > 0) && (k->id == ID_MIDI)) {
-
-			_table.resize(k->maxItemId + 1);
-			for (MacResourceArray::iterator j = k->resources.begin(); j != k->resources.end(); ++j) {
-				_file.seek(macDataOffset + j->dataOffset);
-				_table[j->id].size = _file.readUint32BE();
-				_table[j->id].offset = _file.pos();
+	// Find the MIDI files
+	for (uint16 i = 0; i < numTypes; i++) {
+		uint32 id = _file.readUint32BE();
+		uint16 items = _file.readUint16BE() + 1;
+		uint16 offset = _file.readUint16BE();
+
+		if (id == ID_MIDI) {
+			for (uint16 curMidi = 0; curMidi < items; curMidi++) {
+				// Jump to the header of the entry and read its fields
+				_file.seek(offset + macMapOffset + typeOffset + curMidi * 12);
+				uint16 midiID = _file.readUint16BE();
+				_file.readUint16BE();	// nameOffset
+				uint32 midiOffset = _file.readUint32BE() & 0xFFFFFF;
+				_file.readUint32BE();	// macResSize
+
+				// Jump to the actual data and read the file size
+				_file.seek(macDataOffset + midiOffset);
+				uint32 midiSize = _file.readUint32BE();
+
+				// Add the entry
+				if (_table.size() <= midiID)
+					_table.resize(midiID + 1);
+				_table[midiID].offset = macDataOffset + midiOffset + 4;
+				_table[midiID].size = midiSize;
 			}
-			notSagaContext = true;
-			break;
 		}
 	}
 
-	if ((!notSagaContext) && (!loadRes(MAC_BINARY_HEADER_SIZE, macDataSize))) {
-		return false;
-	}
-
 	return true;
 }
 
-uint32 Resource_RSC::convertResourceId(uint32 resourceId) {
-
-	if (_vm->isMacResources()) {
-		if (resourceId > 1537) {
-			return resourceId - 2;
-		} else {
-			if (resourceId == 1535 || resourceId == 1536) {
-				error("Wrong resource number %d for Mac ITE", resourceId);
-			}
-		}
-	}
-
-	return resourceId;
-}
-
 } // End of namespace Saga