From 025a14b32656e651f376e5cc9a5e09c6d02095d1 Mon Sep 17 00:00:00 2001
From: Matthew Hoops
Date: Sun, 7 Nov 2010 02:08:02 +0000
Subject: MOHAWK: Remove console FIXME's

svn-id: r54109
---
 engines/mohawk/console.cpp | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/engines/mohawk/console.cpp b/engines/mohawk/console.cpp
index a9b5928712..bb64401f37 100644
--- a/engines/mohawk/console.cpp
+++ b/engines/mohawk/console.cpp
@@ -545,10 +545,12 @@ bool RivenConsole::Cmd_DumpScript(int argc, const char **argv) {
 
 	// Get CARD/HSPT data and dump their scripts
 	if (!scumm_stricmp(argv[2], "CARD")) {
-		// FIXME: It's weird to use debugN here and not DebugPrintf. But
-		// then again, dumpScript() uses debugN, and it is non-trivial
-		// to modify that to use DebugPrintf. Hence for now we gotta
-		// make do...
+		// Use debugN to print these because the scripts can get very large and would
+		// really be useless if the the text console is not used. A DumpFile could also
+		// theoretically be used, but I (clone2727) typically use this dynamically and
+		// don't want countless files laying around without game context. If one would
+		// want a file of a script they could just redirect stdout to a file or use
+		// deriven.
 		debugN("\n\nDumping scripts for %s\'s card %d!\n", argv[1], (uint16)atoi(argv[3]));
 		debugN("==================================\n\n");
 		Common::SeekableReadStream *cardStream = _vm->getRawData(MKID_BE('CARD'), (uint16)atoi(argv[3]));
@@ -560,7 +562,7 @@ bool RivenConsole::Cmd_DumpScript(int argc, const char **argv) {
 		}
 		delete cardStream;
 	} else if (!scumm_stricmp(argv[2], "HSPT")) {
-		// FIXME: It's weird to use debugN here and not DebugPrintf. See above!
+		// See above for why this is printed via debugN
 		debugN("\n\nDumping scripts for %s\'s card %d hotspots!\n", argv[1], (uint16)atoi(argv[3]));
 		debugN("===========================================\n\n");
 
@@ -583,7 +585,7 @@ bool RivenConsole::Cmd_DumpScript(int argc, const char **argv) {
 		DebugPrintf("%s doesn't have any scripts!\n", argv[2]);
 	}
 
-	// FIXME: It's weird to use debugN here and not DebugPrintf. See above!
+	// See above for why this is printed via debugN
 	debugN("\n\n");
 
 	_vm->changeToStack(oldStack);
-- 
cgit v1.2.3