From ee70457667b7480226b3af3bf3047e098a0344c5 Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Wed, 11 Aug 2004 21:49:58 +0000
Subject: Fix for bug #1007093 (GUI: removing game configuration from Launcher
 buggy); moral: either document how things work, or at least provide well
 named constants for certain return values... using the cheap way out here,
 i.e. method (b) :-)

svn-id: r14557
---
 gui/message.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'gui/message.cpp')

diff --git a/gui/message.cpp b/gui/message.cpp
index 206f145cdf..95b6ecfaf4 100644
--- a/gui/message.cpp
+++ b/gui/message.cpp
@@ -142,11 +142,12 @@ int MessageDialog::addLine(StringList &lines, const char *line, int size) {
 }
 
 void MessageDialog::handleCommand(CommandSender *sender, uint32 cmd, uint32 data) {
+	// FIXME: It's a really bad thing that we use two arbitrary constants 
 	if (cmd == kOkCmd) {
-		setResult(1);
+		setResult(kMessageOK);
 		close();
 	} else if (cmd == kCancelCmd) {
-		setResult(2);
+		setResult(kMessageCancel);
 		close();
 	} else {
 		Dialog::handleCommand(sender, cmd, data);
-- 
cgit v1.2.3