[Scummvm-cvs-logs] CVS: scummvm/scumm resource_v7he.cpp,1.14,1.15

[Eugene Sandulenko]

Update of /cvsroot/scummvm/scummvm/scumm
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv17171

Modified Files:
	resource_v7he.cpp 
Log Message:
Remove debug output and improve error messages.

To clarify my last commit message. I _do_ work with unprocessed files.

Just I happened to have one demo with resource fork stripped from game binary.
It was told that sometimes Mac conversion programs on intel platform silently
strip it down. Now I fail with error message, but we may use some fallback.

At least we need to know a standard way of copying binary files with resources
unstripped and describe it in Mac HOWTO.


Index: resource_v7he.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/resource_v7he.cpp,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -d -r1.14 -r1.15
--- resource_v7he.cpp	31 Jan 2005 03:53:35 -0000	1.14
+++ resource_v7he.cpp	31 Jan 2005 04:04:04 -0000	1.15
@@ -1302,14 +1302,13 @@
 	// we haven't calculated it
 	if (_resOffset == -1) {
 		if (!init(in))
-			error("Invalid file format (%s)", _fileName);
-		debug(0, "ResOffset: %d", _resOffset);
+			error("Resource fork is missing in file '%s'", _fileName);
 	}
 
 	*buf = getResource(in, "crsr", 1000 + id, &size);
 
 	if (*buf == NULL)
-		error("Cannot read cursor ID: %d", id);
+		error("There is no cursor ID #%d", id);
 
 	return size;
 }
@@ -1362,7 +1361,7 @@
 	_dataLength = in.readUint32BE();
 	_mapLength = in.readUint32BE();
 
-	debug(0, "got header: data %d [%d] map %d [%d]", 
+	debug(7, "got header: data %d [%d] map %d [%d]", 
 		_dataOffset, _dataLength, _mapOffset, _mapLength);
 
 	readMap(in);
@@ -1418,9 +1417,6 @@
 	_resMap.numTypes = in.readUint16BE();
 	_resMap.numTypes++;
 
-	debug(0, "Read %d types, type offset %d, name offset %d", _resMap.numTypes,
-		_resMap.typeOffset, _resMap.nameOffset);
-
 	in.seek(_mapOffset + _resMap.typeOffset + 2);
 	_resTypes = new ResType[_resMap.numTypes];
 
@@ -1431,13 +1427,6 @@
 		_resTypes[i].items++;
 	}
 	
-	for (i = 0; i < _resMap.numTypes; i++) {
-		debug(0, "resource '%c%c%c%c': items %d, offset %d", 
-			_resTypes[i].id[0], _resTypes[i].id[1], 
-			_resTypes[i].id[2], _resTypes[i].id[3],
-			_resTypes[i].items, _resTypes[i].offset);
-	}	
-
 	_resLists = new ResPtr[_resMap.numTypes];
 	
 	for (i = 0; i < _resMap.numTypes; i++) {
@@ -1455,11 +1444,6 @@
 			
 			resPtr->attr = resPtr->dataOffset >> 24;
 			resPtr->dataOffset &= 0xFFFFFF;
-
-			debug(0, "resource '%c%c%c%c' %d: name %d, data %d, attr %d", 
-				_resTypes[i].id[0], _resTypes[i].id[1], 
-				_resTypes[i].id[2], _resTypes[i].id[3],
-				resPtr->id, resPtr->nameOffset, resPtr->dataOffset, resPtr->attr);
 		}
 
 		for (j = 0; j < _resTypes[i].items; j++) {
@@ -1470,7 +1454,6 @@
 				_resLists[i][j].name = new byte[len + 1];
 				_resLists[i][j].name[len] = 0;
 				in.read(_resLists[i][j].name, len);
-				debug(0, "name of %d: %s", _resLists[i][j].id, _resLists[i][j].name);
 			}
 		}
 	}