NH-3964 - remove obsolete workaround implying some reflection.

Author: fredericDelaporte

src/NHibernate.Test/NHSpecificTest/NH3964/Fixture.cs

@@ -0,0 +1,18 @@
+using System.Collections;
+using NHibernate.Util;
+using NUnit.Framework;
+
+namespace NHibernate.Test.NHSpecificTest.NH3964
+{
+	[TestFixture]
+	public class Fixture
+	{
+		[Test(Description = "Test for removal of a workaround for an old Fx bug (<v4)")]
+		public void AddingNullToNonGenericListShouldNotThrow()
+		{
+			var a1 = new ArrayList { null };
+			var a2 = new ArrayList();
+			Assert.DoesNotThrow(() => ArrayHelper.AddAll(a2, a1));
+		}
+	}
+}

src/NHibernate.Test/NHibernate.Test.csproj

@@ -742,6 +742,7 @@
     <Compile Include="NHSpecificTest\NH3961\DateParametersComparedTo.cs" />
     <Compile Include="NHSpecificTest\NH3963\Entity.cs" />
     <Compile Include="NHSpecificTest\NH3963\MappedAsFixture.cs" />
+    <Compile Include="NHSpecificTest\NH3964\Fixture.cs" />
     <Compile Include="NHSpecificTest\NH3950\Entity.cs" />
     <Compile Include="NHSpecificTest\NH3950\Fixture.cs" />
     <Compile Include="NHSpecificTest\NH3952\Entity.cs" />

src/NHibernate/Util/ArrayHelper.cs

@@ -95,40 +95,9 @@ public static string ToString(object[] array)
 		/// <param name="from"></param>
 		public static void AddAll(IList to, IList from)
 		{
-			System.Action addNull = null;
 			foreach (object obj in from)
 			{
-				// There is bug in .NET, before version 4, where adding null to a List<Nullable<T>> through the non-generic IList interface throws an exception.
-				// TODO: Everything but the to.Add(obj) should be conditionally compiled only for versions of .NET earlier than 4.
-				if (obj == null)
-				{
-					if (addNull == null)
-					{
-						var toType = to.GetType();
-						if (toType.IsGenericType &&
-							toType.GetGenericTypeDefinition() == typeof(List<>) &&
-							toType.GetGenericArguments()[0].IsNullable())
-						{
-							MethodInfo addMethod = toType.GetMethod("Add");
-							var addMethodCall =	System.Linq.Expressions.Expression.Call(System.Linq.Expressions.Expression.Constant(to),
-																		addMethod,
-																		System.Linq.Expressions.Expression.Constant(null, toType.GetGenericArguments()[0]));
-							System.Linq.Expressions.LambdaExpression addLambda =
-								System.Linq.Expressions.Expression.Lambda(addMethodCall);
-
-							addNull = (System.Action)addLambda.Compile();
-						}
-						else
-						{
-							addNull = () => to.Add(null);
-						}
-					}
-					addNull();
-				}
-				else
-				{
-					to.Add(obj);
-				}
+				to.Add(obj);
 			}
 		}