Adding various FIXME comments to bogus (non-self-explanatory) existing FIXME comments; and some other related cleanup

svn-id: r28890

6 files changed, 11 insertions, 3 deletions

diff --git a/engines/agi/loader_preagi.cpp b/engines/agi/loader_preagi.cpp
index 09bdf9f921..7d155b961f 100644
--- a/engines/agi/loader_preagi.cpp
+++ b/engines/agi/loader_preagi.cpp
@@ -273,6 +273,8 @@ 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].flags |= RES_LOADED;
 		} else {
diff --git a/engines/agi/loader_v2.cpp b/engines/agi/loader_v2.cpp
index 593ca40384..8131efbca1 100644
--- a/engines/agi/loader_v2.cpp
+++ b/engines/agi/loader_v2.cpp
@@ -174,9 +174,7 @@ uint8 *AgiLoader_v2::loadVolRes(struct AgiDir *agid) {
 			}
 		} else {
 #if 0
-			/* FIXME: call some panic handler instead of
-			 *        deiniting directly
-			 */
+			// FIXME: call some panic handler instead of deiniting directly
 			deinitVideoMode();
 #endif
 			report("Error: bad signature %04x\n", sig);
diff --git a/engines/agi/loader_v3.cpp b/engines/agi/loader_v3.cpp
index dc419a7afe..8a3a7d2b29 100644
--- a/engines/agi/loader_v3.cpp
+++ b/engines/agi/loader_v3.cpp
@@ -229,6 +229,8 @@ uint8 *AgiLoader_v3::loadVolRes(AgiDir *agid) {
 
 		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
diff --git a/engines/agi/predictive.cpp b/engines/agi/predictive.cpp
index eef4360cbf..8da7f22889 100644
--- a/engines/agi/predictive.cpp
+++ b/engines/agi/predictive.cpp
@@ -199,6 +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
 				if (i == 10 && !_addIsActive) { // Add
 					color2 = 7;
diff --git a/engines/agi/sound.cpp b/engines/agi/sound.cpp
index 64b04784d8..99d29bcc6b 100644
--- a/engines/agi/sound.cpp
+++ b/engines/agi/sound.cpp
@@ -823,6 +823,8 @@ uint32 SoundMgr::mixSound(void) {
 
 				p += (uint32) 118600 *4 / _chn[c].freq;
 
+				// FIXME: Fingolfin asks: why is there a FIXME here? Please either clarify what
+				// needs fixing, or remove it!
 				/* FIXME */
 				if (_chn[c].flags & AGI_SOUND_LOOP) {
 					p %= _chn[c].size << 8;
diff --git a/engines/agi/text.cpp b/engines/agi/text.cpp
index 1d653a9415..bb334d0401 100644
--- a/engines/agi/text.cpp
+++ b/engines/agi/text.cpp
@@ -67,6 +67,8 @@ void AgiEngine::printText2(int l, const char *msg, int foff, int xoff, int yoff,
 		for (m = (const unsigned char *)msg, x1 = y1 = 0; *m; m++) {
 
 			if (*m >= 0x20 || *m == 1 || *m == 2 || *m == 3) {
+				// FIXME: Fingolfin asks: why is there a FIXME here? Please either clarify what
+				// needs fixing, or remove it!
 				/* FIXME */
 				int ypos;