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

1 files changed, 3 insertions, 2 deletions

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);