aboutsummaryrefslogtreecommitdiff
path: root/engines
diff options
context:
space:
mode:
authorlukaslw2014-10-09 17:54:46 +0200
committerlukaslw2014-10-09 17:54:46 +0200
commit9db1d77d478fafe1ecd2d2723d4374a80bfba90e (patch)
tree3091d7eca9a5edf4eec14880bb22f7b80d035c0b /engines
parentf80d72d10ae78c29b21a4ca0533a2f1b05e729ac (diff)
downloadscummvm-rg350-9db1d77d478fafe1ecd2d2723d4374a80bfba90e.tar.gz
scummvm-rg350-9db1d77d478fafe1ecd2d2723d4374a80bfba90e.tar.bz2
scummvm-rg350-9db1d77d478fafe1ecd2d2723d4374a80bfba90e.zip
PRINCE: Remove readScript and LittleEndianReader templates from Script
Change all of READ_UINT16 to READ_LE_UINT16 and all READ_UINT32 to READ_LE_UINT32 for endian-safety
Diffstat (limited to 'engines')
-rw-r--r--engines/prince/hero.cpp18
-rw-r--r--engines/prince/prince.cpp76
-rw-r--r--engines/prince/script.cpp103
-rw-r--r--engines/prince/script.h20
4 files changed, 111 insertions, 106 deletions
diff --git a/engines/prince/hero.cpp b/engines/prince/hero.cpp
index 69afb06ea2..8c13c6d1f9 100644
--- a/engines/prince/hero.cpp
+++ b/engines/prince/hero.cpp
@@ -462,18 +462,18 @@ void Hero::showHeroShadow(Graphics::Surface *screen, DrawNode *drawNode) {
}
//skip_line
//next_line
- if (READ_UINT16(shadowLineStart + 2) < READ_UINT16(shadowLineStart - 2)) {
+ if (READ_LE_UINT16(shadowLineStart + 2) < READ_LE_UINT16(shadowLineStart - 2)) {
//minus_y
shadBitAddr -= PrinceEngine::kMaxPicWidth / 8;
shadPosY--;
diffY--;
- } else if (READ_UINT16(shadowLineStart + 2) > READ_UINT16(shadowLineStart - 2)) {
+ } else if (READ_LE_UINT16(shadowLineStart + 2) > READ_LE_UINT16(shadowLineStart - 2)) {
shadBitAddr += PrinceEngine::kMaxPicWidth / 8;
shadPosY++;
diffY++;
}
//no_change_y
- if (READ_UINT16(shadowLineStart) < READ_UINT16(shadowLineStart - 4)) {
+ if (READ_LE_UINT16(shadowLineStart) < READ_LE_UINT16(shadowLineStart - 4)) {
//minus_x
shadPosX--;
//rol
@@ -484,7 +484,7 @@ void Hero::showHeroShadow(Graphics::Surface *screen, DrawNode *drawNode) {
shadBitMask <<= 1;
}
diffX--;
- } else if (READ_UINT16(shadowLineStart) > READ_UINT16(shadowLineStart - 4)) {
+ } else if (READ_LE_UINT16(shadowLineStart) > READ_LE_UINT16(shadowLineStart - 4)) {
shadPosX++;
//ror
if (shadBitMask == 1) {
@@ -752,9 +752,9 @@ void Hero::showHero() {
//go_for_it:
while (1) {
if (_currCoords != nullptr) {
- if (READ_UINT32(_currCoords) != 0xFFFFFFFF) {
- x = READ_UINT16(_currCoords);
- y = READ_UINT16(_currCoords + 2);
+ if (READ_LE_UINT32(_currCoords) != 0xFFFFFFFF) {
+ x = READ_LE_UINT16(_currCoords);
+ y = READ_LE_UINT16(_currCoords + 2);
_currCoords += 4;
dir = *_currDirTab;
_currDirTab++;
@@ -801,8 +801,8 @@ void Hero::showHero() {
}
} else {
//finito
- _middleX = READ_UINT16(_currCoords - 4);
- _middleY = READ_UINT16(_currCoords - 2);
+ _middleX = READ_LE_UINT16(_currCoords - 4);
+ _middleY = READ_LE_UINT16(_currCoords - 2);
selectZoom();
if (_coords != nullptr) {
diff --git a/engines/prince/prince.cpp b/engines/prince/prince.cpp
index a0cb863dc2..98398e8d4e 100644
--- a/engines/prince/prince.cpp
+++ b/engines/prince/prince.cpp
@@ -2160,8 +2160,8 @@ void PrinceEngine::prepareInventoryToView() {
tempMobItem._name = "";
tempMobItem._examText = "";
- int txtOffset = READ_UINT32(&_invTxt[itemNr * 8]);
- int examTxtOffset = READ_UINT32(&_invTxt[itemNr * 8 + 4]);
+ int txtOffset = READ_LE_UINT32(&_invTxt[itemNr * 8]);
+ int examTxtOffset = READ_LE_UINT32(&_invTxt[itemNr * 8 + 4]);
stream.seek(txtOffset);
while ((c = stream.readByte())) {
@@ -2747,7 +2747,7 @@ void PrinceEngine::displayInventory() {
void PrinceEngine::createDialogBox(int dialogBoxNr) {
_dialogLines = 0;
int amountOfDialogOptions = 0;
- int dialogDataValue = (int)READ_UINT32(_dialogData);
+ int dialogDataValue = (int)READ_LE_UINT32(_dialogData);
byte c;
int sentenceNumber;
@@ -2799,7 +2799,7 @@ void PrinceEngine::runDialog() {
byte *dialogText = _dialogText;
byte *dialogCurrentText = nullptr;
int dialogSelected = -1;
- int dialogDataValue = (int)READ_UINT32(_dialogData);
+ int dialogDataValue = (int)READ_LE_UINT32(_dialogData);
while ((sentenceNumber = *dialogText) != 0xFF) {
dialogText++;
@@ -2871,7 +2871,7 @@ void PrinceEngine::dialogLeftMouseButton(byte *string, int dialogSelected) {
_interpreter->setString(string);
talkHero(0);
- int dialogDataValue = (int)READ_UINT32(_dialogData);
+ int dialogDataValue = (int)READ_LE_UINT32(_dialogData);
dialogDataValue |= (1u << dialogSelected);
WRITE_UINT32(_dialogData, dialogDataValue);
@@ -3776,7 +3776,7 @@ int PrinceEngine::cpe() {
if ((*(_checkBitmap + kPBW) & _checkMask)) {
switch (_checkMask) {
case 128:
- value = READ_UINT16(_checkBitmap - 1);
+ value = READ_LE_UINT16(_checkBitmap - 1);
value &= 0x4001;
if (value != 0x4001) {
return 0;
@@ -3825,7 +3825,7 @@ int PrinceEngine::cpe() {
}
break;
case 1:
- value = READ_UINT16(_checkBitmap);
+ value = READ_LE_UINT16(_checkBitmap);
value &= 0x8002;
if (value != 0x8002) {
return 0;
@@ -4130,8 +4130,8 @@ bool PrinceEngine::tracePath(int x1, int y1, int x2, int y2) {
} else if (drawLineFlag == -1 && _traceLineLen >= 2) {
byte *tempCorrds = bcad;
while (tempCorrds != _coords) {
- x = READ_UINT16(tempCorrds);
- y = READ_UINT16(tempCorrds + 2);
+ x = READ_LE_UINT16(tempCorrds);
+ y = READ_LE_UINT16(tempCorrds + 2);
tempCorrds += 4;
specialPlot2(x, y);
}
@@ -4203,13 +4203,13 @@ bool PrinceEngine::tracePath(int x1, int y1, int x2, int y2) {
byte *tempCoords = _coords;
tempCoords -= 4;
if (tempCoords > _coordsBuf) {
- int tempX = READ_UINT16(tempCoords);
- int tempY = READ_UINT16(tempCoords + 2);
+ int tempX = READ_LE_UINT16(tempCoords);
+ int tempY = READ_LE_UINT16(tempCoords + 2);
if (_checkX == tempX && _checkY == tempY) {
_coords = tempCoords;
}
- x = READ_UINT16(tempCoords);
- y = READ_UINT16(tempCoords + 2);
+ x = READ_LE_UINT16(tempCoords);
+ y = READ_LE_UINT16(tempCoords + 2);
} else {
return false;
}
@@ -4261,10 +4261,10 @@ void PrinceEngine::approxPath() {
if (tempCoordsBuf != tempCoords) {
tempCoords -= 4; // last point on path
while (tempCoordsBuf != tempCoords) {
- x1 = READ_UINT16(tempCoords);
- y1 = READ_UINT16(tempCoords + 2);
- x2 = READ_UINT16(tempCoordsBuf);
- y2 = READ_UINT16(tempCoordsBuf + 2);
+ x1 = READ_LE_UINT16(tempCoords);
+ y1 = READ_LE_UINT16(tempCoords + 2);
+ x2 = READ_LE_UINT16(tempCoordsBuf);
+ y2 = READ_LE_UINT16(tempCoordsBuf + 2);
tempCoordsBuf += 4;
//TracePoint
oldCoords = _coords2;
@@ -4273,8 +4273,8 @@ void PrinceEngine::approxPath() {
WRITE_UINT16(_coords2 + 2, y1);
_coords2 += 4;
} else {
- int testX = READ_UINT16(_coords2 - 4);
- int testY = READ_UINT16(_coords2 - 2);
+ int testX = READ_LE_UINT16(_coords2 - 4);
+ int testY = READ_LE_UINT16(_coords2 - 2);
if (testX != x1 || testY != y1) {
WRITE_UINT16(_coords2, x1);
WRITE_UINT16(_coords2 + 2, y1);
@@ -4315,8 +4315,8 @@ int PrinceEngine::scanDirectionsFindNext(byte *tempCoordsBuf, int xDiff, int yDi
}
while (1) {
- againPointX1 = READ_UINT16(tempCoordsBuf);
- againPointY1 = READ_UINT16(tempCoordsBuf + 2);
+ againPointX1 = READ_LE_UINT16(tempCoordsBuf);
+ againPointY1 = READ_LE_UINT16(tempCoordsBuf + 2);
tempCoordsBuf += 4;
if (tempCoordsBuf == _coords) {
@@ -4324,8 +4324,8 @@ int PrinceEngine::scanDirectionsFindNext(byte *tempCoordsBuf, int xDiff, int yDi
break;
}
- dX = againPointX1 - READ_UINT16(tempCoordsBuf);
- dY = againPointY1 - READ_UINT16(tempCoordsBuf + 2);
+ dX = againPointX1 - READ_LE_UINT16(tempCoordsBuf);
+ dY = againPointY1 - READ_LE_UINT16(tempCoordsBuf + 2);
if (dX != xDiff) {
direction = tempY;
@@ -4352,14 +4352,14 @@ void PrinceEngine::scanDirections() {
int x1, y1, x2, y2, xDiff, yDiff;
while (1) {
- x1 = READ_UINT16(tempCoordsBuf);
- y1 = READ_UINT16(tempCoordsBuf + 2);
+ x1 = READ_LE_UINT16(tempCoordsBuf);
+ y1 = READ_LE_UINT16(tempCoordsBuf + 2);
tempCoordsBuf += 4;
if (tempCoordsBuf == _coords) {
break;
}
- x2 = READ_UINT16(tempCoordsBuf);
- y2 = READ_UINT16(tempCoordsBuf + 2);
+ x2 = READ_LE_UINT16(tempCoordsBuf);
+ y2 = READ_LE_UINT16(tempCoordsBuf + 2);
xDiff = x1 - x2;
yDiff = y1 - y2;
@@ -4420,8 +4420,8 @@ void PrinceEngine::moveShandria() {
_secondHero->freeHeroAnim();
_secondHero->freeOldMove();
byte *shanCoords = _mainHero->_currCoords + shanLen1 * 4 - 4;
- int shanX = READ_UINT16(shanCoords - 4);
- int shanY = READ_UINT16(shanCoords - 2);
+ int shanX = READ_LE_UINT16(shanCoords - 4);
+ int shanY = READ_LE_UINT16(shanCoords - 2);
int xDiff = shanX - _secondHero->_middleX;
if (xDiff < 0) {
xDiff *= -1;
@@ -4440,8 +4440,8 @@ void PrinceEngine::moveShandria() {
if (shanCoords == _mainHero->_currCoords) {
break;
}
- int x = READ_UINT16(shanCoords);
- int y = READ_UINT16(shanCoords + 2);
+ int x = READ_LE_UINT16(shanCoords);
+ int y = READ_LE_UINT16(shanCoords + 2);
int pointDiffX = x - shanX;
if (pointDiffX < 0) {
pointDiffX *= -1;
@@ -4459,8 +4459,8 @@ void PrinceEngine::moveShandria() {
int pathSizeDiff = (shanCoords - _mainHero->_currCoords) / 4;
int destDir = *(_mainHero->_currDirTab + pathSizeDiff);
_secondHero->_destDirection = destDir;
- int destX = READ_UINT16(shanCoords);
- int destY = READ_UINT16(shanCoords + 2);
+ int destX = READ_LE_UINT16(shanCoords);
+ int destY = READ_LE_UINT16(shanCoords + 2);
_secondHero->_coords = makePath(kSecondHero, _secondHero->_middleX, _secondHero->_middleY, destX, destY);
if (_secondHero->_coords != nullptr) {
_secondHero->_currCoords = _secondHero->_coords;
@@ -4571,15 +4571,15 @@ byte *PrinceEngine::makePath(int heroId, int currX, int currY, int destX, int de
if (choosenLength) {
if (chosenCoordsBuf != nullptr) {
- int tempXBegin = READ_UINT16(chosenCoordsBuf);
- int tempYBegin = READ_UINT16(chosenCoordsBuf + 2);
+ int tempXBegin = READ_LE_UINT16(chosenCoordsBuf);
+ int tempYBegin = READ_LE_UINT16(chosenCoordsBuf + 2);
if (stX != tempXBegin || stY != tempYBegin) {
SWAP(chosenCoordsBuf, choosenCoords);
chosenCoordsBuf -= 4;
int cord;
byte *tempCoordsBuf = _coordsBuf;
while (1) {
- cord = READ_UINT32(chosenCoordsBuf);
+ cord = READ_LE_UINT32(chosenCoordsBuf);
WRITE_UINT32(tempCoordsBuf, cord);
tempCoordsBuf += 4;
if (chosenCoordsBuf == choosenCoords) {
@@ -4611,10 +4611,10 @@ byte *PrinceEngine::makePath(int heroId, int currX, int currY, int destX, int de
newCoords = (byte *)malloc(normCoordsSize);
newCoordsBegin = newCoords;
while (tempCoordsBuf != tempCoords) {
- newValueX = READ_UINT16(tempCoordsBuf);
+ newValueX = READ_LE_UINT16(tempCoordsBuf);
WRITE_UINT16(newCoords, newValueX * 2);
newCoords += 2;
- newValueY = READ_UINT16(tempCoordsBuf + 2);
+ newValueY = READ_LE_UINT16(tempCoordsBuf + 2);
WRITE_UINT16(newCoords, newValueY * 2);
newCoords += 2;
tempCoordsBuf += 4;
diff --git a/engines/prince/script.cpp b/engines/prince/script.cpp
index 09f45af2e4..0181f6912d 100644
--- a/engines/prince/script.cpp
+++ b/engines/prince/script.cpp
@@ -135,16 +135,28 @@ bool Script::loadStream(Common::SeekableReadStream &stream) {
return true;
}
+uint16 Script::readScript16(uint32 address) {
+ assert((_data + address + sizeof(uint16)) <= (_data + _dataSize));
+ uint16 data = READ_LE_UINT16(&_data[address]);
+ return data;
+}
+
+uint32 Script::readScript32(uint32 address) {
+ assert((_data + address + sizeof(uint32)) <= (_data + _dataSize));
+ uint32 data = READ_LE_UINT32(&_data[address]);
+ return data;
+}
+
int16 Script::getLightX(int locationNr) {
- return (int)READ_UINT16(&_data[_scriptInfo.lightSources + locationNr * 8]);
+ return (int)READ_LE_UINT16(&_data[_scriptInfo.lightSources + locationNr * 8]);
}
int16 Script::getLightY(int locationNr) {
- return (int)READ_UINT16(&_data[_scriptInfo.lightSources + locationNr * 8 + 2]);
+ return (int)READ_LE_UINT16(&_data[_scriptInfo.lightSources + locationNr * 8 + 2]);
}
int32 Script::getShadowScale(int locationNr) {
- return (int)READ_UINT16(&_data[_scriptInfo.lightSources + locationNr * 8 + 4]);
+ return (int)READ_LE_UINT16(&_data[_scriptInfo.lightSources + locationNr * 8 + 4]);
}
uint32 Script::getStartGameOffset() {
@@ -152,7 +164,7 @@ uint32 Script::getStartGameOffset() {
}
uint32 Script::getLocationInitScript(int initRoomTableOffset, int roomNr) {
- return (uint32)READ_UINT32(&_data[initRoomTableOffset + roomNr * 4]);
+ return (uint32)READ_LE_UINT32(&_data[initRoomTableOffset + roomNr * 4]);
}
byte Script::getMobVisible(int roomMobOffset, uint16 mob) {
@@ -193,7 +205,7 @@ uint8 *Script::getHeroAnimName(int offset) {
}
uint32 Script::getBackAnimId(int roomBackAnimOffset, int slot) {
- uint32 animId = READ_UINT32(&_data[roomBackAnimOffset + slot * 4]);
+ uint32 animId = READ_LE_UINT32(&_data[roomBackAnimOffset + slot * 4]);
return animId;
}
@@ -215,9 +227,9 @@ int Script::scanMobEvents(int mobMask, int dataEventOffset) {
int16 mob;
int32 code;
do {
- mob = (int)READ_UINT16(&_data[dataEventOffset + i * 6]);
+ mob = (int)READ_LE_UINT16(&_data[dataEventOffset + i * 6]);
if (mob == mobMask) {
- code = (int)READ_UINT32(&_data[dataEventOffset + i * 6 + 2]);
+ code = (int)READ_LE_UINT32(&_data[dataEventOffset + i * 6 + 2]);
debug("code: %d", code);
return code;
}
@@ -233,11 +245,11 @@ int Script::scanMobEventsWithItem(int mobMask, int dataEventOffset, int itemMask
int16 item;
int32 code;
do {
- mob = (int)READ_UINT16(&_data[dataEventOffset + i * 8]);
+ mob = (int)READ_LE_UINT16(&_data[dataEventOffset + i * 8]);
if (mob == mobMask) {
- item = (int)READ_UINT16(&_data[dataEventOffset + i * 8 + 2]);
+ item = (int)READ_LE_UINT16(&_data[dataEventOffset + i * 8 + 2]);
if (item == itemMask) {
- code = (int)READ_UINT32(&_data[dataEventOffset + i * 8 + 4]);
+ code = (int)READ_LE_UINT32(&_data[dataEventOffset + i * 8 + 4]);
debug("itemMask: %d", item);
debug("code: %d", code);
return code;
@@ -256,8 +268,8 @@ void Script::installSingleBackAnim(Common::Array<BackgroundAnim> &backAnimList,
BackgroundAnim newBackgroundAnim; // BackgroundAnim seq data and its array of Anim
- int animOffset = READ_UINT32(&_data[offset]); // pos of BackgroundAnim data in script
- int anims = READ_UINT32(&_data[animOffset + 8]); // amount of Anim in BackgroundAnim
+ int animOffset = READ_LE_UINT32(&_data[offset]); // pos of BackgroundAnim data in script
+ int anims = READ_LE_UINT32(&_data[animOffset + 8]); // amount of Anim in BackgroundAnim
if (anims == 0) {
anims = 1; // anims with 0 as amount in game data has just 1 animation
@@ -463,7 +475,7 @@ uint32 Interpreter::step(uint32 opcodePC) {
_lastInstruction = _currentInstruction;
// Get the current opcode
- _lastOpcode = readScript<uint16>();
+ _lastOpcode = readScript16();
if (_lastOpcode > kNumOpcodes)
error(
@@ -538,15 +550,20 @@ void Interpreter::setFgOpcodePC(uint32 value) {
_fgOpcodePC = value;
}
-template <typename T>
-T Interpreter::readScript() {
- T data = _script->read<T>(_currentInstruction);
- _currentInstruction += sizeof(data);
+uint16 Interpreter::readScript16() {
+ uint16 data = _script->readScript16(_currentInstruction);
+ _currentInstruction += sizeof(uint16);
+ return data;
+}
+
+uint32 Interpreter::readScript32() {
+ uint32 data = _script->readScript32(_currentInstruction);
+ _currentInstruction += sizeof(uint32);
return data;
}
int32 Interpreter::readScriptFlagValue() {
- uint16 value = readScript<uint16>();
+ uint16 value = readScript16();
if (value & InterpreterFlags::kFlagMask) {
return _flags->getFlagValue((Flags::Id)value);
}
@@ -554,7 +571,7 @@ int32 Interpreter::readScriptFlagValue() {
}
Flags::Id Interpreter::readScriptFlagId() {
- return (Flags::Id)readScript<uint16>();
+ return (Flags::Id)readScript16();
}
void Interpreter::O_WAITFOREVER() {
@@ -583,7 +600,7 @@ void Interpreter::O_INITROOM() {
void Interpreter::O_SETSAMPLE() {
int32 sampleId = readScriptFlagValue();
- int32 sampleNameOffset = readScript<uint32>();
+ int32 sampleNameOffset = readScript32();
const char *sampleName = _script->getString(_currentInstruction + sampleNameOffset - 4);
_vm->loadSample(sampleId, sampleName);
debugInterpreter("O_SETSAMPLE %d %s", sampleId, sampleName);
@@ -597,7 +614,7 @@ void Interpreter::O_FREESAMPLE() {
void Interpreter::O_PLAYSAMPLE() {
int32 sampleId = readScriptFlagValue();
- uint16 loopType = readScript<uint16>();
+ uint16 loopType = readScript16();
_vm->playSample(sampleId, loopType);
debugInterpreter("O_PLAYSAMPLE sampleId %d loopType %d", sampleId, loopType);
}
@@ -697,7 +714,7 @@ void Interpreter::O_CHECKANIMFRAME() {
void Interpreter::O_PUTBACKANIM() {
int32 roomId = readScriptFlagValue();
int32 slot = readScriptFlagValue();
- int32 animId = readScript<uint32>();
+ int32 animId = readScript32();
Room *room = new Room();
room->loadRoom(_script->getRoomOffset(roomId));
_vm->_script->setBackAnimId(room->_backAnim, slot, animId);
@@ -738,7 +755,7 @@ void Interpreter::O_FREEALLSAMPLES() {
}
void Interpreter::O_SETMUSIC() {
- uint16 musicId = readScript<uint16>();
+ uint16 musicId = readScript16();
_vm->loadMusic(musicId);
debugInterpreter("O_SETMUSIC musicId %d", musicId);
}
@@ -787,7 +804,7 @@ void Interpreter::O_CLS() {
}
void Interpreter::O__CALL() {
- int32 address = readScript<uint32>();
+ int32 address = readScript32();
_stack[_stacktop] = _currentInstruction;
_stacktop++;
_currentInstruction += address - 4;
@@ -805,7 +822,7 @@ void Interpreter::O_RETURN() {
}
void Interpreter::O_GO() {
- int32 opPC = readScript<uint32>();
+ int32 opPC = readScript32();
_currentInstruction += opPC - 4;
debugInterpreter("O_GO 0x%04X", opPC);
}
@@ -854,7 +871,7 @@ void Interpreter::O_COMPARE() {
}
void Interpreter::O_JUMPZ() {
- int32 offset = readScript<uint32>();
+ int32 offset = readScript32();
if (!_result) {
_currentInstruction += offset - 4;
}
@@ -862,7 +879,7 @@ void Interpreter::O_JUMPZ() {
}
void Interpreter::O_JUMPNZ() {
- int32 offset = readScript<uint32>();
+ int32 offset = readScript32();
if (_result) {
_currentInstruction += offset - 4;
}
@@ -913,7 +930,7 @@ void Interpreter::O_SUBFLAG() {
}
void Interpreter::O_SETSTRING() {
- int32 offset = readScript<uint32>();
+ int32 offset = readScript32();
_currentString = offset;
if (offset >= 80000) {
_string = _vm->_variaTxt->getString(offset - 80000);
@@ -1073,7 +1090,7 @@ void Interpreter::O_CLSTEXT() {
void Interpreter::O_CALLTABLE() {
Flags::Id flagId = readScriptFlagId();
int roomNr = _flags->getFlagValue(flagId);
- int32 tableOffset = readScript<uint32>();
+ int32 tableOffset = readScript32();
int initLocationScript = _script->getLocationInitScript(tableOffset, roomNr);
if (initLocationScript) {
_stack[_stacktop] = _currentInstruction;
@@ -1214,7 +1231,7 @@ void Interpreter::O_WAITTEXT() {
void Interpreter::O_SETHEROANIM() {
int32 heroId = readScriptFlagValue();
- int32 offset = readScript<uint32>();
+ int32 offset = readScript32();
Hero *hero = nullptr;
if (!heroId) {
hero = _vm->_mainHero;
@@ -1323,7 +1340,7 @@ void Interpreter::O_GETANIMDATA() {
}
void Interpreter::O_SETBGCODE() {
- int32 offset = readScript<uint32>();
+ int32 offset = readScript32();
_bgOpcodePC = _currentInstruction + offset - 4;
debugInterpreter("O_SETBGCODE next %08x, offset %08x", _bgOpcodePC, offset);
}
@@ -1340,7 +1357,7 @@ void Interpreter::O_SETBACKFRAME() {
void Interpreter::O_GETRND() {
Flags::Id flag = readScriptFlagId();
- uint16 rndSeed = readScript<uint16>();
+ uint16 rndSeed = readScript16();
int value = _vm->_randomSource.getRandomNumber(rndSeed - 1);
_flags->setFlagValue(flag, value);
debugInterpreter("O_GETRND flag %d, rndSeed %d, value %d", flag, rndSeed, value);
@@ -1355,7 +1372,7 @@ void Interpreter::O_TALKBACKANIM() {
// Simplifying, because used only once in Location 20
void Interpreter::O_LOADPATH() {
- readScript<uint32>();
+ readScript32();
_vm->loadPath("path2");
debugInterpreter("O_LOADPATH - path2");
}
@@ -1369,7 +1386,7 @@ void Interpreter::O_GETCHAR() {
void Interpreter::O_SETDFLAG() {
Flags::Id flagId = readScriptFlagId();
- int32 address = readScript<uint32>();
+ int32 address = readScript32();
_flags->setFlagValue((Flags::Id)(flagId), _currentInstruction + address - 4);
debugInterpreter("O_SETDFLAG 0x%04X (%s) = 0x%04X", flagId, Flags::getFlagName(flagId), _currentInstruction + address - 4);
}
@@ -1482,7 +1499,7 @@ void Interpreter::O_INITDIALOG() {
byte *stringCurrOff = _string;
byte *string = _string;
stringCurrOff++;
- int32 adressOfFirstSequence = (int)READ_UINT16(stringCurrOff);
+ int32 adressOfFirstSequence = (int)READ_LE_UINT16(stringCurrOff);
stringCurrOff += 2;
_string = string + adressOfFirstSequence;
@@ -1499,7 +1516,7 @@ void Interpreter::O_INITDIALOG() {
byte *line = nullptr;
int dialogBox = 0;
- while ((off = (int)READ_UINT16(stringCurrOff)) != -1) {
+ while ((off = (int)READ_LE_UINT16(stringCurrOff)) != -1) {
stringCurrOff += 2;
if (off) {
line = string + off;
@@ -1510,7 +1527,7 @@ void Interpreter::O_INITDIALOG() {
stringCurrOff += 2;
int dialogOpt = 0;
- while ((off = (int)READ_UINT16(stringCurrOff)) != -1) {
+ while ((off = (int)READ_LE_UINT16(stringCurrOff)) != -1) {
stringCurrOff += 2;
if (off) {
line = string + off;
@@ -1548,7 +1565,7 @@ void Interpreter::O_INITDIALOG() {
void Interpreter::O_ENABLEDIALOGOPT() {
int32 opt = readScriptFlagValue();
- int dialogDataValue = (int)READ_UINT32(_vm->_dialogData);
+ int dialogDataValue = (int)READ_LE_UINT32(_vm->_dialogData);
dialogDataValue &= ~(1u << opt);
WRITE_UINT32(_vm->_dialogData, dialogDataValue);
debugInterpreter("O_ENABLEDIALOGOPT opt %d", opt);
@@ -1556,7 +1573,7 @@ void Interpreter::O_ENABLEDIALOGOPT() {
void Interpreter::O_DISABLEDIALOGOPT() {
int32 opt = readScriptFlagValue();
- int dialogDataValue = (int)READ_UINT32(_vm->_dialogData);
+ int dialogDataValue = (int)READ_LE_UINT32(_vm->_dialogData);
dialogDataValue |= (1u << opt);
WRITE_UINT32(_vm->_dialogData, dialogDataValue);
debugInterpreter("O_DISABLEDIALOGOPT opt %d", opt);
@@ -1584,7 +1601,7 @@ void Interpreter::O_STOPSAMPLE() {
void Interpreter::O_BACKANIMRANGE() {
int32 slotId = readScriptFlagValue();
- uint16 animId = readScript<uint16>();
+ uint16 animId = readScript16();
int32 low = readScriptFlagValue();
int32 high = readScriptFlagValue();
if (animId != 0xFFFF) {
@@ -1672,7 +1689,7 @@ void Interpreter::O_POPSTRING() {
}
void Interpreter::O_SETFGCODE() {
- int32 offset = readScript<uint32>();
+ int32 offset = readScript32();
_fgOpcodePC = _currentInstruction + offset - 4;
debugInterpreter("O_SETFGCODE next %08x, offset %08x", _fgOpcodePC, offset);
}
@@ -1723,8 +1740,8 @@ void Interpreter::O_RUNHERO() {
}
void Interpreter::O_SETBACKANIMDATA() {
- uint16 animNumber = readScript<uint16>();
- uint16 animDataOffset = readScript<uint16>();
+ uint16 animNumber = readScript16();
+ uint16 animDataOffset = readScript16();
Flags::Id flagId = readScriptFlagId();
uint16 value = _flags->getFlagValue((Flags::Id)(flagId));
int currAnim = _vm->_backAnimList[animNumber]._seq._currRelative;
diff --git a/engines/prince/script.h b/engines/prince/script.h
index fa7f9b7469..4799e84944 100644
--- a/engines/prince/script.h
+++ b/engines/prince/script.h
@@ -39,13 +39,6 @@ struct Anim;
struct BackgroundAnim;
struct Mask;
-// TODO - change this to sth else?
-namespace Detail {
- template <typename T> T LittleEndianReader(void *data);
- template <> inline uint16 LittleEndianReader<uint16>(void *data) { return READ_LE_UINT16(data); }
- template <> inline uint32 LittleEndianReader<uint32>(void *data) { return READ_LE_UINT32(data); }
-}
-
class Room {
public:
Room();
@@ -125,11 +118,8 @@ public:
bool loadStream(Common::SeekableReadStream &stream);
- template <typename T>
- T read(uint32 address) {
- assert((_data + address + sizeof(T)) <= (_data + _dataSize));
- return Detail::LittleEndianReader<T>(&_data[address]);
- }
+ uint16 readScript16(uint32 address);
+ uint32 readScript32(uint32 address);
uint32 getStartGameOffset();
uint32 getLocationInitScript(int initRoomTableOffset, int roomNr);
@@ -240,14 +230,12 @@ private:
// Helper functions
uint32 step(uint32 opcodePC);
-
+ uint16 readScript16();
+ uint32 readScript32();
int32 readScriptFlagValue();
Flags::Id readScriptFlagId();
int checkSeq(byte *string);
- // instantiation not needed here
- template <typename T> T readScript();
-
void debugInterpreter(const char *s, ...);
typedef void (Interpreter::*OpcodeFunc)();