cpp/openttd-patchpack/source Changeset - r25888:e1974fe974d6

Changeset - r25888:e1974fe974d6

Parent rev.

Child rev.

[Not reviewed]

master

0 3 0

Patric Stout - 3 years ago 2021-08-14 22:17:10
truebrain@openttd.org

Add: open Online Players GUI on starting/joining a server (#9479)

3 files changed with 21 insertions and 13 deletions:

src/network/network_client.cpp

src/network/network_gui.cpp

src/openttd.cpp

0 comments (0 inline, 0 general)

src/network/network_client.cpp

➞

Show inline comments

@@ @@ -868,6 +868,8 @@ NetworkRecvStatus ClientNetworkGameSocke @@
 	/* Say we received the map and loaded it correctly! */
 	SendMapOk();
 	ShowClientList();
 	/* New company/spectator (invalid company) or company we want to join is not active
 	 * Switch local company to spectator and await the server's judgement */
 	if (_network_join.company == COMPANY_NEW_COMPANY || !Company::IsValidID(_network_join.company)) {

src/network/network_gui.cpp

➞

Show inline comments

@@ @@ -1579,15 +1579,15 @@ private: @@
 	/**
 	 * Part of RebuildList() to create the information for a single company.
 	 * @param company_id The company to build the list for.
-	 * @param own_ci The NetworkClientInfo of the client itself.
+	 * @param client_playas The company the client is joined as.
 	 */
-	void RebuildListCompany(CompanyID company_id, const NetworkClientInfo *own_ci)
+	void RebuildListCompany(CompanyID company_id, CompanyID client_playas)
+	{
 		ButtonCommon *chat_button = new CompanyButton(SPR_CHAT, company_id == COMPANY_SPECTATOR ? STR_NETWORK_CLIENT_LIST_CHAT_SPECTATOR_TOOLTIP : STR_NETWORK_CLIENT_LIST_CHAT_COMPANY_TOOLTIP, COLOUR_ORANGE, company_id, &NetworkClientListWindow::OnClickCompanyChat);
 		if (_network_server) this->buttons[line_count].emplace_back(new CompanyButton(SPR_ADMIN, STR_NETWORK_CLIENT_LIST_ADMIN_COMPANY_TOOLTIP, COLOUR_RED, company_id, &NetworkClientListWindow::OnClickCompanyAdmin, company_id == COMPANY_SPECTATOR));
 		this->buttons[line_count].emplace_back(chat_button);
-		if (own_ci->client_playas != company_id) this->buttons[line_count].emplace_back(new CompanyButton(SPR_JOIN, STR_NETWORK_CLIENT_LIST_JOIN_TOOLTIP, COLOUR_ORANGE, company_id, &NetworkClientListWindow::OnClickCompanyJoin, company_id != COMPANY_SPECTATOR && Company::Get(company_id)->is_ai));
 		if (client_playas != company_id) this->buttons[line_count].emplace_back(new CompanyButton(SPR_JOIN, STR_NETWORK_CLIENT_LIST_JOIN_TOOLTIP, COLOUR_ORANGE, company_id, &NetworkClientListWindow::OnClickCompanyJoin, company_id != COMPANY_SPECTATOR && Company::Get(company_id)->is_ai));
 		this->line_count += 1;
@@ @@ -1618,6 +1618,7 @@ private: @@
 	void RebuildList()
+	{
 		const NetworkClientInfo *own_ci = NetworkClientInfo::GetByClientID(_network_own_client_id);
 		CompanyID client_playas = own_ci == nullptr ? COMPANY_SPECTATOR : own_ci->client_playas;
 		this->buttons.clear();
 		this->line_count = 0;
@@ @@ -1625,24 +1626,24 @@ private: @@
 		this->player_self_index = -1;
 		/* As spectator, show a line to create a new company. */
-		if (own_ci->client_playas == COMPANY_SPECTATOR && !NetworkMaxCompaniesReached()) {
 		if (client_playas == COMPANY_SPECTATOR && !NetworkMaxCompaniesReached()) {
 			this->buttons[line_count].emplace_back(new CompanyButton(SPR_JOIN, STR_NETWORK_CLIENT_LIST_NEW_COMPANY_TOOLTIP, COLOUR_ORANGE, COMPANY_SPECTATOR, &NetworkClientListWindow::OnClickCompanyNew));
 			this->line_count += 1;
+		}
 		if (own_ci->client_playas != COMPANY_SPECTATOR) {
 			this->RebuildListCompany(own_ci->client_playas, own_ci);
 		if (client_playas != COMPANY_SPECTATOR) {
 			this->RebuildListCompany(client_playas, client_playas);
+		}
 		/* Companies */
 		for (const Company *c : Company::Iterate()) {
-			if (c->index == own_ci->client_playas) continue;
 			if (c->index == client_playas) continue;
-			this->RebuildListCompany(c->index, own_ci);
+			this->RebuildListCompany(c->index, client_playas);
+		}
 		/* Spectators */
-		this->RebuildListCompany(COMPANY_SPECTATOR, own_ci);
+		this->RebuildListCompany(COMPANY_SPECTATOR, client_playas);
 		this->vscroll->SetCount(this->line_count);
+	}
@@ @@ -2063,16 +2064,18 @@ public: @@
+				}
 				NetworkClientInfo *own_ci = NetworkClientInfo::GetByClientID(_network_own_client_id);
 				if (own_ci->client_playas == COMPANY_SPECTATOR && !NetworkMaxCompaniesReached()) {
 				CompanyID client_playas = own_ci == nullptr ? COMPANY_SPECTATOR : own_ci->client_playas;
 				if (client_playas == COMPANY_SPECTATOR && !NetworkMaxCompaniesReached()) {
 					this->DrawCompany(COMPANY_NEW_COMPANY, r.left, r.right, r.top, line);
+				}
 				if (own_ci->client_playas != COMPANY_SPECTATOR) {
 					this->DrawCompany(own_ci->client_playas, r.left, r.right, r.top, line);
 				if (client_playas != COMPANY_SPECTATOR) {
 					this->DrawCompany(client_playas, r.left, r.right, r.top, line);
+				}
 				for (const Company *c : Company::Iterate()) {
-					if (own_ci->client_playas == c->index) continue;
 					if (client_playas == c->index) continue;
 					this->DrawCompany(c->index, r.left, r.right, r.top, line);
+				}

src/openttd.cpp

➞

Show inline comments

@@ @@ -65,6 +65,7 @@ @@
 #include "viewport_sprite_sorter.h"
 #include "framerate_type.h"
 #include "industry.h"
 #include "network/network_gui.h"
 #include "linkgraph/linkgraphschedule.h"
@@ @@ -884,6 +885,8 @@ static void MakeNewGameDone() @@
 	CheckEngines();
 	CheckIndustries();
 	MarkWholeScreenDirty();
 	if (_network_server && !_network_dedicated) ShowClientList();
+}
 static void MakeNewGame(bool from_heightmap, bool reset_settings)

0 comments (0 inline, 0 general)