fixed #999 -- replace order/first[ordefault] with .ClosestTo(point)

OpenRA.Game/WorldUtils.cs

@@ -34,6 +34,11 @@ namespace OpenRA
 			return world.WorldActor.Trait<SpatialBins>().ActorsInBox(u,v);
 		}
 
+		public static Actor ClosestTo( this IEnumerable<Actor> actors, int2 px )
+		{
+			return actors.OrderBy( a => (a.CenterLocation - px).LengthSquared ).FirstOrDefault();
+		}
+
 		public static IEnumerable<Actor> FindUnitsInCircle(this World world, int2 a, int r)
 		{
 			using (new PerfSample("FindUnitsInCircle"))

OpenRA.Mods.RA/Activities/LayMines.cs

@@ -32,15 +32,13 @@ namespace OpenRA.Mods.RA.Activities
 				var buildings = self.Info.Traits.Get<MinelayerInfo>().RearmBuildings;
 				var rearmTarget = self.World.Actors.Where(a => a.Owner != null && self.Owner.Stances[a.Owner] == Stance.Ally
 					&& buildings.Contains(a.Info.Name))
-					.OrderBy( a => (a.Location - self.Location).LengthSquared )
+					.ClosestTo( self.CenterLocation );
-					.FirstOrDefault();
 
 				if (rearmTarget == null)
 					return new Wait(20);
 
 				return Util.SequenceActivities(
 					new Enter(rearmTarget),
-					//new Move(Util.CellContaining(rearmTarget.CenterLocation), rearmTarget),
 					new Rearm(self),
 					new Repair(rearmTarget),
 					this );

OpenRA.Mods.RA/Air/ReturnToBase.cs

@@ -25,13 +25,13 @@ namespace OpenRA.Mods.RA.Air
 
 		public static Actor ChooseAirfield(Actor self)
 		{
+			var rearmBuildings = self.Info.Traits.Get<PlaneInfo>().RearmBuildings;
 			return self.World.ActorsWithTrait<Reservable>()
 				.Where(a => a.Actor.Owner == self.Owner)
-                .Where(a => self.Info.Traits.Get<PlaneInfo>().RearmBuildings.Contains(a.Actor.Info.Name)
+				.Where(a => rearmBuildings.Contains(a.Actor.Info.Name)
 					&& !Reservable.IsReserved(a.Actor))
-                .OrderBy(a => (a.Actor.CenterLocation - self.CenterLocation).LengthSquared)
 				.Select(a => a.Actor)
-                .FirstOrDefault();
+				.ClosestTo( self.CenterLocation );
 		}
 
 		void Calculate(Actor self)

OpenRA.Mods.RA/AutoHeal.cs

@@ -28,8 +28,7 @@ namespace OpenRA.Mods.RA
 				.Where(a => a.IsInWorld && !a.IsDead())
 				.Where(a => a.HasTrait<Health>() && a.GetDamageState() > DamageState.Undamaged)
 				.Where(a => attack.HasAnyValidWeapons(Target.FromActor(a)))
-				.OrderBy(a => (a.CenterLocation - self.CenterLocation).LengthSquared)
+				.ClosestTo( self.CenterLocation );
-				.FirstOrDefault();
 			
 			if( target != null )
 				self.QueueActivity(self.Trait<AttackBase>().GetAttackActivity(self, Target.FromActor( target ), false ));

OpenRA.Mods.RA/AutoTarget.cs

@@ -101,8 +101,7 @@ namespace OpenRA.Mods.RA
 				.Where(a => a.Owner != null && a.AppearsHostileTo(self))
 				.Where(a => !a.HasTrait<AutoTargetIgnore>())
 				.Where(a => attack.HasAnyValidWeapons(Target.FromActor(a)))
-				.OrderBy(a => (a.CenterLocation - self.CenterLocation).LengthSquared)
+				.ClosestTo( self.CenterLocation );
-				.FirstOrDefault();
 		}
 	}
 

OpenRA.Mods.RA/HackyAI.cs

@@ -305,7 +305,7 @@ namespace OpenRA.Mods.RA
 					{
 						// Found enemy units nearby.
 						foundEnemy = true;
-						var enemy = enemyUnits.OrderBy(e => (e.Location - a1.Location).LengthSquared).First();
+						var enemy = enemyUnits.ClosestTo( a1.CenterLocation );
 
 						// Check how many own units we have gathered nearby...
 						var ownUnits = world.FindUnitsInCircle(a1.CenterLocation, Game.CellSize * 2)

OpenRA.Mods.RA/ProximityCapturable.cs

@@ -174,8 +174,7 @@ namespace OpenRA.Mods.RA
         // TODO exclude other NeutralActor that arent permanent
         Actor GetInRange(Actor self)
         {
-            return CaptorsInRange(self).OrderBy(a => (a.CenterLocation - self.CenterLocation).LengthSquared)
+            return CaptorsInRange(self).ClosestTo( self.CenterLocation );
-                .FirstOrDefault();
         }
 
         int CountPlayersNear(Actor self, Player ignoreMe)