From 496d19d46176200c3d8486a202382b7016591f5c Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Fri, 7 Nov 2003 14:17:34 +0000
Subject: err, *strnlen(,,3)* used to check for empty strings? I think not :-)

svn-id: r11190
---
 base/main.cpp | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

(limited to 'base/main.cpp')

diff --git a/base/main.cpp b/base/main.cpp
index bfc54b2106..4c335eef7e 100644
--- a/base/main.cpp
+++ b/base/main.cpp
@@ -258,17 +258,15 @@ int main(int argc, char *argv[]) {
 	if (detector.detectMain()) {
 
 		// Set the window caption to the game name
-		prop.caption = ConfMan.get("description", detector._targetName).c_str();
-		if (strnlen(prop.caption, 3) == 0)	
-			prop.caption = detector._game.description;
-		if (strnlen(prop.caption, 3) == 0)	
-			prop.caption = detector._targetName.c_str();
-		if (strnlen(prop.caption, 3) != 0)	
+		Common::String caption(ConfMan.get("description", detector._targetName));
+		if (caption.isEmpty() && detector._game.description)
+			caption = detector._game.description;
+		if (caption.isEmpty())	
+			caption = detector._targetName;
+		if (!caption.isEmpty())	{
+			prop.caption = caption.c_str();
 			system->property(OSystem::PROP_SET_WINDOW_CAPTION, &prop);
-
-		// FIXME: It seem not logical that we first might set the gfx mode to
-		// 1x, and then immediately after might override it again. We probably
-		// should combine both checks into one.
+		}
 
 		// See if the game should default to 1x scaler
 		if (!ConfMan.hasKey("gfx_mode", detector._targetName) && 
-- 
cgit v1.2.3