diff options
| author | Eugene Sandulenko | 2007-09-11 15:49:19 +0000 |
|---|---|---|
| committer | Eugene Sandulenko | 2007-09-11 15:49:19 +0000 |
| commit | 35d68621171128c885f5064fb1a3deccb984d95b (patch) | |
| tree | 6b883286a615808eb8d9e9569be4e66f4e3f4039 /engines/agi | |
| parent | 541b68e0e432aefa583cc4777439b86be75fd982 (diff) | |
| download | scummvm-rg350-35d68621171128c885f5064fb1a3deccb984d95b.tar.gz scummvm-rg350-35d68621171128c885f5064fb1a3deccb984d95b.tar.bz2 scummvm-rg350-35d68621171128c885f5064fb1a3deccb984d95b.zip | |
Remove/clarify several FIXMEs
svn-id: r28892
Diffstat (limited to 'engines/agi')
| -rw-r--r-- | engines/agi/loader_preagi.cpp | 4 | ||||
| -rw-r--r-- | engines/agi/loader_v2.cpp | 4 | ||||
| -rw-r--r-- | engines/agi/loader_v3.cpp | 6 | ||||
| -rw-r--r-- | engines/agi/predictive.cpp | 3 |
4 files changed, 2 insertions, 15 deletions
diff --git a/engines/agi/loader_preagi.cpp b/engines/agi/loader_preagi.cpp index 7d155b961f..135472bca3 100644 --- a/engines/agi/loader_preagi.cpp +++ b/engines/agi/loader_preagi.cpp @@ -273,9 +273,7 @@ int AgiLoader_preagi::loadResource(int t, byte* n) { if (n != NULL) { _vm->_game.pictures[0].rdata = n; - // FIXME: Fingolfin asks: why is there a FIXME here? Please either clarify what - // needs fixing, or remove it! - _vm->_game.dirPic[0].len = 4096; //FIXME + _vm->_game.dirPic[0].len = 4096; //FIXME: set up real resource length _vm->_game.dirPic[0].flags |= RES_LOADED; } else { ec = errBadResource; diff --git a/engines/agi/loader_v2.cpp b/engines/agi/loader_v2.cpp index 8131efbca1..c0c6df2fcb 100644 --- a/engines/agi/loader_v2.cpp +++ b/engines/agi/loader_v2.cpp @@ -173,10 +173,6 @@ uint8 *AgiLoader_v2::loadVolRes(struct AgiDir *agid) { abort(); } } else { -#if 0 - // FIXME: call some panic handler instead of deiniting directly - deinitVideoMode(); -#endif report("Error: bad signature %04x\n", sig); // fprintf (stderr, "ACK! BAD RESOURCE!!!\n"); return 0; diff --git a/engines/agi/loader_v3.cpp b/engines/agi/loader_v3.cpp index 8a3a7d2b29..65b9e6592b 100644 --- a/engines/agi/loader_v3.cpp +++ b/engines/agi/loader_v3.cpp @@ -228,12 +228,6 @@ uint8 *AgiLoader_v3::loadVolRes(AgiDir *agid) { fp.read(&x, 7); if (READ_BE_UINT16((uint8 *) x) != 0x1234) { -#if 0 - // FIXME: Fingolfin asks: why is there a FIXME here? Please either clarify what - // needs fixing, or remove it! - /* FIXME */ - deinitVideoMode(); -#endif debugC(3, kDebugLevelResources, "path = %s", path.c_str()); debugC(3, kDebugLevelResources, "offset = %d", agid->offset); debugC(3, kDebugLevelResources, "x = %x %x", x[0], x[1]); diff --git a/engines/agi/predictive.cpp b/engines/agi/predictive.cpp index 8da7f22889..5ffa3d159f 100644 --- a/engines/agi/predictive.cpp +++ b/engines/agi/predictive.cpp @@ -199,9 +199,8 @@ bool AgiEngine::predictiveDialog(void) { color2 = 7; } - // FIXME: Fingolfin asks: why is there a FIXME here? Please either clarify what // needs fixing, or remove it! - bool _addIsActive = false; // FIXME + bool _addIsActive = false; // FIXME: word adding is not implemented if (i == 10 && !_addIsActive) { // Add color2 = 7; } |