[Scummvm-cvs-logs] scummvm master -> 26f1643e8646cac71b0adb7b88876674134f3613

bluegr md5 at scummvm.org
Wed Dec 28 11:16:22 CET 2011


This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
26f1643e86 DREAMWEB: Remove a lot of unused/duplicate constants and move some of them to dreambase.h


Commit: 26f1643e8646cac71b0adb7b88876674134f3613
    https://github.com/scummvm/scummvm/commit/26f1643e8646cac71b0adb7b88876674134f3613
Author: Filippos Karapetis (md5 at scummvm.org)
Date: 2011-12-28T02:15:52-08:00

Commit Message:
DREAMWEB: Remove a lot of unused/duplicate constants and move some of them to dreambase.h

Changed paths:
    devtools/tasmrecover/tasm-recover
    engines/dreamweb/backdrop.cpp
    engines/dreamweb/dreambase.h
    engines/dreamweb/dreamgen.h
    engines/dreamweb/saveload.cpp
    engines/dreamweb/sprite.cpp
    engines/dreamweb/stubs.cpp



diff --git a/devtools/tasmrecover/tasm-recover b/devtools/tasmrecover/tasm-recover
index 856b11c..e7791c4 100755
--- a/devtools/tasmrecover/tasm-recover
+++ b/devtools/tasmrecover/tasm-recover
@@ -513,6 +513,21 @@ p = parser(skip_binary_data = [
 	'frframedata',
 	'zoomx',
 	'zoomy',
+	'menux',
+	'menuy',
+	'headerlen',
+	'freedatlen',
+	'diaryx',
+	'diaryy',
+	'inventx',
+	'inventy',
+	'screenwidth',
+	'mapwidth',
+	'opsx',
+	'opsy',
+	'symbolx',
+	'symboly',
+	'numchanges',
 	# vgagrafx.asm
 	'cityname',
 	'extragraphics1',
diff --git a/engines/dreamweb/backdrop.cpp b/engines/dreamweb/backdrop.cpp
index 2a17a7a..7c0713c 100644
--- a/engines/dreamweb/backdrop.cpp
+++ b/engines/dreamweb/backdrop.cpp
@@ -26,7 +26,7 @@ namespace DreamGen {
 
 void DreamBase::doBlocks() {
 	uint16 dstOffset = _mapAdY * 320 + _mapAdX;
-	uint16 mapOffset = _mapY * kMapwidth + _mapX;
+	uint16 mapOffset = _mapY * kMapWidth + _mapX;
 	const uint8 *mapData = _mapData + mapOffset;
 	uint8 *dstBuffer = workspace() + dstOffset;
 
@@ -57,7 +57,7 @@ void DreamBase::doBlocks() {
 				memset(dst, 0xdf, 16);
 			}
 		}
-		mapData += kMapwidth;
+		mapData += kMapWidth;
 	}
 }
 
@@ -236,12 +236,12 @@ void DreamBase::showAllFree() {
 
 void DreamBase::drawFlags() {
 	MapFlag *mapFlag = _mapFlags;
-	uint16 mapOffset = _mapY * kMapwidth + _mapX;
+	uint16 mapOffset = _mapY * kMapWidth + _mapX;
 	const uint8 *mapData = _mapData + mapOffset;
 
 	for (size_t i = 0; i < 10; ++i) {
 		for (size_t j = 0; j < 11; ++j) {
-			uint8 tile = mapData[i * kMapwidth + j];
+			uint8 tile = mapData[i * kMapWidth + j];
 			mapFlag->_flag = _backdropFlags[tile]._flag;
 			mapFlag->_flagEx = _backdropFlags[tile]._flagEx;
 			mapFlag->_type = tile;
diff --git a/engines/dreamweb/dreambase.h b/engines/dreamweb/dreambase.h
index 671664c..e9bf664 100644
--- a/engines/dreamweb/dreambase.h
+++ b/engines/dreamweb/dreambase.h
@@ -52,6 +52,17 @@ const unsigned int kMapWidth = 66;
 const unsigned int kMapHeight = 60;
 const unsigned int kLengthOfMap = kMapWidth * kMapHeight;
 const unsigned int kNumExObjects = 114;
+const unsigned int kScreenwidth = 320;
+const unsigned int kDiaryx = (68+24);
+const unsigned int kDiaryy = (48+12);
+const unsigned int kInventx = 80;
+const unsigned int kInventy = 58;
+const unsigned int kMenux = (80+40);
+const unsigned int kMenuy = 60;
+const unsigned int kOpsx = 60;
+const unsigned int kOpsy = 52;
+const unsigned int kSymbolx = 64;
+const unsigned int kSymboly = 56;
 
 /**
  * This class is one of the parent classes of DreamGenContext. Its sole purpose
diff --git a/engines/dreamweb/dreamgen.h b/engines/dreamweb/dreamgen.h
index bee03f9..d533538 100644
--- a/engines/dreamweb/dreamgen.h
+++ b/engines/dreamweb/dreamgen.h
@@ -105,25 +105,10 @@ static const uint16 kLengthofextra = (0+2080+30000+(16*114)+((114+2)*2)+18000);
 static const uint16 kForeign = (1);
 static const uint16 kCd = (1);
 static const uint16 kNumexobjects = (114);
-static const uint16 kFreedatlen = (16*80);
 static const uint16 kExtextlen = (18000);
-static const uint16 kNumchanges = (250);
 static const uint16 kExframeslen = (30000);
-static const uint16 kScreenwidth = (320);
 static const uint16 kItempicsize = (44);
-static const uint16 kDiaryy = (48+12);
-static const uint16 kOpsy = (52);
-static const uint16 kSymboly = (56);
-static const uint16 kInventy = (58);
-static const uint16 kMenuy = (60);
-static const uint16 kOpsx = (60);
-static const uint16 kSymbolx = (64);
-static const uint16 kMapwidth = (66);
-static const uint16 kDiaryx = (68+24);
 static const uint16 kLengthofvars = (68-0);
-static const uint16 kInventx = (80);
-static const uint16 kMenux = (80+40);
-static const uint16 kHeaderlen = (96);
 
 
 class DreamGenContext : public DreamBase, public Context {
diff --git a/engines/dreamweb/saveload.cpp b/engines/dreamweb/saveload.cpp
index 9faf93a..06cf7e0 100644
--- a/engines/dreamweb/saveload.cpp
+++ b/engines/dreamweb/saveload.cpp
@@ -464,7 +464,7 @@ void DreamBase::savePosition(unsigned int slot, const char *descbuf) {
 
 	// fill length fields in savegame file header
 	uint16 len[6] = { 17, kLengthofvars, kLengthofextra,
-	                  4*kNumchanges, 48, kNumReelRoutines*8+1 };
+	                  4*kNumChanges, 48, kNumReelRoutines*8+1 };
 	for (int i = 0; i < 6; ++i)
 		header.setLen(i, len[i]);
 
@@ -551,7 +551,7 @@ void DreamBase::loadPosition(unsigned int slot) {
 	inSaveFile->read(data.ptr(kStartvars, len[1]), len[1]);
 
 	// the Extras segment:
-	inSaveFile->read((uint8 *)_exFrames._frames, 2080);
+	inSaveFile->read((uint8 *)_exFrames._frames, kExframes);
 	inSaveFile->read((uint8 *)_exFrames._data, kExframeslen);
 	inSaveFile->read((uint8 *)_exData, sizeof(DynObject)*kNumexobjects);
 	inSaveFile->read((uint8 *)_exText._offsetsLE, 2*(kNumExObjects+2));
diff --git a/engines/dreamweb/sprite.cpp b/engines/dreamweb/sprite.cpp
index d5617da..c2646d6 100644
--- a/engines/dreamweb/sprite.cpp
+++ b/engines/dreamweb/sprite.cpp
@@ -1092,14 +1092,13 @@ void DreamBase::soundOnReels(uint16 reelPointer) {
 }
 
 void DreamBase::clearBeforeLoad() {
-	const uint16 kMaplen = 66*60;
 	if (_roomLoaded != 1)
 		return; // noclear
 
 	clearReels();
 
 	//clearRest
-	memset(_mapData, 0, kMaplen);
+	memset(_mapData, 0, kLengthOfMap);
 	delete[] _backdropBlocks;
 	_backdropBlocks = 0;
 	_setFrames.clear();
diff --git a/engines/dreamweb/stubs.cpp b/engines/dreamweb/stubs.cpp
index dfc39f8..59fb00e 100644
--- a/engines/dreamweb/stubs.cpp
+++ b/engines/dreamweb/stubs.cpp
@@ -1879,12 +1879,11 @@ void DreamBase::zoomOnOff() {
 }
 
 void DreamBase::sortOutMap() {
-	const uint16 kMaplength = 60;
 	const uint8 *src = workspace();
 	uint8 *dst = _mapData;
-	for (uint16 y = 0; y < kMaplength; ++y) {
-		memcpy(dst, src, kMapwidth);
-		dst += kMapwidth;
+	for (uint16 y = 0; y < kMapHeight; ++y) {
+		memcpy(dst, src, kMapWidth);
+		dst += kMapWidth;
 		src += 132;
 	}
 }
@@ -2135,6 +2134,7 @@ void DreamBase::getRidOfAll() {
 // if skipDat, skip clearing and loading Setdat and Freedat
 void DreamBase::loadRoomData(const Room &room, bool skipDat) {
 	const uint16 kSetdatlen = 64*128;
+	const uint16 kFreedatlen = 16*80;
 
 	engine->openFile(room.name);
 
@@ -3545,7 +3545,7 @@ void DreamBase::clearBuffers() {
 }
 
 void DreamBase::clearChanges() {
-	memset(_listOfChanges, 0xFF, 4*kNumchanges);
+	memset(_listOfChanges, 0xFF, 4*kNumChanges);
 
 	setupInitialReelRoutines();
 






More information about the Scummvm-git-logs mailing list