Move SetSnapShot to separate files.

Author: oskarb

src/NHibernate/Collection/Generic/SetHelpers/ISetSnapshot.cs

@@ -0,0 +1,10 @@
+using System.Collections;
+using System.Collections.Generic;
+
+namespace NHibernate.Collection.Generic.SetHelpers
+{
+	internal interface ISetSnapshot<T> : ICollection<T>, ICollection
+	{
+		T this[T element] { get; }
+	}
+}

src/NHibernate/Collection/Generic/SetHelpers/SetSnapShot.cs

@@ -0,0 +1,104 @@
+using System;
+using System.Collections;
+using System.Collections.Generic;
+
+namespace NHibernate.Collection.Generic.SetHelpers
+{
+	[Serializable]
+	internal class SetSnapShot<T> : ISetSnapshot<T>
+	{
+		private readonly List<T> elements;
+		public SetSnapShot()
+		{
+			elements = new List<T>();
+		}
+
+		public SetSnapShot(int capacity)
+		{
+			elements = new List<T>(capacity);
+		}
+
+		public SetSnapShot(IEnumerable<T> collection)
+		{
+			elements = new List<T>(collection);
+		}
+
+		public IEnumerator<T> GetEnumerator()
+		{
+			return elements.GetEnumerator();
+		}
+
+		IEnumerator IEnumerable.GetEnumerator()
+		{
+			return GetEnumerator();
+		}
+
+		public void Add(T item)
+		{
+			elements.Add(item);
+		}
+
+		public void Clear()
+		{
+			throw new InvalidOperationException();
+		}
+
+		public bool Contains(T item)
+		{
+			return elements.Contains(item);
+		}
+
+		public void CopyTo(T[] array, int arrayIndex)
+		{
+			elements.CopyTo(array, arrayIndex);
+		}
+
+		public bool Remove(T item)
+		{
+			throw new InvalidOperationException();
+		}
+
+		public void CopyTo(Array array, int index)
+		{
+			((ICollection)elements).CopyTo(array, index);
+		}
+
+		int ICollection.Count
+		{
+			get { return elements.Count; }
+		}
+
+		public object SyncRoot
+		{
+			get { return ((ICollection)elements).SyncRoot; }
+		}
+
+		public bool IsSynchronized
+		{
+			get { return ((ICollection)elements).IsSynchronized; }
+		}
+
+		int ICollection<T>.Count
+		{
+			get { return elements.Count; }
+		}
+
+		public bool IsReadOnly
+		{
+			get { return ((ICollection<T>)elements).IsReadOnly; }
+		}
+
+		public T this[T element]
+		{
+			get
+			{
+				var idx = elements.IndexOf(element);
+				if (idx >= 0)
+				{
+					return elements[idx];
+				}
+				return default(T);
+			}
+		}
+	}
+}

src/NHibernate/Collection/PersistentSet.cs

@@ -4,6 +4,7 @@
 using System.Data;
 using System.Diagnostics;
 using Iesi.Collections;
+using NHibernate.Collection.Generic.SetHelpers;
 using NHibernate.DebugHelpers;
 using NHibernate.Engine;
 using NHibernate.Loader;
@@ -13,109 +14,6 @@
 
 namespace NHibernate.Collection
 {
-	internal interface ISetSnapshot<T> : ICollection<T>, ICollection
-	{
-		T this[T element] { get; }
-	}
-
-	[Serializable]
-	internal class SetSnapShot<T> : ISetSnapshot<T>
-	{
-		private readonly List<T> elements;
-		public SetSnapShot()
-		{
-			elements = new List<T>();
-		}
-
-		public SetSnapShot(int capacity)
-		{
-			elements = new List<T>(capacity);
-		}
-
-		public SetSnapShot(IEnumerable<T> collection)
-		{
-			elements = new List<T>(collection);
-		}
-
-		public IEnumerator<T> GetEnumerator()
-		{
-			return elements.GetEnumerator();
-		}
-
-		IEnumerator IEnumerable.GetEnumerator()
-		{
-			return GetEnumerator();
-		}
-
-		public void Add(T item)
-		{
-			elements.Add(item);
-		}
-
-		public void Clear()
-		{
-			throw new InvalidOperationException();
-		}
-
-		public bool Contains(T item)
-		{
-			return elements.Contains(item);
-		}
-
-		public void CopyTo(T[] array, int arrayIndex)
-		{
-			elements.CopyTo(array, arrayIndex);
-		}
-
-		public bool Remove(T item)
-		{
-			throw new InvalidOperationException();
-		}
-
-		public void CopyTo(Array array, int index)
-		{
-			((ICollection)elements).CopyTo(array, index);
-		}
-
-		int ICollection.Count
-		{
-			get { return elements.Count; }
-		}
-
-		public object SyncRoot
-		{
-			get { return ((ICollection)elements).SyncRoot; }
-		}
-
-		public bool IsSynchronized
-		{
-			get { return ((ICollection)elements).IsSynchronized; }
-		}
-
-		int ICollection<T>.Count
-		{
-			get { return elements.Count; }
-		}
-
-		public bool IsReadOnly
-		{
-			get { return ((ICollection<T>)elements).IsReadOnly; }
-		}
-
-		public T this[T element]
-		{
-			get
-			{
-				var idx = elements.IndexOf(element);
-				if (idx >= 0)
-				{
-					return elements[idx];
-				}
-				return default(T);
-			}
-		}
-	}
-
 	/// <summary>
 	/// .NET has no design equivalent for Java's Set so we are going to use the
 	/// Iesi.Collections library. This class is internal to NHibernate and shouldn't

src/NHibernate/NHibernate.csproj

@@ -131,6 +131,8 @@
     <Compile Include="Cfg\Settings.cs" />
     <Compile Include="Cfg\SettingsFactory.cs" />
     <Compile Include="Collection\IPersistentCollection.cs" />
+    <Compile Include="Collection\Generic\SetHelpers\ISetSnapshot.cs" />
+    <Compile Include="Collection\Generic\SetHelpers\SetSnapShot.cs" />
     <Compile Include="Connection\ConnectionProvider.cs" />
     <Compile Include="Connection\ConnectionProviderFactory.cs" />
     <Compile Include="Connection\DriverConnectionProvider.cs" />