.NET returns the method for the last delegate of the multicast delegate.

The comparison was failing because we were still using the method field (instead of the Method property), which would then return, in the case of a delegate to a virtual function, the method on the base class, instead of the method on the derived class. Updated the test cases to check that too.

Replace the reversed linked implementation by an array based implementation. This will improve Combine performance, as well as ease delegate to virtual function optimization.

The previous fix to handle curried delegates with DynamicInvoke did not correctly handle
those create with IL (in contrast to Delegate.CreateDelegate), as they don't hit any managed code.

The solution is to always require DelegateData to be filled for InvokeDynamic, if not, lazily
calculate it.

[corlib] Fix Delegate::DynamicInvoke for open delegates of instance methods and closed delegates of static methods

This should aid in tracking down bugs such as #11871 (or possibly even
fixing them) - if the problem still exists the process will run into
an infinite loop, which is a lot easier to debug (at least on iOS, and
likely other platforms too) than a stackoverflow.

	* Delegate.cs (CreateDelegate): Allow binding del(vt&) to an instance
	method of vt.

	* DelegateTest.cs: Add regression tests.

	Fixes #695978

	* Delegate.cs (CreateDelegate): Fix visibility and make
	sure that we fail to bind to first argument of static
	arguments to overloads that don't pass it.

	* DelegateTest.cs: Add tests for the previous change.

	Fixes #655439

2ec0ab59 caused a regression in
Create.Delegate where the overload which doesn't take a firstArgument
could be used to create closed over null delegates while this
overload doesn't allow it.

svn path=/trunk/mcs/; revision=154243

	* Delegate.cs (GetHashCode): Fix this so it is constant.

svn path=/trunk/mcs/; revision=146903

	* Jumbo patch to drop support for pre-NET_2_0 code:

	Remove ONLY_1_1 code
	Remove NET_1_0 only code
	Remove NET_2_0 defines assuming the value is true.

	Improve one MonoTODO;   Implement one MonoTODO/Niex.

	Ran a textual diff on the resulting assembly to validate that the
	restul is the same on NET_2_0


svn path=/trunk/mcs/; revision=144298

	* Delegate.cs (CreateDelegate): Support for delegates
	on static methods closed on a null target.


svn path=/trunk/mcs/; revision=130324

	* Delegate.cs: Add new parameter to CreateDelegate_internal (icall)
	* Environment.cs: Bump mono_corlib_version to 73


svn path=/trunk/mcs/; revision=130213

	* Delegate.cs (CreateDelegate): fix checks to deal with
	valuetypes -> obj conversions for arguments, and avoid
	issues with such unallowed conversion for return types.


svn path=/trunk/mcs/; revision=130175

svn path=/trunk/mcs/; revision=111730

svn path=/trunk/mcs/; revision=111713

	* Delegate.cs: Move the rarely used fields to a separate object, add
	'method_code' field.

	* Environment.cs: Bump corlib version.

svn path=/trunk/mcs/; revision=108706

2008-07-04  Rodrigo Kumpera  <rkumpera@novell.com>

	* Delegate.cs (get_Method): Use the internal version
	of GetMethodFromHandle to avoid the generic class check.

In Test/System:
2008-07-04 Rodrigo Kumpera  <rkumpera@novell.com>

	* DelegateTest.cs (GetMethodFromGenericClass): Test for a
	regression the fix for #377324 introduced.


svn path=/trunk/mcs/; revision=107259

	* Delegate.cs: Fix parameter names for 2.0 profile
	* Int64.cs: Fix parameter names

svn path=/trunk/mcs/; revision=107221

	* Predicate.cs:
	* Object.cs:
	* Nullable.cs
	* MultiCastDelegate.cs
	* ModuleHandle.cs
	* Math.cs
	* MarshalByRefObject.cs
	* Int64.cs
	* Int32.cs
	* Int16.cs
	* Environment.cs
	* Delegate.cs
	* DateTimeOffset.cs
	* ConsoleKeyInfo.cs
	* Console.cs
	* Array.cs
	* AppDomain.cs: Fix parameter names

svn path=/trunk/mcs/; revision=107197