Mercurial > hg > openttd

changeset 9481:cbb53dc04d5b draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
(svn r13433) -Codechange: remove a now useless global variable. -Fix: MP advertising was not advertising.
author rubidium <rubidium@openttd.org>
date Mon, 09 Jun 2008 12:17:01 +0000
parents 7d7fafc872f5
children 18a4b3350c4f
files src/main_gui.cpp src/network/network.cpp src/network/network.h src/network/network_gui.cpp src/network/network_udp.cpp
diffstat 5 files changed, 7 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/src/main_gui.cpp
+++ b/src/main_gui.cpp
@@ -289,7 +289,7 @@
 
 			case '1' | WKC_ALT: // Gimme money
 				/* Server can not cheat in advertise mode either! */
-				if (!_networking || !_network_server || !_network_advertise)
+				if (!_networking || !_network_server || !_settings_client.network.server_advertise)
 					DoCommandP(0, 10000000, 0, NULL, CMD_MONEY_CHEAT);
 				break;
 
--- a/src/network/network.cpp
+++ b/src/network/network.cpp
@@ -45,7 +45,6 @@
 bool _network_server;     ///< network-server is active
 bool _network_available;  ///< is network mode available?
 bool _network_dedicated;  ///< are we a dedicated server?
-bool _network_advertise;  ///< is the server advertising to the master server?
 bool _is_network_server;  ///< Does this client wants to be a network-server?
 NetworkServerGameInfo _network_game_info;
 NetworkPlayerInfo _network_player_info[MAX_PLAYERS];
@@ -1110,7 +1109,7 @@
 		}
 	}
 
-	if (_network_advertise) NetworkUDPRemoveAdvertise();
+	if (_settings_client.network.server_advertise) NetworkUDPRemoveAdvertise();
 
 	DeleteWindowById(WC_NETWORK_STATUS_WINDOW, 0);
 
--- a/src/network/network.h
+++ b/src/network/network.h
@@ -16,7 +16,6 @@
 extern bool _network_server;     ///< network-server is active
 extern bool _network_available;  ///< is network mode available?
 extern bool _network_dedicated;  ///< are we a dedicated server?
-extern bool _network_advertise;  ///< is the server advertising to the master server?
 extern bool _is_network_server;  ///< Does this client wants to be a network-server?
 
 #else /* ENABLE_NETWORK */
@@ -29,7 +28,6 @@
 #define _network_server 0
 #define _network_available 0
 #define _network_dedicated 0
-#define _network_advertise 0
 #define _is_network_server 0
 
 #endif /* ENABLE_NETWORK */
--- a/src/network/network_gui.cpp
+++ b/src/network/network_gui.cpp
@@ -686,7 +686,7 @@
 		const FiosItem *item;
 
 		/* draw basic widgets */
-		SetDParam(1, _connection_types_dropdown[_network_advertise]);
+		SetDParam(1, _connection_types_dropdown[_settings_client.network.server_advertise]);
 		SetDParam(2, _settings_client.network.max_clients);
 		SetDParam(3, _settings_client.network.max_companies);
 		SetDParam(4, _settings_client.network.max_spectators);
@@ -748,7 +748,7 @@
 			} break;
 
 			case NSSW_CONNTYPE_BTN: // Connection type
-				ShowDropDownMenu(this, _connection_types_dropdown, _network_advertise, NSSW_CONNTYPE_BTN, 0, 0); // do it for widget NSSW_CONNTYPE_BTN
+				ShowDropDownMenu(this, _connection_types_dropdown, _settings_client.network.server_advertise, NSSW_CONNTYPE_BTN, 0, 0); // do it for widget NSSW_CONNTYPE_BTN
 				break;
 
 			case NSSW_CLIENTS_BTND:    case NSSW_CLIENTS_BTNU:    // Click on up/down button for number of clients
@@ -836,7 +836,7 @@
 	{
 		switch (widget) {
 			case NSSW_CONNTYPE_BTN:
-				_network_advertise = (index != 0);
+				_settings_client.network.server_advertise = (index != 0);
 				break;
 			case NSSW_LANGUAGE_BTN:
 				_settings_client.network.server_lang = _language_dropdown[index] - STR_NETWORK_LANG_ANY;
--- a/src/network/network_udp.cpp
+++ b/src/network/network_udp.cpp
@@ -52,7 +52,7 @@
 	DEBUG(net, 2, "[udp] advertising on master server successful");
 
 	/* We are advertised, but we don't want to! */
-	if (!_network_advertise) NetworkUDPRemoveAdvertise();
+	if (!_settings_client.network.server_advertise) NetworkUDPRemoveAdvertise();
 }
 
 ///*** Communication with clients (we are server) ***/
@@ -562,7 +562,7 @@
 	struct sockaddr_in out_addr;
 
 	/* Check if we should send an advertise */
-	if (!_networking || !_network_server || !_network_udp_server || !_network_advertise)
+	if (!_networking || !_network_server || !_network_udp_server || !_settings_client.network.server_advertise)
 		return;
 
 	/* check for socket */