[Scummvm-cvs-logs] SF.net SVN: scummvm: [23885] scummvm/trunk/engines/scumm

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Sat Sep 16 15:39:10 CEST 2006


Revision: 23885
          http://svn.sourceforge.net/scummvm/?rev=23885&view=rev
Author:   fingolfin
Date:     2006-09-16 06:38:43 -0700 (Sat, 16 Sep 2006)

Log Message:
-----------
Replaced checkRange by assertRange, which has (a) an (IMO) more logical order of parameters, and (b) removes lots of useless duplicate information in error messages

Modified Paths:
--------------
    scummvm/trunk/engines/scumm/actor.cpp
    scummvm/trunk/engines/scumm/akos.cpp
    scummvm/trunk/engines/scumm/boxes.cpp
    scummvm/trunk/engines/scumm/charset.cpp
    scummvm/trunk/engines/scumm/he/palette_he.cpp
    scummvm/trunk/engines/scumm/he/script_v100he.cpp
    scummvm/trunk/engines/scumm/he/script_v60he.cpp
    scummvm/trunk/engines/scumm/he/script_v72he.cpp
    scummvm/trunk/engines/scumm/he/script_v90he.cpp
    scummvm/trunk/engines/scumm/he/sound_he.cpp
    scummvm/trunk/engines/scumm/he/sprite_he.cpp
    scummvm/trunk/engines/scumm/object.cpp
    scummvm/trunk/engines/scumm/palette.cpp
    scummvm/trunk/engines/scumm/resource.cpp
    scummvm/trunk/engines/scumm/resource_v3.cpp
    scummvm/trunk/engines/scumm/resource_v4.cpp
    scummvm/trunk/engines/scumm/room.cpp
    scummvm/trunk/engines/scumm/script.cpp
    scummvm/trunk/engines/scumm/script_v2.cpp
    scummvm/trunk/engines/scumm/script_v5.cpp
    scummvm/trunk/engines/scumm/script_v6.cpp
    scummvm/trunk/engines/scumm/script_v8.cpp
    scummvm/trunk/engines/scumm/scumm.cpp
    scummvm/trunk/engines/scumm/util.cpp
    scummvm/trunk/engines/scumm/util.h

Modified: scummvm/trunk/engines/scumm/actor.cpp
===================================================================
--- scummvm/trunk/engines/scumm/actor.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/actor.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -1185,7 +1185,7 @@
 			if (talkState == 0)
 				talkState = _vm->_rnd.getRandomNumberRng(1, 10);
 
-			checkRange(13, 1, talkState, "Talk state %d out of range");
+			assertRange(1, talkState, 13, "Talk state");
 			setTalkCondition(talkState);
 		} else {
 			setTalkCondition(1);
@@ -1935,12 +1935,12 @@
 }
 
 int Actor::getAnimVar(byte var) const {
-	checkRange(26, 0, var, "getAnimVar %d out of range(r)");
+	assertRange(0, var, 26, "getAnimVar:");
 	return _animVariable[var];
 }
 
 void Actor::setAnimVar(byte var, int value) {
-	checkRange(26, 0, var, "setAnimVar %d out of range(r)");
+	assertRange(0, var, 26, "setAnimVar:");
 	_animVariable[var] = value;
 }
 
@@ -2052,7 +2052,7 @@
 
 void Actor::setUserCondition(int slot, int set) {
 	const int condMaskCode = (_vm->_game.heversion >= 85) ? 0x1FFF : 0x3FF;
-	checkRange(32, 1, slot, "Condition %d out of range");
+	assertRange(1, slot, 32, "setUserCondition: Condition");
 	if (set == 0) {
 		_heCondMask &= ~(1 << (slot + 0xF));
 	} else {
@@ -2066,13 +2066,13 @@
 }
 
 bool Actor::isUserConditionSet(int slot) const {
-	checkRange(32, 1, slot, "Condition %d out of range");
+	assertRange(1, slot, 32, "isUserConditionSet: Condition");
 	return (_heCondMask & (1 << (slot + 0xF))) != 0;
 }
 
 void Actor::setTalkCondition(int slot) {
 	const int condMaskCode = (_vm->_game.heversion >= 85) ? 0x1FFF : 0x3FF;
-	checkRange(32, 1, slot, "Condition %d out of range");
+	assertRange(1, slot, 32, "setTalkCondition: Condition");
 	_heCondMask = (_heCondMask & ~condMaskCode) | 1;
 	if (slot != 1) {
 		_heCondMask |= 1 << (slot - 1);
@@ -2085,7 +2085,7 @@
 }
 
 bool Actor::isTalkConditionSet(int slot) const {
-	checkRange(32, 1, slot, "Condition %d out of range");
+	assertRange(1, slot, 32, "isTalkConditionSet: Condition");
 	return (_heCondMask & (1 << (slot - 1))) != 0;
 }
 

Modified: scummvm/trunk/engines/scumm/akos.cpp
===================================================================
--- scummvm/trunk/engines/scumm/akos.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/akos.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -1717,7 +1717,7 @@
 
 void ScummEngine_v6::akos_queCommand(byte cmd, Actor *a, int param_1, int param_2) {
 	_akosQueuePos++;
-	checkRange(31, 0, _akosQueuePos, "akos_queCommand overflow");
+	assertRange(0, _akosQueuePos, 31, "akos_queCommand: _akosQueuePos");
 
 	_akosQueue[_akosQueuePos].cmd = cmd;
 	_akosQueue[_akosQueuePos].actor = a->_number;

Modified: scummvm/trunk/engines/scumm/boxes.cpp
===================================================================
--- scummvm/trunk/engines/scumm/boxes.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/boxes.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -401,7 +401,7 @@
 	if (_game.version <= 4 && ptr[0] == box)
 		box--;
 
-	checkRange(ptr[0] - 1, 0, box, "Illegal box %d");
+	assertRange(0, box, ptr[0] - 1, "box");
 	if (_game.version == 0)
 		return (Box *)(ptr + box * SIZEOF_BOX_C64 + 1);
 	else if (_game.version <= 2)

Modified: scummvm/trunk/engines/scumm/charset.cpp
===================================================================
--- scummvm/trunk/engines/scumm/charset.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/charset.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -226,7 +226,7 @@
 }
 
 void CharsetRendererCommon::setCurID(byte id) {
-	checkRange(_vm->_numCharsets - 1, 0, id, "Printing with bad charset %d");
+	assertRange(0, id, _vm->_numCharsets - 1, "charset");
 
 	_curId = id;
 
@@ -245,7 +245,7 @@
 }
 
 void CharsetRendererV3::setCurID(byte id) {
-	checkRange(_vm->_numCharsets - 1, 0, id, "Printing with bad charset %d");
+	assertRange(0, id, _vm->_numCharsets - 1, "charset");
 
 	_curId = id;
 
@@ -1206,7 +1206,7 @@
 	byte *charPtr, *dst;
 	int is2byte = (chr >= 0x80 && _vm->_useCJKMode) ? 1 : 0;
 
-	checkRange(_vm->_numCharsets - 1, 0, _curId, "Printing with bad charset %d");
+	assertRange(0, _curId, _vm->_numCharsets - 1, "charset");
 
 	if ((vs = _vm->findVirtScreen(_top)) == NULL)
 		return;
@@ -1316,7 +1316,7 @@
 	const byte *charPtr;
 	int is2byte = (chr >= 0x80 && _vm->_useCJKMode) ? 1 : 0;
 
-	checkRange(_vm->_numCharsets - 1, 1, _curId, "Printing with bad charset %d");
+	assertRange(1, _curId, _vm->_numCharsets - 1, "charset");
 
 	if ((vs = _vm->findVirtScreen(_top)) == NULL && (vs = _vm->findVirtScreen(_top + getFontHeight())) == NULL)
 		return;

Modified: scummvm/trunk/engines/scumm/he/palette_he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/palette_he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/palette_he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -78,9 +78,9 @@
 }
 
 int ScummEngine_v90he::getHEPaletteSimilarColor(int palSlot, int red, int green, int start, int end) {
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
-	checkRange(255, 0, start, "Invalid palette slot %d");
-	checkRange(255, 0, end, "Invalid palette slot %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
+	assertRange(0, start, 255, "start palette slot");
+	assertRange(0, end, 255, "pend alette slot");
 
 	uint8 *pal = _hePalettes + palSlot * 1024 + start * 3;
 
@@ -104,22 +104,22 @@
 }
 
 int ScummEngine_v90he::getHEPaletteColorComponent(int palSlot, int color, int component) {
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
-	checkRange(255, 0, color, "Invalid palette slot %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
+	assertRange(0, color, 255, "palette slot");
 
 	return _hePalettes[palSlot * 1024 + color * 3 + component % 3];
 }
 
 int ScummEngine_v90he::getHEPaletteColor(int palSlot, int color) {
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
-	checkRange(255, 0, color, "Invalid palette slot %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
+	assertRange(0, color, 255, "palette slot");
 
 	return _hePalettes[palSlot * 1024 + 768 + color];
 }
 
 void ScummEngine_v90he::setHEPaletteColor(int palSlot, uint8 color, uint8 r, uint8 g, uint8 b) {
 	debug(7, "setHEPaletteColor(%d, %d, %d, %d, %d)", palSlot, color, r, g, b);
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	uint8 *p = _hePalettes + palSlot * 1024 + color * 3;
 	*(p + 0) = r;
 	*(p + 1) = g;
@@ -128,7 +128,7 @@
 }
 
 void ScummEngine_v90he::setHEPaletteFromPtr(int palSlot, const uint8 *palData) {
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	uint8 *pc = _hePalettes + palSlot * 1024;
 	uint8 *pi = pc + 768;
 	for (int i = 0; i < 256; ++i) {
@@ -141,7 +141,7 @@
 
 void ScummEngine_v90he::setHEPaletteFromCostume(int palSlot, int resId) {
 	debug(7, "setHEPaletteFromCostume(%d, %d)", palSlot, resId);
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	const uint8 *data = getResourceAddress(rtCostume, resId);
 	assert(data);
 	const uint8 *rgbs = findResourceData(MKID_BE('RGBS'), data);
@@ -151,7 +151,7 @@
 
 void ScummEngine_v90he::setHEPaletteFromImage(int palSlot, int resId, int state) {
 	debug(7, "setHEPaletteFromImage(%d, %d, %d)", palSlot, resId, state);
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	uint8 *data = getResourceAddress(rtImage, resId);
 	assert(data);
 	const uint8 *rgbs = findWrappedBlock(MKID_BE('RGBS'), data, state, 0);
@@ -161,7 +161,7 @@
 
 void ScummEngine_v90he::setHEPaletteFromRoom(int palSlot, int resId, int state) {
 	debug(7, "setHEPaletteFromRoom(%d, %d, %d)", palSlot, resId, state);
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	const uint8 *data = getResourceAddress(rtRoom, resId);
 	assert(data);
 	const uint8 *pals = findResourceData(MKID_BE('PALS'), data);
@@ -173,7 +173,7 @@
 
 void ScummEngine_v90he::restoreHEPalette(int palSlot) {
 	debug(7, "restoreHEPalette(%d)", palSlot);
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	if (palSlot != 1) {
 		memcpy(_hePalettes + palSlot * 1024, _hePalettes + 1024, 1024);
 	}
@@ -190,7 +190,7 @@
 
 void ScummEngine_v90he::copyHEPaletteColor(int palSlot, uint8 dstColor, uint8 srcColor) {
 	debug(7, "copyHEPaletteColor(%d, %d, %d)", palSlot, dstColor, srcColor);
-	checkRange(_numPalettes, 1, palSlot, "Invalid palette %d");
+	assertRange(1, palSlot, _numPalettes, "palette");
 	uint8 *dstPal = _hePalettes + palSlot * 1024 + dstColor * 3;
 	uint8 *srcPal = _hePalettes + 1024 + srcColor * 3;
 	memcpy(dstPal, srcPal, 3);
@@ -205,7 +205,7 @@
 		numcolor = getResourceDataSize(ptr) / 3;
 	}
 
-	checkRange(256, 0, numcolor, "Too many colors (%d) in Palette");
+	assertRange(0, numcolor, 256, "setPaletteFromPtr: numcolor");
 
 	dest = _hePalettes + 1024;
 

Modified: scummvm/trunk/engines/scumm/he/script_v100he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/script_v100he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/script_v100he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -455,7 +455,7 @@
 	case 57:		// SO_PALETTE
 		j = pop();
 		i = pop();
-		checkRange(255, 0, i, "o100_actorOps: Illegal palette slot %d");
+		assertRange(0, i, 255, "palette slot");
 		a->remapActorPaletteColor(i, j);
 		a->_needRedraw = true;
 		break;

Modified: scummvm/trunk/engines/scumm/he/script_v60he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/script_v60he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/script_v60he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -554,7 +554,7 @@
 	case 187:		// SO_CYCLE_SPEED
 		b = pop();
 		a = pop();
-		checkRange(16, 1, a, "o60_roomOps: 187: color cycle out of range (%d)");
+		assertRange(1, a, 16, "o60_roomOps: 187: color cycle");
 		_colorCycle[a - 1].delay = (b != 0) ? 0x4000 / (b * 0x4C) : 0;
 		break;
 
@@ -682,7 +682,7 @@
 	case 86:		// SO_PALETTE
 		j = pop();
 		i = pop();
-		checkRange(255, 0, i, "Illegal palette slot %d");
+		assertRange(0, i, 255, "o60_actorOps: palette slot");
 		a->remapActorPaletteColor(i, j);
 		a->_needRedraw = true;
 		break;

Modified: scummvm/trunk/engines/scumm/he/script_v72he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/script_v72he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/script_v72he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -1174,7 +1174,7 @@
 	case 86:		// SO_PALETTE
 		j = pop();
 		i = pop();
-		checkRange(255, 0, i, "Illegal palette slot %d");
+		assertRange(0, i, 255, "palette slot");
 		a->remapActorPaletteColor(i, j);
 		a->_needRedraw = true;
 		break;
@@ -1279,7 +1279,7 @@
 	if (subOp == 196) {
 		_curVerb = pop();
 		_curVerbSlot = getVerbSlot(_curVerb, 0);
-		checkRange(_numVerbs - 1, 0, _curVerbSlot, "Illegal new verb slot %d");
+		assertRange(0, _curVerbSlot, _numVerbs - 1, "new verb slot");
 		return;
 	}
 	vs = &_verbs[_curVerbSlot];

Modified: scummvm/trunk/engines/scumm/he/script_v90he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/script_v90he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/script_v90he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -466,7 +466,7 @@
 		push(a->isUserConditionSet(val));
 		break;
 	case 2:
-		checkRange(15, 0, val, "Limb %d out of range");
+		assertRange(0, val, 15, "o90_getActorData: Limb");
 		push(a->_cost.frame[val]);
 		break;
 	case 3:

Modified: scummvm/trunk/engines/scumm/he/sound_he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/sound_he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/sound_he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -262,7 +262,7 @@
 		return isSoundCodeUsed(sound);
 	}
 
-	checkRange(25, 0, var, "Illegal sound variable %d");
+	assertRange(0, var, 25, "sound variable");
 
 	int chan = -1;
 	for (int i = 0; i < ARRAYSIZE(_heChannel); i ++) {
@@ -279,7 +279,7 @@
 }
 
 void SoundHE::setSoundVar(int sound, int var, int val) {
-	checkRange(25, 0, var, "Illegal sound variable %d");
+	assertRange(0, var, 25, "sound variable");
 
 	int chan = -1;
 	for (int i = 0; i < ARRAYSIZE(_heChannel); i ++) {

Modified: scummvm/trunk/engines/scumm/he/sprite_he.cpp
===================================================================
--- scummvm/trunk/engines/scumm/he/sprite_he.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/he/sprite_he.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -49,7 +49,7 @@
 }
 
 void Sprite::getSpriteBounds(int spriteId, bool checkGroup, Common::Rect &bound) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 	int32 spr_wiz_x, spr_wiz_y;
 	int angle, scale, x1, y1;
 	int32 w, h;
@@ -115,7 +115,7 @@
 		for (int j = 0; j < num; j++) {
 			code = classId = args[j];
 			classId &= 0x7F;
-			checkRange(32, 1, classId, "class %d out of range in statement");
+			assertRange(1, classId, 32, "class");
 			if (code & 0x80) {
 				if (!(spi->classFlags & (1 << (classId - 1))))
 					cond = 0;
@@ -204,7 +204,7 @@
 }
 
 int Sprite::getSpriteClass(int spriteId, int num, int *args) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 	int code, classId;
 
 	if (num == 0)
@@ -213,7 +213,7 @@
 	for (int i = 0; i < num; i++) {
 		code = classId = args[i];
 		classId &= 0x7F;
-		checkRange(32, 1, classId, "class %d out of range in statement");
+		assertRange(1, classId, 32, "class");
 		if (code & 0x80) {
 			if (!(_spriteTable[spriteId].classFlags & (1 << (classId - 1))))
 				return 0;
@@ -227,85 +227,85 @@
 }
 
 int Sprite::getSpriteFlagDoubleBuffered(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFDoubleBuffered) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagYFlipped(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFYFlipped) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagXFlipped(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFXFlipped) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagActive(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFActive) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagRemapPalette(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFRemapPalette) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagAutoAnim(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFAutoAnim) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagUpdateType(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFMarkDirty) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteFlagEraseType(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return ((_spriteTable[spriteId].flags & kSFImageless) != 0) ? 1 : 0;
 }
 
 int Sprite::getSpriteImage(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].image;
 }
 
 int Sprite::getSpriteImageState(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].imageState;
 }
 
 int Sprite::getSpriteGroup(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].group;
 }
 
 int Sprite::getSpritePalette(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].palette;
 }
 
 int Sprite::getSpritePriority(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].priority;
 }
 
 int Sprite::getSpriteDisplayX(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].group)
 		return _spriteTable[spriteId].tx + _spriteGroups[_spriteTable[spriteId].group].tx;
@@ -314,7 +314,7 @@
 }
 
 int Sprite::getSpriteDisplayY(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].group)
 		return _spriteTable[spriteId].ty + _spriteGroups[_spriteTable[spriteId].group].ty;
@@ -323,50 +323,50 @@
 }
 
 int Sprite::getSpriteUserValue(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].userValue;
 }
 
 int Sprite::getSpriteShadow(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].shadow;
 }
 
 int Sprite::getSpriteImageStateCount(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].imageStateCount;
 }
 
 int Sprite::getSpriteScale(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].scale;
 }
 
 int Sprite::getSpriteAnimSpeed(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].animSpeed;
 }
 
 int Sprite::getSpriteSourceImage(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].sourceImage;
 }
 
 int Sprite::getSpriteMaskImage(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	return _spriteTable[spriteId].maskImage;
 }
 
 int Sprite::getSpriteGeneralProperty(int spriteId, int type) {
 	debug(0, "getSpriteGeneralProperty: spriteId %d type 0x%x", spriteId, type);
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	// XXX U32 related check
 
@@ -383,7 +383,7 @@
 }
 
 void Sprite::getSpriteImageDim(int spriteId, int32 &w, int32 &h) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].image) {
 		_vm->_wiz->getWizImageDim(_spriteTable[spriteId].image, _spriteTable[spriteId].imageState, w, h);
@@ -394,14 +394,14 @@
 }
 
 void Sprite::getSpritePosition(int spriteId, int32 &tx, int32 &ty) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	tx = _spriteTable[spriteId].tx;
 	ty = _spriteTable[spriteId].ty;
 }
 
 void Sprite::getSpriteDist(int spriteId, int32 &dx, int32 &dy) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	dx = _spriteTable[spriteId].dx;
 	dy = _spriteTable[spriteId].dy;
@@ -413,7 +413,7 @@
 int ScummEngine_v90he::getGroupSpriteArray(int spriteGroupId) {
 	int i, numSprites = 0;
 
-	checkRange(_sprite->_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _sprite->_varNumSpriteGroups, "sprite group");
 
 	for (i = (_sprite->_varNumSprites - 1); i > 0; i--) {
 		if (_sprite->_spriteTable[i].group == spriteGroupId)
@@ -439,43 +439,43 @@
 }
 
 int Sprite::getGroupPriority(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	return _spriteGroups[spriteGroupId].priority;
 }
 
 int Sprite::getGroupDstResNum(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	return _spriteGroups[spriteGroupId].image;
 }
 
 int Sprite::getGroupXMul(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	return _spriteGroups[spriteGroupId].scale_x_ratio_mul;
 }
 
 int Sprite::getGroupXDiv(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	return _spriteGroups[spriteGroupId].scale_x_ratio_div;
 }
 
 int Sprite::getGroupYMul(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	return _spriteGroups[spriteGroupId].scale_y_ratio_mul;
 }
 
 int Sprite::getGroupYDiv(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	return _spriteGroups[spriteGroupId].scale_y_ratio_div;
 }
 
 void Sprite::getGroupPosition(int spriteGroupId, int32 &tx, int32 &ty) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	tx = _spriteGroups[spriteGroupId].tx;
 	ty = _spriteGroups[spriteGroupId].ty;
@@ -485,7 +485,7 @@
 // spriteInfoSet functions
 //
 void Sprite::setSpritePalette(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].palette != value) {
 		_spriteTable[spriteId].palette = value;
@@ -494,7 +494,7 @@
 }
 
 void Sprite::setSpriteSourceImage(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].sourceImage != value) {
 		_spriteTable[spriteId].sourceImage = value;
@@ -503,13 +503,13 @@
 }
 
 void Sprite::setSpriteMaskImage(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].maskImage = value;
 }
 
 void Sprite::setSpriteImageState(int spriteId, int state) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].image) {
 		int imageStateCount = _spriteTable[spriteId].imageStateCount - 1;
@@ -524,7 +524,7 @@
 }
 
 void Sprite::setSpritePosition(int spriteId, int tx, int ty) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (_spriteTable[spriteId].tx != tx || _spriteTable[spriteId].ty != ty) {
 		_spriteTable[spriteId].tx = tx;
@@ -534,22 +534,22 @@
 }
 
 void Sprite::setSpriteGroup(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
-	checkRange(_varNumSpriteGroups, 0, value, "Invalid sprite group %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
+	assertRange(0, value, _varNumSpriteGroups, "sprite group");
 
 	_spriteTable[spriteId].group = value;
 	_spriteTable[spriteId].flags |= kSFChanged | kSFNeedRedraw;
 }
 
 void Sprite::setSpriteDist(int spriteId, int value1, int value2) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].dx = value1;
 	_spriteTable[spriteId].dy = value2;
 }
 
 void Sprite::setSpriteShadow(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].shadow = value;
 	if (_spriteTable[spriteId].image)
@@ -557,19 +557,19 @@
 }
 
 void Sprite::setSpriteUserValue(int spriteId, int value1, int value2) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].userValue = value2;
 }
 
 void Sprite::setSpritePriority(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].priority = value;
 }
 
 void Sprite::moveSprite(int spriteId, int value1, int value2) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].tx += value1;
 	_spriteTable[spriteId].ty += value2;
@@ -579,7 +579,7 @@
 }
 
 void Sprite::setSpriteScale(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].flags |= kSFScaled;
 
@@ -592,7 +592,7 @@
 }
 
 void Sprite::setSpriteAngle(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].flags |= kSFRotated;
 
@@ -605,7 +605,7 @@
 }
 
 void Sprite::setSpriteFlagDoubleBuffered(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	int oldFlags = _spriteTable[spriteId].flags;
 	if (value)
@@ -618,7 +618,7 @@
 }
 
 void Sprite::setSpriteFlagYFlipped(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	int oldFlags = _spriteTable[spriteId].flags;
 	if (value)
@@ -631,7 +631,7 @@
 }
 
 void Sprite::setSpriteFlagXFlipped(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	int oldFlags = _spriteTable[spriteId].flags;
 	if (value)
@@ -644,7 +644,7 @@
 }
 
 void Sprite::setSpriteFlagActive(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (value)
 		_spriteTable[spriteId].flags |= kSFActive;
@@ -653,7 +653,7 @@
 }
 
 void Sprite::setSpriteFlagRemapPalette(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	int oldFlags = _spriteTable[spriteId].flags;
 	if (value)
@@ -666,7 +666,7 @@
 }
 
 void Sprite::setSpriteFlagAutoAnim(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	if (value)
 		_spriteTable[spriteId].flags |= kSFAutoAnim;
@@ -675,7 +675,7 @@
 }
 
 void Sprite::setSpriteFlagUpdateType(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	switch (value) {
 	case 2:
@@ -694,7 +694,7 @@
 }
 
 void Sprite::setSpriteFlagEraseType(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	// Note that condition is inverted
 	if (!value)
@@ -704,15 +704,15 @@
 }
 
 void Sprite::setSpriteAnimSpeed(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].animSpeed = value;
 	_spriteTable[spriteId].animProgress = value;
 }
 
 void Sprite::setSpriteSetClass(int spriteId, int classId, int toggle) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
-	checkRange(32, 1, classId, "class %d out of range in statement");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
+	assertRange(1, classId, 32, "class");
 
 	if (toggle) {
 		_spriteTable[spriteId].classFlags |= (1 << (classId - 1));
@@ -722,20 +722,20 @@
 }
 
 void Sprite::setSpriteResetClass(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].classFlags = 0;
 }
 
 void Sprite::setSpriteField84(int spriteId, int value) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].field_84 = value;
 }
 
 void Sprite::setSpriteGeneralProperty(int spriteId, int type, int value) {
 	debug(0, "setSpriteGeneralProperty: spriteId %d type 0x%x", spriteId, type);
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 	int32 delay;
 
 	// XXX U32 related check
@@ -761,7 +761,7 @@
 }
 
 void Sprite::resetSprite(int spriteId) {
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	_spriteTable[spriteId].angle = 0;
 	_spriteTable[spriteId].scale = 0;
@@ -793,7 +793,7 @@
 void Sprite::setSpriteImage(int spriteId, int imageNum) {
 	int origResId, origResWizStates;
 
-	checkRange(_varNumSprites, 1, spriteId, "Invalid sprite %d");
+	assertRange(1, spriteId, _varNumSprites, "sprite");
 
 	origResId = _spriteTable[spriteId].image;
 	origResWizStates = _spriteTable[spriteId].imageStateCount;
@@ -832,7 +832,7 @@
 }
 
 void Sprite::moveGroupMembers(int spriteGroupId, int value1, int value2) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId) {
@@ -846,7 +846,7 @@
 }
 
 void Sprite::setGroupMembersPriority(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId)
@@ -855,7 +855,7 @@
 }
 
 void Sprite::setGroupMembersGroup(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId) {
@@ -866,7 +866,7 @@
 }
 
 void Sprite::setGroupMembersUpdateType(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId)
@@ -875,7 +875,7 @@
 }
 
 void Sprite::setGroupMembersResetSprite(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId)
@@ -884,7 +884,7 @@
 }
 
 void Sprite::setGroupMembersAnimationSpeed(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId) {
@@ -895,7 +895,7 @@
 }
 
 void Sprite::setGroupMembersAutoAnimFlag(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId) {
@@ -908,7 +908,7 @@
 }
 
 void Sprite::setGroupMembersShadow(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	for (int i = 1; i < _varNumSprites; i++) {
 		if (_spriteTable[i].group == spriteGroupId) {
@@ -920,7 +920,7 @@
 }
 
 void Sprite::setGroupBounds(int spriteGroupId, int x1, int y1, int x2, int y2) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	_spriteGroups[spriteGroupId].flags |= kSGFClipBox;
 	_spriteGroups[spriteGroupId].bbox.left = x1;
@@ -933,7 +933,7 @@
 }
 
 void Sprite::setGroupPriority(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (_spriteGroups[spriteGroupId].priority != value) {
 		_spriteGroups[spriteGroupId].priority = value;
@@ -942,7 +942,7 @@
 }
 
 void Sprite::setGroupPosition(int spriteGroupId, int value1, int value2) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (_spriteGroups[spriteGroupId].tx != value1 || _spriteGroups[spriteGroupId].ty != value2) {
 		_spriteGroups[spriteGroupId].tx = value1;
@@ -952,7 +952,7 @@
 }
 
 void Sprite::moveGroup(int spriteGroupId, int value1, int value2) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (value1 || value2) {
 		_spriteGroups[spriteGroupId].tx += value1;
@@ -962,7 +962,7 @@
 }
 
 void Sprite::setGroupImage(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (_spriteGroups[spriteGroupId].image != value) {
 		_spriteGroups[spriteGroupId].image = value;
@@ -979,7 +979,7 @@
 }
 
 void Sprite::setGroupXMul(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (_spriteGroups[spriteGroupId].scale_x_ratio_mul != value) {
 		_spriteGroups[spriteGroupId].scale_x_ratio_mul = value;
@@ -989,7 +989,7 @@
 }
 
 void Sprite::setGroupXDiv(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (value == 0)
 		error("setGroupXDiv: Divisor must not be 0");
@@ -1002,7 +1002,7 @@
 }
 
 void Sprite::setGroupYMul(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (_spriteGroups[spriteGroupId].scale_y_ratio_mul != value) {
 		_spriteGroups[spriteGroupId].scale_y_ratio_mul = value;
@@ -1012,7 +1012,7 @@
 }
 
 void Sprite::setGroupYDiv(int spriteGroupId, int value) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	if (value == 0)
 		error("setGroupYDiv: Divisor must not be 0");
@@ -1025,7 +1025,7 @@
 }
 
 void Sprite::resetGroupBounds(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 
 	_spriteGroups[spriteGroupId].flags &= ~(kSGFClipBox);
 	redrawSpriteGroup(spriteGroupId);
@@ -1042,7 +1042,7 @@
 }
 
 void Sprite::resetGroup(int spriteGroupId) {
-	checkRange(_varNumSpriteGroups, 1, spriteGroupId, "Invalid sprite group %d");
+	assertRange(1, spriteGroupId, _varNumSpriteGroups, "sprite group");
 	SpriteGroup *spg = &_spriteGroups[spriteGroupId];
 
 	spg->priority = 0;

Modified: scummvm/trunk/engines/scumm/object.cpp
===================================================================
--- scummvm/trunk/engines/scumm/object.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/object.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -55,9 +55,9 @@
 
 
 bool ScummEngine::getClass(int obj, int cls) const {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in getClass");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
 	cls &= 0x7F;
-	checkRange(32, 1, cls, "Class %d out of range in getClass");
+	assertRange(1, cls, 32, "class");
 
 	if (_game.features & GF_SMALL_HEADER) {
 		// Translate the new (V5) object classes to the old classes
@@ -82,9 +82,9 @@
 }
 
 void ScummEngine::putClass(int obj, int cls, bool set) {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in putClass");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
 	cls &= 0x7F;
-	checkRange(32, 1, cls, "Class %d out of range in putClass");
+	assertRange(1, cls, 32, "class");
 
 	if (_game.features & GF_SMALL_HEADER) {
 		// Translate the new (V5) object classes to the old classes
@@ -116,18 +116,18 @@
 }
 
 int ScummEngine::getOwner(int obj) const {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in getOwner");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
 	return _objectOwnerTable[obj];
 }
 
 void ScummEngine::putOwner(int obj, int owner) {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in putOwner");
-	checkRange(0xFF, 0, owner, "Owner %d out of range in putOwner");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
+	assertRange(0, owner, 0xFF, "owner");
 	_objectOwnerTable[obj] = owner;
 }
 
 int ScummEngine::getState(int obj) {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in getState");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
 
 	if (!_copyProtection) {
 		// I knew LucasArts sold cracked copies of the original Maniac Mansion,
@@ -147,13 +147,13 @@
 }
 
 void ScummEngine::putState(int obj, int state) {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in putState");
-	checkRange(0xFF, 0, state, "State %d out of range in putState");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
+	assertRange(0, state, 0xFF, "state");
 	_objectStateTable[obj] = state;
 }
 
 int ScummEngine::getObjectRoom(int obj) const {
-	checkRange(_numGlobalObjects - 1, 0, obj, "Object %d out of range in getObjectRoom");
+	assertRange(0, obj, _numGlobalObjects - 1, "object");
 	return _objectRoomTable[obj];
 }
 
@@ -427,7 +427,7 @@
 	if (od.obj_nr == 0)
 		return;
 
-	checkRange(_numGlobalObjects - 1, 0, od.obj_nr, "Object %d out of range in drawObject");
+	assertRange(0, od.obj_nr, _numGlobalObjects - 1, "object");
 
 	const int xpos = od.x_pos / 8;
 	const int ypos = od.y_pos;
@@ -1619,7 +1619,7 @@
 
 	vs = &virtscr[0];
 
-	checkRange(_numGlobalObjects - 1, 30, eo->number, "Illegal Blast object %d");
+	assertRange(30, eo->number, _numGlobalObjects - 1, "blast object");
 
 	objnum = getObjectIndex(eo->number);
 	if (objnum == -1)

Modified: scummvm/trunk/engines/scumm/palette.cpp
===================================================================
--- scummvm/trunk/engines/scumm/palette.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/palette.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -268,7 +268,7 @@
 		}
 	}
 
-	checkRange(256, 0, numcolor, "Too many colors (%d) in Palette");
+	assertRange(0, numcolor, 256, "setPaletteFromPtr: numcolor");
 
 	dest = _currentPalette;
 
@@ -357,7 +357,7 @@
 void ScummEngine::stopCycle(int i) {
 	ColorCycle *cycl;
 
-	checkRange(16, 0, i, "Stop Cycle %d Out Of Range");
+	assertRange(0, i, 16, "stopCycle: cycle");
 	if (i != 0) {
 		_colorCycle[i - 1].delay = 0;
 		return;

Modified: scummvm/trunk/engines/scumm/resource.cpp
===================================================================
--- scummvm/trunk/engines/scumm/resource.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/resource.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -579,7 +579,7 @@
 	}
 
 	assert(no < (int)sizeof(_charsetData) / 16);
-	checkRange(_numCharsets - 1, 1, no, "Loading illegal charset %d");
+	assertRange(1, no, _numCharsets - 1, "charset");
 
 	ptr = getResourceAddress(rtCharset, no);
 
@@ -589,7 +589,7 @@
 }
 
 void ScummEngine::nukeCharset(int i) {
-	checkRange(_numCharsets - 1, 1, i, "Nuking illegal charset %d");
+	assertRange(1, i, _numCharsets - 1, "charset");
 	res.nukeResource(rtCharset, i);
 }
 

Modified: scummvm/trunk/engines/scumm/resource_v3.cpp
===================================================================
--- scummvm/trunk/engines/scumm/resource_v3.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/resource_v3.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -102,7 +102,7 @@
 	uint32 size;
 	memset(_charsetData, 0, sizeof(_charsetData));
 
-	checkRange(2, 0, no, "Loading illegal charset %d");
+	assertRange(0, no, 2, "charset");
 	closeRoom();
 
 	Common::File file;

Modified: scummvm/trunk/engines/scumm/resource_v4.cpp
===================================================================
--- scummvm/trunk/engines/scumm/resource_v4.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/resource_v4.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -135,7 +135,7 @@
 	uint32 size;
 	memset(_charsetData, 0, sizeof(_charsetData));
 
-	checkRange(4, 0, no, "Loading illegal charset %d");
+	assertRange(0, no, 4, "charset");
 	closeRoom();
 
 	Common::File file;

Modified: scummvm/trunk/engines/scumm/room.cpp
===================================================================
--- scummvm/trunk/engines/scumm/room.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/room.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -331,7 +331,7 @@
 
 			id = READ_LE_UINT32(ptr);
 
-			checkRange(_numLocalScripts + _numGlobalScripts, _numGlobalScripts, id, "Invalid local script %d");
+			assertRange(_numGlobalScripts, id, _numLocalScripts + _numGlobalScripts, "local script");
 			_localScriptOffsets[id - _numGlobalScripts] = ptr + 4 - roomResPtr;
 
 			if (_dumpScripts) {
@@ -366,11 +366,11 @@
 
 			if (_game.version == 8) {
 				id = READ_LE_UINT32(ptr);
-				checkRange(_numLocalScripts + _numGlobalScripts, _numGlobalScripts, id, "Invalid local script %d");
+				assertRange(_numGlobalScripts, id, _numLocalScripts + _numGlobalScripts, "local script");
 				_localScriptOffsets[id - _numGlobalScripts] = ptr + 4 - roomResPtr;
 			} else if (_game.version == 7) {
 				id = READ_LE_UINT16(ptr);
-				checkRange(_numLocalScripts + _numGlobalScripts, _numGlobalScripts, id, "Invalid local script %d");
+				assertRange(_numGlobalScripts, id, _numLocalScripts + _numGlobalScripts, "local script");
 				_localScriptOffsets[id - _numGlobalScripts] = ptr + 2 - roomResPtr;
 			} else {
 				id = ptr[0];

Modified: scummvm/trunk/engines/scumm/script.cpp
===================================================================
--- scummvm/trunk/engines/scumm/script.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/script.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -542,14 +542,14 @@
 			return !ConfMan.getBool("subtitles");
 		}
 
-		checkRange(_numVariables - 1, 0, var, "Variable %d out of range(r)");
+		assertRange(0, var, _numVariables - 1, "variable (reading)");
 		return _scummVars[var];
 	}
 
 	if (var & 0x8000) {
 		if (_game.heversion >= 80) {
 			var &= 0xFFF;
-			checkRange(_numRoomVariables - 1, 0, var, "Room variable %d out of range(w)");
+			assertRange(0, var, _numRoomVariables - 1, "room variable (reading)");
 			return _roomVars[var];
 
 		} else if (_game.version <= 3 && !(_game.id == GID_INDY3 && (_game.platform == Common::kPlatformFMTowns))) {
@@ -564,7 +564,7 @@
 				}
 			}
 
-			checkRange(_numVariables - 1, 0, var, "Variable %d out of range(rzb)");
+			assertRange(0, var, _numVariables - 1, "variable (reading)");
 			return (_scummVars[ var ] & ( 1 << bit ) ) ? 1 : 0;
 		} else {
 			var &= 0x7FFF;
@@ -573,7 +573,7 @@
 					return 0;
 			}
 
-			checkRange(_numBitVariables - 1, 0, var, "Bit variable %d out of range(r)");
+			assertRange(0, var, _numBitVariables - 1, "variable (reading)");
 			return (_bitVars[var >> 3] & (1 << (var & 7))) ? 1 : 0;
 		}
 	}
@@ -586,9 +586,9 @@
 		}
 
 		if (_game.heversion >= 80)
-			checkRange(25, 0, var, "Local variable %d out of range(r)");
+			assertRange(0, var, 25, "local variable (reading)");
 		else
-			checkRange(20, 0, var, "Local variable %d out of range(r)");
+			assertRange(0, var, 20, "local variable (reading)");
 		return vm.localvar[_currentScript][var];
 	}
 
@@ -600,7 +600,7 @@
 	debugC(DEBUG_VARS, "writeVar(%d, %d)", var, value);
 
 	if (!(var & 0xF000)) {
-		checkRange(_numVariables - 1, 0, var, "Variable %d out of range(w)");
+		assertRange(0, var, _numVariables - 1, "variable (writing)");
 
 		if (VAR_SUBTITLES != 0xFF && var == VAR_SUBTITLES) {
 			// Ignore default setting in HE72-73 games
@@ -641,7 +641,7 @@
 	if (var & 0x8000) {
 		if (_game.heversion >= 80) {
 			var &= 0xFFF;
-			checkRange(_numRoomVariables - 1, 0, var, "Room variable %d out of range(w)");
+			assertRange(0, var, _numRoomVariables - 1, "room variable (writing)");
 			_roomVars[var] = value;
 
 		} else if (_game.version <= 3 && !(_game.id == GID_INDY3 && (_game.platform == Common::kPlatformFMTowns))) {
@@ -649,14 +649,14 @@
 			// as the normal variables!
 			int bit = var & 0xF;
 			var = (var >> 4) & 0xFF;
-			checkRange(_numVariables - 1, 0, var, "Variable %d out of range(wzb)");
+			assertRange(0, var, _numVariables - 1, "variable (writing)");
 			if (value)
 				_scummVars[var] |= ( 1 << bit );
 			else
 				_scummVars[var] &= ~( 1 << bit );
 		} else {
 			var &= 0x7FFF;
-			checkRange(_numBitVariables - 1, 0, var, "Bit variable %d out of range(w)");
+			assertRange(0, var, _numBitVariables - 1, "bit variable (writing)");
 
 			if (value)
 				_bitVars[var >> 3] |= (1 << (var & 7));
@@ -674,9 +674,9 @@
 		}
 
 		if (_game.heversion >= 80)
-			checkRange(25, 0, var, "Local variable %d out of range(w)");
+			assertRange(0, var, 25, "local variable (writing)");
 		else
-			checkRange(20, 0, var, "Local variable %d out of range(w)");
+			assertRange(0, var, 20, "local variable (writing)");
 
 		vm.localvar[_currentScript][var] = value;
 		return;

Modified: scummvm/trunk/engines/scumm/script_v2.cpp
===================================================================
--- scummvm/trunk/engines/scumm/script_v2.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/script_v2.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -434,13 +434,13 @@
 	if (var >= 14 && var <= 16)
 		var = _scummVars[var];
 
-	checkRange(_numVariables - 1, 0, var, "Variable %d out of range(r)");
+	assertRange(0, var, _numVariables - 1, "variable (reading)");
 	debugC(DEBUG_VARS, "readvar(%d) = %d", var, _scummVars[var]);
 	return _scummVars[var];
 }
 
 void ScummEngine_v2::writeVar(uint var, int value) {
-	checkRange(_numVariables - 1, 0, var, "Variable %d out of range(r)");
+	assertRange(0, var, _numVariables - 1, "variable (writing)");
 	debugC(DEBUG_VARS, "writeVar(%d) = %d", var, value);
 
 	if (VAR_CUTSCENEEXIT_KEY != 0xFF && var == VAR_CUTSCENEEXIT_KEY) {

Modified: scummvm/trunk/engines/scumm/script_v5.cpp
===================================================================
--- scummvm/trunk/engines/scumm/script_v5.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/script_v5.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -462,7 +462,7 @@
 		case 11:		// SO_PALETTE
 			i = getVarOrDirectByte(PARAM_1);
 			j = getVarOrDirectByte(PARAM_2);
-			checkRange(31, 0, i, "Illegal palette slot %d");
+			assertRange(0, i, 31, "o5_actorOps: palette slot");
 			a->setPalette(i, j);
 			break;
 		case 12:		// SO_TALK_COLOR
@@ -1817,7 +1817,7 @@
 				a = getVarOrDirectWord(PARAM_1);
 				b = getVarOrDirectWord(PARAM_2);
 			}
-			checkRange(256, 0, a, "o5_roomOps: 2: Illegal room color slot (%d)");
+			assertRange(0, a, 256, "o5_roomOps: 2: room color slot");
 			_roomPalette[b] = a;
 			_fullRedraw = true;
 		} else {
@@ -1838,7 +1838,7 @@
 				a = getVarOrDirectWord(PARAM_1);
 				b = getVarOrDirectWord(PARAM_2);
 			}
-			checkRange(256, 0, a, "o5_roomOps: 2: Illegal room color slot (%d)");
+			assertRange(0, a, 256, "o5_roomOps: 4: room color slot");
 			_shadowPalette[b] = a;
 			setDirtyColors(b, b);
 		} else {
@@ -1992,7 +1992,7 @@
 	case 16:	// SO_CYCLE_SPEED
 		a = getVarOrDirectByte(PARAM_1);
 		b = getVarOrDirectByte(PARAM_2);
-		checkRange(16, 1, a, "o5_roomOps: 16: color cycle out of range (%d)");
+		assertRange(1, a, 16, "o5_roomOps: 16: color cycle");
 		_colorCycle[a - 1].delay = (b != 0) ? 0x4000 / (b * 0x4C) : 0;
 		break;
 	default:
@@ -2342,7 +2342,7 @@
 	verb = getVarOrDirectByte(PARAM_1);
 
 	slot = getVerbSlot(verb, 0);
-	checkRange(_numVerbs - 1, 0, slot, "Illegal new verb slot %d");
+	assertRange(0, slot, _numVerbs - 1, "new verb slot");
 
 	vs = &_verbs[slot];
 	vs->verbid = verb;

Modified: scummvm/trunk/engines/scumm/script_v6.cpp
===================================================================
--- scummvm/trunk/engines/scumm/script_v6.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/script_v6.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -1749,7 +1749,7 @@
 	case 187:		// SO_CYCLE_SPEED
 		b = pop();
 		a = pop();
-		checkRange(16, 1, a, "o6_roomOps: 187: color cycle out of range (%d)");
+		assertRange(1, a, 16, "o6_roomOps: 187: color cycle");
 		_colorCycle[a - 1].delay = (b != 0) ? 0x4000 / (b * 0x4C) : 0;
 		break;
 
@@ -1833,7 +1833,7 @@
 	case 86:		// SO_PALETTE
 		j = pop();
 		i = pop();
-		checkRange(255, 0, i, "Illegal palette slot %d");
+		assertRange(0, i, 255, "o6_actorOps: palette slot");
 		a->setPalette(i, j);
 		break;
 	case 87:		// SO_TALK_COLOR
@@ -1933,7 +1933,7 @@
 	if (subOp == 196) {
 		_curVerb = pop();
 		_curVerbSlot = getVerbSlot(_curVerb, 0);
-		checkRange(_numVerbs - 1, 0, _curVerbSlot, "Illegal new verb slot %d");
+		assertRange(0, _curVerbSlot, _numVerbs - 1, "new verb slot");
 		return;
 	}
 	vs = &_verbs[_curVerbSlot];

Modified: scummvm/trunk/engines/scumm/script_v8.cpp
===================================================================
--- scummvm/trunk/engines/scumm/script_v8.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/script_v8.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -393,19 +393,19 @@
 	debugC(DEBUG_VARS, "readvar(%d)", var);
 
 	if (!(var & 0xF0000000)) {
-		checkRange(_numVariables - 1, 0, var, "Variable %d out of range(r)");
+		assertRange(0, var, _numVariables - 1, "variable");
 		return _scummVars[var];
 	}
 
 	if (var & 0x80000000) {
 		var &= 0x7FFFFFFF;
-		checkRange(_numBitVariables - 1, 0, var, "Bit variable %d out of range(r)");
+		assertRange(0, var, _numBitVariables - 1, "bit variable (reading)");
 		return (_bitVars[var >> 3] & (1 << (var & 7))) ? 1 : 0;
 	}
 
 	if (var & 0x40000000) {
 		var &= 0xFFFFFFF;
-		checkRange(25, 0, var, "Local variable %d out of range(r)");
+		assertRange(0, var, 25, "local variable (reading)");
 		return vm.localvar[_currentScript][var];
 	}
 
@@ -417,7 +417,7 @@
 	debugC(DEBUG_VARS, "writeVar(%d, %d)", var, value);
 
 	if (!(var & 0xF0000000)) {
-		checkRange(_numVariables - 1, 0, var, "Variable %d out of range(w)");
+		assertRange(0, var, _numVariables - 1, "variable (writing)");
 
 		if (var == VAR_CHARINC) {
 			if (ConfMan.hasKey("talkspeed")) {
@@ -441,7 +441,7 @@
 
 	if (var & 0x80000000) {
 		var &= 0x7FFFFFFF;
-		checkRange(_numBitVariables - 1, 0, var, "Bit variable %d out of range(w)");
+		assertRange(0, var, _numBitVariables - 1, "bit variable (writing)");
 
 		if (value)
 			_bitVars[var >> 3] |= (1 << (var & 7));
@@ -452,7 +452,7 @@
 
 	if (var & 0x40000000) {
 		var &= 0xFFFFFFF;
-		checkRange(25, 0, var, "Local variable %d out of range(w)");
+		assertRange(0, var, 25, "local variable (writing)");
 		vm.localvar[_currentScript][var] = value;
 		return;
 	}
@@ -934,7 +934,7 @@
 	case 0x6F:		// SO_ACTOR_PALETTE Set actor palette
 		j = pop();
 		i = pop();
-		checkRange(31, 0, i, "Illegal palette slot %d");
+		assertRange(0, i, 31, "o8_actorOps: palette slot");
 		a->setPalette(i, j);
 		break;
 	case 0x70:		// SO_ACTOR_TALK_COLOR Set actor talk color
@@ -1055,7 +1055,7 @@
 	if (subOp == 0x96) {
 		_curVerb = pop();
 		_curVerbSlot = getVerbSlot(_curVerb, 0);
-		checkRange(_numVerbs - 1, 0, _curVerbSlot, "Illegal new verb slot %d");
+		assertRange(0, _curVerbSlot, _numVerbs - 1, "new verb slot");
 		//printf("Setting current actor to %d\n", _curActor);
 		return;
 	}

Modified: scummvm/trunk/engines/scumm/scumm.cpp
===================================================================
--- scummvm/trunk/engines/scumm/scumm.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/scumm.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -1954,13 +1954,13 @@
 #pragma mark -
 
 int ScummEngine_v60he::getHETimer(int timer) {
-	checkRange(15, 1, timer, "getHETimer: Timer out of range(%d)");
+	assertRange(1, timer, 15, "getHETimer: Timer");
 	int time = _system->getMillis() - _heTimers[timer];
 	return time;
 }
 
 void ScummEngine_v60he::setHETimer(int timer) {
-	checkRange(15, 1, timer, "setHETimer: Timer out of range(%d)");
+	assertRange(1, timer, 15, "setHETimer: Timer");
 	_heTimers[timer] = _system->getMillis();
 }
 

Modified: scummvm/trunk/engines/scumm/util.cpp
===================================================================
--- scummvm/trunk/engines/scumm/util.cpp	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/util.cpp	2006-09-16 13:38:43 UTC (rev 23885)
@@ -29,11 +29,9 @@
 #pragma mark --- Utilities ---
 #pragma mark -
 
-void checkRange(int max, int min, int no, const char *str) {
-	if (no < min || no > max) {
-		char buf[256];
-		snprintf(buf, sizeof(buf), str, no);
-		error("Value %d is out of bounds (%d,%d) (%s)", no, min, max, buf);
+void assertRange(int min, int value, int max, const char *desc) {
+	if (value < min || value > max) {
+		error("%s %d is out of bounds (%d,%d)", desc, value, min, max);
 	}
 }
 

Modified: scummvm/trunk/engines/scumm/util.h
===================================================================
--- scummvm/trunk/engines/scumm/util.h	2006-09-16 12:43:18 UTC (rev 23884)
+++ scummvm/trunk/engines/scumm/util.h	2006-09-16 13:38:43 UTC (rev 23885)
@@ -38,7 +38,7 @@
 int fromSimpleDir(int dirtype, int dir);
 int toSimpleDir(int dirtype, int dir);
 
-void checkRange(int max, int min, int no, const char *str);
+void assertRange(int min, int value, int max, const char *desc);
 
 const char *tag2str(uint32 tag);
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list