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

Pass world directly to UnitOrders.ResolveOrder

Browse Source 
Avoids order.Subject.World look-ups.

Also removes validOrders parameter,
we can get that directly from World now.
This commit is contained in:
reaperrr
2020-09-15 10:42:50 +02:00
committed by Paul Chote
parent 904a5f60d1
commit a4a409f39b
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -336,22 +336,22 @@ namespace OpenRA.Network
break;
if (order.GroupedActors == null)
ResolveOrder(order, world.OrderValidators, orderManager, clientId);
ResolveOrder(order, world, orderManager, clientId);
else
foreach (var subject in order.GroupedActors)
ResolveOrder(Order.FromGroupedOrder(order, subject), world.OrderValidators, orderManager, clientId);
ResolveOrder(Order.FromGroupedOrder(order, subject), world, orderManager, clientId);
break;
}
}
}
static void ResolveOrder(Order order, IEnumerable<IValidateOrder> validateOrders, OrderManager orderManager, int clientId)
static void ResolveOrder(Order order, World world, OrderManager orderManager, int clientId)
{
if (order.Subject == null || order.Subject.IsDead)
return;
if (validateOrders.All(vo => vo.OrderValidation(orderManager, order.Subject.World, clientId, order)))
if (world.OrderValidators.All(vo => vo.OrderValidation(orderManager, world, clientId, order)))
foreach (var t in order.Subject.TraitsImplementing<IResolveOrder>())
t.ResolveOrder(order.Subject, order);
}