split remap classes

OpenRa.FileFormats/OpenRa.FileFormats.csproj

@@ -60,10 +60,11 @@
     <Compile Include="PackageEntry.cs" />
     <Compile Include="Package.cs" />
     <Compile Include="Palette.cs" />
-    <Compile Include="PaletteRemap.cs" />
+    <Compile Include="PlayerColorRemap.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="ShpReader.cs" />
     <Compile Include="ShroudPaletteRemap.cs" />
+    <Compile Include="SingleColorRemap.cs" />
     <Compile Include="Terrain.cs" />
     <Compile Include="TileReference.cs" />
     <Compile Include="TileSet.cs" />

OpenRa.FileFormats/PlayerColorRemap.cs

@@ -4,13 +4,12 @@ using System.IO;
 
 namespace OpenRa.FileFormats
 {
-	public class PaletteRemap : IPaletteRemap
+	public class PlayerColorRemap : IPaletteRemap
 	{
 		int offset;
 		List<Color> remapColors = new List<Color>();
-		Color shadowColor;
 
-		public PaletteRemap(Stream s)
+		public PlayerColorRemap(Stream s)
 		{
 			using (BinaryReader reader = new BinaryReader(s))
 			{
@@ -27,22 +26,12 @@ namespace OpenRa.FileFormats
 			offset = 80;
 		}
 
-		public PaletteRemap( Color shadowColor )
-		{
-			this.shadowColor = shadowColor;
-		}
-
 		public Color GetRemappedColor(Color original, int index)
 		{
-			if (remapColors.Count > 0)
+			if (index < offset || index >= offset + remapColors.Count)
-			{
+				return original;
-				if (index < offset || index >= offset + remapColors.Count)
-					return original;
 
-				return remapColors[index - offset];
+			return remapColors[index - offset];
-			}
-
-			return original.A > 0 ? shadowColor : original;
 		}
 	}
 }

OpenRa.FileFormats/SingleColorRemap.cs

@@ -0,0 +1,18 @@
+using System.Drawing;
+
+namespace OpenRa.FileFormats
+{
+	public class SingleColorRemap : IPaletteRemap
+	{
+		Color c;
+		public SingleColorRemap(Color c)
+		{
+			this.c = c;
+		}
+
+		public Color GetRemappedColor(Color original, int index)
+		{
+			return original.A > 0 ? c : original;
+		}
+	}
+}

OpenRa.Game/Graphics/HardwarePalette.cs

@@ -21,9 +21,9 @@ namespace OpenRa.Game.Graphics
 			AddPalette(pal);
 
 			foreach (string remap in new string[] { "blue", "red", "orange", "teal", "salmon", "green", "gray" })
-				AddPalette(new Palette(pal, new PaletteRemap(FileSystem.Open(remap + ".rem"))));
+				AddPalette(new Palette(pal, new PlayerColorRemap(FileSystem.Open(remap + ".rem"))));
 
-			AddPalette(new Palette(pal, new PaletteRemap(Color.FromArgb(140, 0, 0, 0))));
+			AddPalette(new Palette(pal, new SingleColorRemap(Color.FromArgb(140, 0, 0, 0))));
 			AddPalette(pal);	// iron curtain. todo: remap!
 			AddPalette(pal);	// chrome (it's like gold, but we're not going to hax it in palettemods)
 			AddPalette(new Palette(pal, new ShroudPaletteRemap()));