TODO comment

svn-id: r5305

2 files changed, 7 insertions, 1 deletions

diff --git a/scumm/saveload.cpp b/scumm/saveload.cpp
index f11189b929..228bc09b07 100644
--- a/scumm/saveload.cpp
+++ b/scumm/saveload.cpp
@@ -585,6 +585,10 @@ void Scumm::saveOrLoad(Serializer *s)
 	};
 
 	const SaveLoadEntry stringTabEntries[] = {
+		// TODO - It makes no sense to have all these t_* fields in StringTab
+		// Rather let's dump them all when the save game format changes, and 
+		// keep two StringTab objects: one normal, and a "t_" one.
+		// Then copying them can be done in one line etc.
 		MKLINE(StringTab, xpos, sleInt16),
 		MKLINE(StringTab, t_xpos, sleInt16),
 		MKLINE(StringTab, ypos, sleInt16),
diff --git a/scumm/string.cpp b/scumm/string.cpp
index a8e1dd2ff7..71553db06a 100644
--- a/scumm/string.cpp
+++ b/scumm/string.cpp
@@ -170,6 +170,8 @@ void Scumm::unkMessage2()
 	if (_string[3].color == 0)
 		_string[3].color = 4;
 
+	// TODO - it appears the this function should display the given message graphically
+	// to the user in a "dialog" looking like the pause dialog, i.e. a single line.
 	warning("unkMessage2(\"%s\")", buf);
 	_messagePtr = tmp;
 }
@@ -717,7 +719,7 @@ byte *Scumm::addMessageToStack(byte *msg)
 		return NULL;
 	}
 
-while ((ptr[num++] = chr = *msg++) != 0) {
+	while ((ptr[num++] = chr = *msg++) != 0) {
 		if (num >= 500)
 			error("Message stack overflow");