epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Unit order client != null check fixed (int != null -> always true!) .

Browse Source 
Removed if-else branch in favour of sugar construct a?b:c.

Note: One should (at least) consider to move var client = ... to the
start of the switch construct. 4 branches of this switch are using this.
(DRY)
This commit is contained in:
Remco van der Zon
2012-06-06 10:54:21 +02:00
parent 8f981977e0
commit 8ad4e08651
1 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
OpenRA.Game/Network/UnitOrders.cs
View File

@@ -97,19 +97,13 @@ namespace OpenRA.Network
case "PauseGame": case "PauseGame":
{ {
if(clientId != null) var client = orderManager.LobbyInfo.ClientWithIndex(clientId);
if(client != null)
{ {
var client = orderManager.LobbyInfo.ClientWithIndex(clientId);
orderManager.GamePaused = !orderManager.GamePaused; orderManager.GamePaused = !orderManager.GamePaused;
if(orderManager.GamePaused) var pausetext = "The game is {0} by {1}".F( orderManager.GamePaused ? "paused" : "un-paused", client.Name );
{ Game.AddChatLine(Color.White, "", pausetext);
Game.AddChatLine(Color.White, "", "The game is paused by "+client.Name);
}
else
{
Game.AddChatLine(Color.White, "", "The game is un-paused by "+client.Name);
}
} }
break; break;
} }