Refactor classes to structs

Author: teinarss

OpenRA.Game/Map/CellRegion.cs

@@ -117,7 +117,7 @@ IEnumerator IEnumerable.GetEnumerator()
 			return GetEnumerator();
 		}
 
-		public sealed class CellRegionEnumerator : IEnumerator<CPos>
+		public struct CellRegionEnumerator : IEnumerator<CPos>
 		{
 			readonly CellRegion r;
 
@@ -128,9 +128,11 @@ public sealed class CellRegionEnumerator : IEnumerator<CPos>
 			CPos current;
 
 			public CellRegionEnumerator(CellRegion region)
+				: this()
 			{
 				r = region;
 				Reset();
+				current = new MPos(u, v).ToCPos(r.gridType);
 			}
 
 			public bool MoveNext()

OpenRA.Game/Map/ProjectedCellRegion.cs

@@ -76,7 +76,7 @@ IEnumerator IEnumerable.GetEnumerator()
 			return GetEnumerator();
 		}
 
-		public sealed class ProjectedCellRegionEnumerator : IEnumerator<PPos>
+		public struct ProjectedCellRegionEnumerator : IEnumerator<PPos>
 		{
 			readonly ProjectedCellRegion r;
 
@@ -86,9 +86,11 @@ public sealed class ProjectedCellRegionEnumerator : IEnumerator<PPos>
 			PPos current;
 
 			public ProjectedCellRegionEnumerator(ProjectedCellRegion region)
+				: this()
 			{
 				r = region;
 				Reset();
+				current = new PPos(u, v);
 			}
 
 			public bool MoveNext()

OpenRA.Game/TraitDictionary.cs

@@ -179,13 +179,14 @@ class MultipleEnumerable : IEnumerable<T>
 				System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator() { return GetEnumerator(); }
 			}
 
-			class MultipleEnumerator : IEnumerator<T>
+			struct MultipleEnumerator : IEnumerator<T>
 			{
 				readonly List<Actor> actors;
 				readonly List<T> traits;
 				readonly uint actor;
 				int index;
 				public MultipleEnumerator(TraitContainer<T> container, uint actor)
+					: this()
 				{
 					actors = container.actors;
 					traits = container.traits;
@@ -236,20 +237,21 @@ public IEnumerable<Actor> Actors(Func<T, bool> predicate)
 				}
 			}
 
-			class AllEnumerable : IEnumerable<TraitPair<T>>
+			struct AllEnumerable : IEnumerable<TraitPair<T>>
 			{
 				readonly TraitContainer<T> container;
 				public AllEnumerable(TraitContainer<T> container) { this.container = container; }
 				public IEnumerator<TraitPair<T>> GetEnumerator() { return new AllEnumerator(container); }
 				System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator() { return GetEnumerator(); }
 			}
 
-			class AllEnumerator : IEnumerator<TraitPair<T>>
+			struct AllEnumerator : IEnumerator<TraitPair<T>>
 			{
 				readonly List<Actor> actors;
 				readonly List<T> traits;
 				int index;
 				public AllEnumerator(TraitContainer<T> container)
+					: this()
 				{
 					actors = container.actors;
 					traits = container.traits;

OpenRA.Mods.Common/Traits/World/ActorMap.cs

@@ -216,29 +216,40 @@ void INotifyCreated.Created(Actor self)
 			}
 		}
 
-		sealed class ActorsAtEnumerator : IEnumerator<Actor>
+		struct ActorsAtEnumerator : IEnumerator<Actor>
 		{
 			InfluenceNode node;
-			public ActorsAtEnumerator(InfluenceNode node) { this.node = node; }
+			Actor current;
+
+			public ActorsAtEnumerator(InfluenceNode node)
+			{
+				this.node = node;
+				current = null;
+			}
+
 			public void Reset() { throw new NotSupportedException(); }
-			public Actor Current { get; private set; }
-			object IEnumerator.Current { get { return Current; } }
+			public Actor Current
+			{
+				get { return current; }
+			}
+
+			object IEnumerator.Current { get { return current; } }
 			public void Dispose() { }
 			public bool MoveNext()
 			{
 				while (node != null)
 				{
-					Current = node.Actor;
+					current = node.Actor;
 					node = node.Next;
-					if (!Current.Disposed)
+					if (!current.Disposed)
 						return true;
 				}
 
 				return false;
 			}
 		}
 
-		sealed class ActorsAtEnumerable : IEnumerable<Actor>
+		struct ActorsAtEnumerable : IEnumerable<Actor>
 		{
 			readonly InfluenceNode node;
 			public ActorsAtEnumerable(InfluenceNode node) { this.node = node; }