diff options
author | Kari Salminen | 2008-06-26 15:16:15 +0000 |
---|---|---|
committer | Kari Salminen | 2008-06-26 15:16:15 +0000 |
commit | 68c598a3f3ad904dab66a2219d97af9eb5a02ccd (patch) | |
tree | 6fa8faef7ba7049ac9b0415e2478907f4355bade | |
parent | 358d2ec44ab9a3d98afcf8419b1613c018688d0b (diff) | |
download | scummvm-rg350-68c598a3f3ad904dab66a2219d97af9eb5a02ccd.tar.gz scummvm-rg350-68c598a3f3ad904dab66a2219d97af9eb5a02ccd.tar.bz2 scummvm-rg350-68c598a3f3ad904dab66a2219d97af9eb5a02ccd.zip |
Comments update.
svn-id: r32801
-rw-r--r-- | engines/cine/script_fw.cpp | 6 | ||||
-rw-r--r-- | engines/cine/script_os.cpp | 2 |
2 files changed, 6 insertions, 2 deletions
diff --git a/engines/cine/script_fw.cpp b/engines/cine/script_fw.cpp index 148e673095..f84904ccc5 100644 --- a/engines/cine/script_fw.cpp +++ b/engines/cine/script_fw.cpp @@ -1760,6 +1760,12 @@ int16 getZoneFromPositionRaw(byte *page, int16 x, int16 y, int16 width) { return zoneVar; } +/*! + * \todo Check whether updating the zoneQuery table is appropriate every time + * this function is called because this function is called also from elsewhere + * than simply o1_checkCollision (e.g. from addAni). In Operation Stealth's + * disassembly this probably isn't the case, so there may be weird side-effects. + */ int16 checkCollision(int16 objIdx, int16 x, int16 y, int16 numZones, int16 zoneIdx) { int16 lx = objectTable[objIdx].x + x; int16 ly = objectTable[objIdx].y + y; diff --git a/engines/cine/script_os.cpp b/engines/cine/script_os.cpp index f958c66fad..62a5cc6aaa 100644 --- a/engines/cine/script_os.cpp +++ b/engines/cine/script_os.cpp @@ -650,8 +650,6 @@ int FWScript::o2_loadBg() { return 0; } -/*! \todo Check the current implementation for correctness - */ int FWScript::o2_wasZoneChecked() { byte param = getNextByte(); _compare = (param < NUM_MAX_ZONE && zoneQuery[param]) ? 1 : 0; |