From b524dc9b7200cd0fd6b2a15699e1a32ffff43ae2 Mon Sep 17 00:00:00 2001 From: reaperrr Date: Thu, 20 Feb 2020 03:19:58 +0100 Subject: [PATCH] Carryall ResolveOrder optimization There can be only one OrderString at a time, so we make 'PickupUnit' an "else if", too. --- OpenRA.Mods.Common/Traits/Carryall.cs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/OpenRA.Mods.Common/Traits/Carryall.cs b/OpenRA.Mods.Common/Traits/Carryall.cs index 7f5a474e25..561bcc60fd 100644 --- a/OpenRA.Mods.Common/Traits/Carryall.cs +++ b/OpenRA.Mods.Common/Traits/Carryall.cs @@ -307,8 +307,7 @@ namespace OpenRA.Mods.Common.Traits self.QueueActivity(order.Queued, new DeliverUnit(self, Info.DropRange)); } - - if (order.OrderString == "PickupUnit") + else if (order.OrderString == "PickupUnit") { if (order.Target.Type != TargetType.Actor) return;