summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatric Stout <truebrain@openttd.org>2021-04-20 16:42:37 +0200
committerPatric Stout <github@truebrain.nl>2021-04-20 17:24:38 +0200
commitf4bd3fff5ebfdd7c58ccaf23977b8447c740b48a (patch)
tree77173187508289e9a5a4b1f1cb67bdee5b251a74
parent17d00537a15b30f06b5a3bb800a7f26ce4eec359 (diff)
downloadopenttd-f4bd3fff5ebfdd7c58ccaf23977b8447c740b48a.tar.xz
Remove: "map_name" from server announcements / listing
The idea back in the days was nice, but it never resulted in anything useful. Most servers either read "(loaded game)" or "Random Map", neither being useful. It was meant for heightmaps, so you could find a server that was using a specific one .. but there are many things wrong with that idea. Mostly, servers tend to save and load savegames from time to time, after which the original heightmap used was lost. All in all, removing map_name all together is just better.
-rw-r--r--src/network/core/game.h1
-rw-r--r--src/network/core/udp.cpp4
-rw-r--r--src/network/network_admin.cpp2
-rw-r--r--src/network/network_gui.cpp1
-rw-r--r--src/network/network_udp.cpp1
-rw-r--r--src/openttd.cpp9
6 files changed, 3 insertions, 15 deletions
diff --git a/src/network/core/game.h b/src/network/core/game.h
index 151ebef28..0a10cfb1f 100644
--- a/src/network/core/game.h
+++ b/src/network/core/game.h
@@ -22,7 +22,6 @@
* be sent to the clients.
*/
struct NetworkServerGameInfo {
- char map_name[NETWORK_NAME_LENGTH]; ///< Map which is played ["random" for a randomized map]
byte clients_on; ///< Current count of clients on server
};
diff --git a/src/network/core/udp.cpp b/src/network/core/udp.cpp
index aa6d39cbb..985c8fa72 100644
--- a/src/network/core/udp.cpp
+++ b/src/network/core/udp.cpp
@@ -205,7 +205,7 @@ void NetworkUDPSocketHandler::SendNetworkGameInfo(Packet *p, const NetworkGameIn
p->Send_uint8 (info->clients_max);
p->Send_uint8 (info->clients_on);
p->Send_uint8 (info->spectators_on);
- p->Send_string(info->map_name);
+ p->Send_string(""); // Used to be map-name.
p->Send_uint16(info->map_width);
p->Send_uint16(info->map_height);
p->Send_uint8 (info->map_set);
@@ -275,7 +275,7 @@ void NetworkUDPSocketHandler::ReceiveNetworkGameInfo(Packet *p, NetworkGameInfo
info->game_date = p->Recv_uint16() + DAYS_TILL_ORIGINAL_BASE_YEAR;
info->start_date = p->Recv_uint16() + DAYS_TILL_ORIGINAL_BASE_YEAR;
}
- p->Recv_string(info->map_name, sizeof(info->map_name));
+ while (p->Recv_uint8() != 0) {} // Used to contain the map-name.
info->map_width = p->Recv_uint16();
info->map_height = p->Recv_uint16();
info->map_set = p->Recv_uint8 ();
diff --git a/src/network/network_admin.cpp b/src/network/network_admin.cpp
index aa2859ef8..825acfed1 100644
--- a/src/network/network_admin.cpp
+++ b/src/network/network_admin.cpp
@@ -170,7 +170,7 @@ NetworkRecvStatus ServerNetworkAdminSocketHandler::SendWelcome()
p->Send_string(GetNetworkRevisionString());
p->Send_bool (_network_dedicated);
- p->Send_string(_network_game_info.map_name);
+ p->Send_string(""); // Used to be map-name.
p->Send_uint32(_settings_game.game_creation.generation_seed);
p->Send_uint8 (_settings_game.game_creation.landscape);
p->Send_uint32(ConvertYMDToDate(_settings_game.game_creation.starting_year, 0, 1));
diff --git a/src/network/network_gui.cpp b/src/network/network_gui.cpp
index 33639584a..591a45ae9 100644
--- a/src/network/network_gui.cpp
+++ b/src/network/network_gui.cpp
@@ -628,7 +628,6 @@ public:
DrawString(r.left + WD_FRAMERECT_LEFT, r.right - WD_FRAMERECT_RIGHT, r.top + 6, STR_NETWORK_SERVER_LIST_GAME_INFO, TC_FROMSTRING, SA_HOR_CENTER);
DrawString(r.left + WD_FRAMERECT_LEFT, r.right - WD_FRAMERECT_RIGHT, r.top + 6 + 4 + FONT_HEIGHT_NORMAL, sel->info.server_name, TC_ORANGE, SA_HOR_CENTER); // game name
- DrawString(r.left + WD_FRAMERECT_LEFT, r.right - WD_FRAMERECT_RIGHT, r.top + 6 + 8 + 2 * FONT_HEIGHT_NORMAL, sel->info.map_name, TC_BLACK, SA_HOR_CENTER); // map name
uint16 y = r.top + detail_height + 4;
diff --git a/src/network/network_udp.cpp b/src/network/network_udp.cpp
index 6d7e6ca69..5b150c45a 100644
--- a/src/network/network_udp.cpp
+++ b/src/network/network_udp.cpp
@@ -190,7 +190,6 @@ void ServerNetworkUDPSocketHandler::Receive_CLIENT_FIND_SERVER(Packet *p, Networ
ngi.dedicated = _network_dedicated;
ngi.grfconfig = _grfconfig;
- strecpy(ngi.map_name, _network_game_info.map_name, lastof(ngi.map_name));
strecpy(ngi.server_name, _settings_client.network.server_name, lastof(ngi.server_name));
strecpy(ngi.server_revision, GetNetworkRevisionString(), lastof(ngi.server_revision));
diff --git a/src/openttd.cpp b/src/openttd.cpp
index 4d820d4db..ff6867c4b 100644
--- a/src/openttd.cpp
+++ b/src/openttd.cpp
@@ -1048,9 +1048,6 @@ void SwitchToMode(SwitchMode new_mode)
case SM_RESTARTGAME: // Restart --> 'Random game' with current settings
case SM_NEWGAME: // New Game --> 'Random game'
- if (_network_server) {
- seprintf(_network_game_info.map_name, lastof(_network_game_info.map_name), "Random Map");
- }
MakeNewGame(false, new_mode == SM_NEWGAME);
break;
@@ -1073,18 +1070,12 @@ void SwitchToMode(SwitchMode new_mode)
IConsoleCmdExec("exec scripts/game_start.scr 0");
/* Decrease pause counter (was increased from opening load dialog) */
DoCommandP(0, PM_PAUSED_SAVELOAD, 0, CMD_PAUSE);
- if (_network_server) {
- seprintf(_network_game_info.map_name, lastof(_network_game_info.map_name), "%s (Loaded game)", _file_to_saveload.title);
- }
}
break;
}
case SM_RESTART_HEIGHTMAP: // Load a heightmap and start a new game from it with current settings
case SM_START_HEIGHTMAP: // Load a heightmap and start a new game from it
- if (_network_server) {
- seprintf(_network_game_info.map_name, lastof(_network_game_info.map_name), "%s (Heightmap)", _file_to_saveload.title);
- }
MakeNewGame(true, new_mode == SM_START_HEIGHTMAP);
break;