cpp/openttd-patchpack/source Changeset - r25463:a9b079004e01

Changeset - r25463:a9b079004e01

Parent rev.

Child rev.

[Not reviewed]

master

0 5 0

rubidium42 - 3 years ago 2021-04-27 19:25:52
rubidium@openttd.org

Codechange: move server name/id in settings to std::string

5 files changed with 19 insertions and 17 deletions:

src/network/network.cpp

src/network/network_gui.cpp

src/network/network_server.cpp

src/settings_type.h

src/table/settings.ini

0 comments (0 inline, 0 general)

src/network/network.cpp

➞

Show inline comments

@@ @@ -833,8 +833,8 @@ void NetworkClientJoinGame() @@
 static void NetworkInitGameInfo()
+{
 	if (StrEmpty(_settings_client.network.server_name)) {
 		strecpy(_settings_client.network.server_name, "Unnamed Server", lastof(_settings_client.network.server_name));
 	if (_settings_client.network.server_name.empty()) {
 		_settings_client.network.server_name = "Unnamed Server";
+	}
 	FillStaticNetworkServerGameInfo();
@@ @@ -863,10 +863,10 @@ static void CheckClientAndServerName() @@
 		strecpy(_settings_client.network.client_name, fallback_client_name, lastof(_settings_client.network.client_name));
+	}
 	static const char *fallback_server_name = "Unnamed Server";
 	if (StrEmpty(_settings_client.network.server_name) || strcmp(_settings_client.network.server_name, fallback_server_name) == 0) {
 		DEBUG(net, 1, "No \"server_name\" has been set, using \"%s\" instead. Please set this now using the \"server_name <new name>\" command", fallback_server_name);
 		strecpy(_settings_client.network.server_name, fallback_server_name, lastof(_settings_client.network.server_name));
 	static const std::string fallback_server_name = "Unnamed Server";
 	if (_settings_client.network.server_name.empty() || _settings_client.network.server_name.compare(fallback_server_name) == 0) {
 		DEBUG(net, 1, "No \"server_name\" has been set, using \"%s\" instead. Please set this now using the \"server_name <new name>\" command", fallback_server_name.c_str());
 		_settings_client.network.server_name = fallback_server_name;
+	}
+}
@@ @@ -1201,7 +1201,7 @@ static void NetworkGenerateServerId() @@
+	}
 	/* _settings_client.network.network_id is our id */
-	seprintf(_settings_client.network.network_id, lastof(_settings_client.network.network_id), "%s", hex_output);
+	_settings_client.network.network_id = hex_output;
+}
 class TCPNetworkDebugConnecter : TCPConnecter {
@@ @@ -1241,7 +1241,7 @@ void NetworkStartUp() @@
 	_network_need_advertise = true;
 	/* Generate an server id when there is none yet */
-	if (StrEmpty(_settings_client.network.network_id)) NetworkGenerateServerId();
+	if (_settings_client.network.network_id.empty()) NetworkGenerateServerId();
 	_network_game_info = {};

src/network/network_gui.cpp

➞

Show inline comments

@@ @@ -990,7 +990,7 @@ struct NetworkStartServerWindow : public @@
 		this->InitNested(WN_NETWORK_WINDOW_START);
 		this->querystrings[WID_NSS_GAMENAME] = &this->name_editbox;
 		this->name_editbox.text.Assign(_settings_client.network.server_name);
+		this->name_editbox.text.Assign(_settings_client.network.server_name.c_str());
 		this->SetFocusedWidget(WID_NSS_GAMENAME);
+	}
@@ @@ -1136,7 +1136,7 @@ struct NetworkStartServerWindow : public @@
 	void OnEditboxChanged(int wid) override
+	{
 		if (wid == WID_NSS_GAMENAME) {
-			strecpy(_settings_client.network.server_name, this->name_editbox.text.buf, lastof(_settings_client.network.server_name));
+			_settings_client.network.server_name = this->name_editbox.text.buf;
+		}
+	}

src/network/network_server.cpp

➞

Show inline comments

@@ @@ -1762,7 +1762,7 @@ void NetworkServerSetCompanyPassword(Com @@
 	if (!Company::IsValidHumanID(company_id)) return;
 	if (!already_hashed) {
 		password = GenerateCompanyPasswordHash(password, _settings_client.network.network_id, _settings_game.game_creation.generation_seed);
+		password = GenerateCompanyPasswordHash(password, _settings_client.network.network_id.c_str(), _settings_game.game_creation.generation_seed);
+	}
 	strecpy(_network_company_states[company_id].password, password, lastof(_network_company_states[company_id].password));

src/settings_type.h

➞

Show inline comments

@@ @@ -262,7 +262,7 @@ struct NetworkSettings { @@
 	uint16 server_port;                                   ///< port the server listens on
 	uint16 server_admin_port;                             ///< port the server listens on for the admin network
 	bool   server_admin_chat;                             ///< allow private chat for the server to be distributed to the admin network
-	char   server_name[NETWORK_NAME_LENGTH];              ///< name of the server
+	std::string server_name;                              ///< name of the server
 	std::string server_password;                          ///< password for joining this server
 	std::string rcon_password;                            ///< password for rconsole (server side)
 	std::string admin_password;                           ///< password for the admin network
@@ @@ -270,7 +270,7 @@ struct NetworkSettings { @@
 	char   client_name[NETWORK_CLIENT_NAME_LENGTH];       ///< name of the player (as client)
 	std::string default_company_pass;                     ///< default password for new companies in encrypted form
 	std::string connect_to_ip;                            ///< default for the "Add server" query
-	char   network_id[NETWORK_SERVER_ID_LENGTH];          ///< network ID for servers
+	std::string network_id;                               ///< network ID for servers
 	bool   autoclean_companies;                           ///< automatically remove companies that are not in use
 	uint8  autoclean_unprotected;                         ///< remove passwordless companies after this many months
 	uint8  autoclean_protected;                           ///< remove the password from passworded companies after this many months

src/table/settings.ini

➞

Show inline comments

@@ @@ -3961,9 +3961,10 @@ length = NETWORK_PASSWORD_LENGTH @@
 flags    = SLF_NOT_IN_SAVE | SLF_NO_NETWORK_SYNC
 def      = nullptr
-[SDTC_STR]
+[SDTC_SSTR]
 var      = network.server_name
 type     = SLE_STRB
 type     = SLE_STR
 length   = NETWORK_NAME_LENGTH
 flags    = SLF_NOT_IN_SAVE | SLF_NO_NETWORK_SYNC
 guiflags = SGF_NETWORK_ONLY
 def      = nullptr
@@ @@ -3977,9 +3978,10 @@ length = 0 @@
 flags    = SLF_NOT_IN_SAVE | SLF_NO_NETWORK_SYNC
 def      = nullptr
-[SDTC_STR]
+[SDTC_SSTR]
 var      = network.network_id
 type     = SLE_STRB
 type     = SLE_STR
 length   = NETWORK_SERVER_ID_LENGTH
 flags    = SLF_NOT_IN_SAVE | SLF_NO_NETWORK_SYNC
 guiflags = SGF_NETWORK_ONLY
 def      = nullptr

0 comments (0 inline, 0 general)