From 4a15a80eb3fad960e391d7de04edfae16a03f2ea Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Wed, 17 Mar 2004 00:07:43 +0000
Subject: Fix for bug #917532 (GUI: BrowserDialog return value)

svn-id: r13320
---
 backends/wince/CELauncherDialog.cpp | 2 +-
 gui/launcher.cpp                    | 8 ++++----
 gui/options.cpp                     | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/backends/wince/CELauncherDialog.cpp b/backends/wince/CELauncherDialog.cpp
index e8cbdfcd45..2aff8319cf 100644
--- a/backends/wince/CELauncherDialog.cpp
+++ b/backends/wince/CELauncherDialog.cpp
@@ -138,7 +138,7 @@ void CELauncherDialog::automaticScanDirectory(const FilesystemNode *node) {
 
 void CELauncherDialog::addGame() {
 	MessageDialog alert("Do you want to perform an automatic scan ?", "Yes", "No");
-	if (alert.runModal() == 1 && _browser->runModal()) {
+	if (alert.runModal() == 1 && _browser->runModal() > 0) {
 			// Clear existing domains
 			ConfigManager::DomainMap &domains = (ConfigManager::DomainMap&)ConfMan.getGameDomains();
 			domains.clear();
diff --git a/gui/launcher.cpp b/gui/launcher.cpp
index 0ab29f97c9..50a1a4340a 100644
--- a/gui/launcher.cpp
+++ b/gui/launcher.cpp
@@ -399,7 +399,7 @@ void LauncherDialog::addGame() {
 	//    game, e.g. 'Monkey German' and 'Monkey English') and set default
 	//    options for that game.
 
-	if (_browser->runModal()) {
+	if (_browser->runModal() > 0) {
 		// User made his choice...
 		FilesystemNode *dir = _browser->getResult();
 		FSList *files = dir->listDir(FilesystemNode::kListFilesOnly);
@@ -476,7 +476,7 @@ void LauncherDialog::addGame() {
 
 			// Display edit dialog for the new entry
 			EditGameDialog editDialog(domain, result);
-			if (editDialog.runModal()) {
+			if (editDialog.runModal() > 0) {
 				// User pressed OK, so make changes permanent
 
 				// Write config to disk
@@ -496,7 +496,7 @@ void LauncherDialog::addGame() {
 void LauncherDialog::removeGame(int item) {
 	MessageDialog alert("Do you really want to remove this game configuration?", "Yes", "No");
 	
-	if (alert.runModal() == 1) {
+	if (alert.runModal() > 0) {
 		// Remove the currently selected game from the list
 		assert(item >= 0);
 		ConfMan.removeGameDomain(_domains[item]);
@@ -522,7 +522,7 @@ void LauncherDialog::editGame(int item) {
 	if (gameId.isEmpty())
 		gameId = _domains[item];
 	EditGameDialog editDialog(_domains[item], GameDetector::findGame(gameId));
-	if (editDialog.runModal()) {
+	if (editDialog.runModal() > 0) {
 		// User pressed OK, so make changes permanent
 
 		// Write config to disk
diff --git a/gui/options.cpp b/gui/options.cpp
index cb42735246..c96f23dc67 100644
--- a/gui/options.cpp
+++ b/gui/options.cpp
@@ -427,7 +427,7 @@ void GlobalOptionsDialog::close() {
 void GlobalOptionsDialog::handleCommand(CommandSender *sender, uint32 cmd, uint32 data) {
 	switch (cmd) {
 	case kChooseSaveDirCmd:
-		if (_browser->runModal()) {
+		if (_browser->runModal() > 0) {
 			// User made his choice...
 			FilesystemNode *dir = _browser->getResult();
 			_savePath->setLabel(dir->path());
-- 
cgit v1.2.3