Extract contrail fading for dead projectiles into an IEffect.

OpenRA.Mods.RA/Effects/Bullet.cs

@@ -1,6 +1,6 @@
 #region Copyright & License Information
 /*
- * Copyright 2007-2011 The OpenRA Developers (see AUTHORS)
+ * Copyright 2007-2013 The OpenRA Developers (see AUTHORS)
  * This file is part of OpenRA, which is free software. It is made
  * available to you under the terms of the GNU General Public License
  * as published by the Free Software Foundation. For more information,
@@ -110,13 +110,6 @@ namespace OpenRA.Mods.RA.Effects
 
 		public void Tick(World world)
 		{
-			// Fade the trail out gradually
-			if (ticks > length && info.ContrailLength > 0)
-			{
-				trail.Update(pos);
-				return;
-			}
-
 			if (anim != null)
 				anim.Tick();
 
@@ -166,9 +159,9 @@ namespace OpenRA.Mods.RA.Effects
 		void Explode(World world)
 		{
 			if (info.ContrailLength > 0)
-				world.AddFrameEndTask(w => w.Add(new DelayedAction(info.ContrailLength, () => w.Remove(this))));
+				world.AddFrameEndTask(w => w.Add(new ContrailFader(pos, trail)));
-			else
+
-				world.AddFrameEndTask(w => w.Remove(this));
+			world.AddFrameEndTask(w => w.Remove(this));
 
 			Combat.DoImpacts(pos, args.SourceActor, args.Weapon, args.FirepowerModifier);
 		}

OpenRA.Mods.RA/Effects/ContrailFader.cs

@@ -0,0 +1,38 @@
+#region Copyright & License Information
+/*
+ * Copyright 2007-2013 The OpenRA Developers (see AUTHORS)
+ * This file is part of OpenRA, which is free software. It is made
+ * available to you under the terms of the GNU General Public License
+ * as published by the Free Software Foundation. For more information,
+ * see COPYING.
+ */
+#endregion
+
+using System.Collections.Generic;
+using OpenRA.Effects;
+using OpenRA.Graphics;
+
+namespace OpenRA.Mods.RA.Effects
+{
+	class ContrailFader : IEffect
+	{
+		WPos pos;
+		ContrailRenderable trail;
+
+		public ContrailFader(WPos pos, ContrailRenderable trail)
+		{
+			this.pos = pos;
+			this.trail = trail;
+		}
+
+		public void Tick(World world)
+		{
+			trail.Update(pos);
+		}
+
+		public IEnumerable<IRenderable> Render(WorldRenderer wr)
+		{
+			yield return trail;
+		}
+	}
+}

OpenRA.Mods.RA/Effects/Missile.cs

@@ -1,6 +1,6 @@
 #region Copyright & License Information
 /*
- * Copyright 2007-2011 The OpenRA Developers (see AUTHORS)
+ * Copyright 2007-2013 The OpenRA Developers (see AUTHORS)
  * This file is part of OpenRA, which is free software. It is made
  * available to you under the terms of the GNU General Public License
  * as published by the Free Software Foundation. For more information,
@@ -63,7 +63,6 @@ namespace OpenRA.Mods.RA.Effects
 		[Sync] WPos targetPosition;
 		[Sync] WVec offset;
 		[Sync] int ticks;
-		[Sync] bool exploded;
 
 		[Sync] public Actor SourceActor { get { return args.SourceActor; } }
 		[Sync] public Target GuidedTarget { get { return args.GuidedTarget; } }
@@ -112,13 +111,6 @@ namespace OpenRA.Mods.RA.Effects
 
 		public void Tick(World world)
 		{
-			// Fade the trail out gradually
-			if (exploded && info.ContrailLength > 0)
-			{
-				trail.Update(pos);
-				return;
-			}
-
 			ticks++;
 			anim.Tick();
 
@@ -174,12 +166,10 @@ namespace OpenRA.Mods.RA.Effects
 
 		void Explode(World world)
 		{
-			exploded = true;
-
 			if (info.ContrailLength > 0)
-				world.AddFrameEndTask(w => w.Add(new DelayedAction(info.ContrailLength, () => w.Remove(this))));
+				world.AddFrameEndTask(w => w.Add(new ContrailFader(pos, trail)));
-			else
+
-				world.AddFrameEndTask(w => w.Remove(this));
+			world.AddFrameEndTask(w => w.Remove(this));
 
 			// Don't blow up in our launcher's face!
 			if (ticks <= info.Arm)

OpenRA.Mods.RA/OpenRA.Mods.RA.csproj

@@ -471,6 +471,7 @@
     <Compile Include="Widgets\Logic\InstallLogic.cs" />
     <Compile Include="CombatDebugOverlay.cs" />
     <Compile Include="World\PathfinderDebugOverlay.cs" />
+    <Compile Include="Effects\ContrailFader.cs" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\OpenRA.FileFormats\OpenRA.FileFormats.csproj">