stripped dead code from move; added rally point support

OpenRa.Game/Controller.cs

@@ -25,7 +25,7 @@ namespace OpenRa.Game
 					if (order.Subject != null && order.Player == Game.LocalPlayer)
 						doVoice = order.Subject;
 				}
-			if (doVoice != null)
+			if (doVoice != null && doVoice.traits.Contains<Mobile>())
 				Game.PlaySound(Game.SovietVoices.First.GetNext() + GetVoiceSuffix(doVoice), false);
 		}
 

OpenRa.Game/OpenRa.Game.csproj

@@ -144,6 +144,7 @@
     <Compile Include="Traits\InfantrySquad.cs" />
     <Compile Include="Traits\McvDeploy.cs" />
     <Compile Include="Traits\Mobile.cs" />
+    <Compile Include="Traits\RallyPoint.cs" />
     <Compile Include="Traits\RenderBuilding.cs" />
     <Compile Include="Traits\RenderBuildingOre.cs" />
     <Compile Include="Traits\RenderBuildingTurreted.cs" />

OpenRa.Game/Order.cs

@@ -133,5 +133,10 @@ namespace OpenRa.Game
 		{
 			return new Order( subject, "CancelProduction", null, null, int2.Zero, item, Cursor.Default );
 		}
+
+		public static Order SetRallyPoint(Actor subject, int2 target)
+		{
+			return new Order(subject.Owner, "SetRallyPoint", subject, null, target, null, Cursor.Move);
+		}
 	}
 }

OpenRa.Game/Traits/Activities/Move.cs

@@ -116,31 +116,12 @@ namespace OpenRa.Game.Traits.Activities
 
 				Game.UnitInfluence.Remove( mobile );
 				var newPath = getPath(self, mobile).TakeWhile(a => a != self.Location).ToList();
-				//var newPath = Game.PathFinder.FindPathToPath( self.Location, path, mobile.GetMovementType() )
+			
-				//    .TakeWhile( a => a != self.Location )
-				//    .ToList();
 				Game.UnitInfluence.Add( mobile );
-				if (newPath.Count == 0)
+				if (newPath.Count != 0)
-					return null;
-				else
-				{
 					path = newPath;
-					return null;
-				}
 
-				//while( path.Count != 0 && path[ path.Count - 1 ] != newPath[ 0 ] )
+				return null;
-				//    path.RemoveAt( path.Count - 1 );
-				//for( int i = 1 ; i < newPath.Count ; i++ )
-				//    path.Add( newPath[ i ] );
-
-				if( path.Count == 0 )
-					return null;
-				nextCell = path[ path.Count - 1 ];
-				if( !CanEnterCell( nextCell, self ) )
-				{
-					path.Clear();
-					return null;
-				}
 			}
 			path.RemoveAt( path.Count - 1 );
 			return nextCell;

OpenRa.Game/Traits/RallyPoint.cs

@@ -0,0 +1,38 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using IjwFramework.Types;
+using OpenRa.Game.Graphics;
+
+namespace OpenRa.Game.Traits
+{
+	class RallyPoint : IRender, IOrder, ITick
+	{
+		public int2 rallyPoint;
+		public Animation anim;
+
+		public RallyPoint(Actor self)
+		{
+			rallyPoint = self.Location + new int2(1, 3);	//self.unitInfo.RallyPoint;
+			anim = new Animation("flagfly");
+			anim.PlayRepeating("idle");
+		}
+
+		public IEnumerable<Pair<Sprite, float2>> Render(Actor self)
+		{
+			var uog = Game.controller.orderGenerator as UnitOrderGenerator;
+			if (uog != null && self.Owner == Game.LocalPlayer && uog.selection.Contains(self))
+				yield return Util.Centered(
+					anim.Image, Game.CellSize * (new float2(.5f, .5f) + rallyPoint.ToFloat2()));
+		}
+
+		public Order Order(Actor self, int2 xy, bool lmb, Actor underCursor)
+		{
+			if (lmb || underCursor != null) return null;
+			return OpenRa.Game.Order.SetRallyPoint(self, xy);
+		}
+
+		public void Tick(Actor self) { anim.Tick(); }
+	}
+}

OpenRa.Game/UnitOrders.cs

@@ -123,6 +123,12 @@ namespace OpenRa.Game
 						order.Player.CancelProduction( Rules.UnitCategory[ order.TargetString ] );
 					break;
 				}
+			case "SetRallyPoint":
+				{
+					var pt = order.Subject.traits.Get<RallyPoint>();
+					pt.rallyPoint = order.TargetLocation;
+					break;
+				}
 			default:
 				throw new NotImplementedException();
 			}

sequences.xml

@@ -507,4 +507,7 @@
   <unit name="v2">
     <sequence name="idle" start="0" length="32" />
   </unit>
+  <unit name="flagfly">
+    <sequence name="idle" start="0" length="14" />
+  </unit>
 </sequences>

units.ini

@@ -204,7 +204,7 @@ Dimensions=2,2
 Footprint=xx xx
 [WEAP]
 Description=War Factory
-Traits=Building, RenderWarFactory
+Traits=Building, RenderWarFactory, RallyPoint
 Dimensions=3,2
 Footprint=xxx xxx
 Produces=Vehicle