(svn r24732) -Codechange: Unify handling of OK and CANCEL actions for editboxes.

2 files changed, 10 insertions, 31 deletions

diff --git a/src/network/network_chat_gui.cpp b/src/network/network_chat_gui.cpp
index 4c1551347..0cd9c0558 100644
--- a/src/network/network_chat_gui.cpp
+++ b/src/network/network_chat_gui.cpp
@@ -512,13 +512,7 @@ struct NetworkChatWindow : public QueryStringBaseWindow {
 			state = ES_HANDLED;
 		} else {
 			_chat_tab_completion_active = false;
-			switch (this->HandleEditBoxKey(WID_NC_TEXTBOX, key, keycode, state)) {
-				default: break;
-				case HEBR_CONFIRM:
-					SendChat(this->text.buf, this->dtype, this->dest);
-					/* FALL THROUGH */
-				case HEBR_CANCEL: delete this; break;
-			}
+			this->HandleEditBoxKey(WID_NC_TEXTBOX, key, keycode, state);
 		}
 		return state;
 	}
diff --git a/src/network/network_gui.cpp b/src/network/network_gui.cpp
index 94c2e3805..7b4735629 100644
--- a/src/network/network_gui.cpp
+++ b/src/network/network_gui.cpp
@@ -835,20 +835,15 @@ public:
 			return ES_HANDLED;
 		}
 
-		switch (this->HandleEditBoxKey(WID_NG_CLIENT, key, keycode, state)) {
-			case HEBR_NOT_FOCUSED:
-				if (this->server != NULL) {
-					if (keycode == WKC_DELETE) { // Press 'delete' to remove servers
-						NetworkGameListRemoveItem(this->server);
-						if (this->server == this->last_joined) this->last_joined = NULL;
-						this->server = NULL;
-						this->list_pos = SLP_INVALID;
-					}
+		if (this->HandleEditBoxKey(WID_NG_CLIENT, key, keycode, state) == HEBR_NOT_FOCUSED) {
+			if (this->server != NULL) {
+				if (keycode == WKC_DELETE) { // Press 'delete' to remove servers
+					NetworkGameListRemoveItem(this->server);
+					if (this->server == this->last_joined) this->last_joined = NULL;
+					this->server = NULL;
+					this->list_pos = SLP_INVALID;
 				}
-				break;
-
-			default:
-				break;
+			}
 		}
 
 		return state;
@@ -2160,17 +2155,7 @@ struct NetworkCompanyPasswordWindow : public QueryStringBaseWindow {
 	virtual EventState OnKeyPress(uint16 key, uint16 keycode)
 	{
 		EventState state = ES_NOT_HANDLED;
-		switch (this->HandleEditBoxKey(WID_NCP_PASSWORD, key, keycode, state)) {
-			default: break;
-
-			case HEBR_CONFIRM:
-				this->OnOk();
-				/* FALL THROUGH */
-
-			case HEBR_CANCEL:
-				delete this;
-				break;
-		}
+		this->HandleEditBoxKey(WID_NCP_PASSWORD, key, keycode, state);
 		return state;
 	}
 };