Fix different generic parameters returning the same delegate

[DemoJameson]

No description provided.

[tristanmcpherson]

Interesting, I find it funny we haven't run into this before, I'm guessing this wasn't introduced recently either?

[DemoJameson]

it wasn't introduced recently, I tried reverting to the CastDelegate<T>() extension method from the forced casting, but it still doesn't work

        var person = new Person();
        Console.WriteLine(person.GetFieldValue<object>("name"));
        Console.WriteLine(person.GetFieldValue<string>("name"));

Unhandled Exception:
System.ArgumentException: method return type is incompatible
  at System.Delegate.CreateDelegate (System.Type type, System.Object firstArgument, System.Reflection.MethodInfo method, System.Boolean throwOnBindFailure, System.Boolean allowClosed) [0x00069] in <32116eccb94d4ed685ca661d98e36637>:0
  at System.Delegate.CreateDelegate (System.Type type, System.Object firstArgument, System.Reflection.MethodInfo method) [0x00000] in <32116eccb94d4ed685ca661d98e36637>:0
  at System.Reflection.RuntimeMethodInfo.CreateDelegate (System.Type delegateType, System.Object target) [0x00000] in <32116eccb94d4ed685ca661d98e36637>:0
  at MonoMod.Utils.Extensions.CastDelegate (System.Delegate source, System.Type type) [0x00023] in <c03b1f3aed35488e835d604b9374363f>:0
  at MonoMod.Utils.Extensions.CastDelegate[T] (System.Delegate source) [0x00000] in <c03b1f3aed35488e835d604b9374363f>:0
  at R2API.Utils.Reflection.GetFieldGetDelegate[TReturn] (System.Reflection.FieldInfo field) [0x0002a] in <c76c8ea2ea7548b8b89f3e6f45901cd4>:0
  at R2API.Utils.Reflection.GetFieldValue[TReturn] (System.Object instance, System.String fieldName) [0x00011] in <c76c8ea2ea7548b8b89f3e6f45901cd4>:0

[tristanmcpherson]

it wasn't introduced recently, I tried reverting to the CastDelegate<T>() extension method from the forced casting, but it still doesn't work

        var person = new Person();
        Console.WriteLine(person.GetFieldValue<object>("name"));
        Console.WriteLine(person.GetFieldValue<string>("name"));

Unhandled Exception:
System.ArgumentException: method return type is incompatible
  at System.Delegate.CreateDelegate (System.Type type, System.Object firstArgument, System.Reflection.MethodInfo method, System.Boolean throwOnBindFailure, System.Boolean allowClosed) [0x00069] in <32116eccb94d4ed685ca661d98e36637>:0
  at System.Delegate.CreateDelegate (System.Type type, System.Object firstArgument, System.Reflection.MethodInfo method) [0x00000] in <32116eccb94d4ed685ca661d98e36637>:0
  at System.Reflection.RuntimeMethodInfo.CreateDelegate (System.Type delegateType, System.Object target) [0x00000] in <32116eccb94d4ed685ca661d98e36637>:0
  at MonoMod.Utils.Extensions.CastDelegate (System.Delegate source, System.Type type) [0x00023] in <c03b1f3aed35488e835d604b9374363f>:0
  at MonoMod.Utils.Extensions.CastDelegate[T] (System.Delegate source) [0x00000] in <c03b1f3aed35488e835d604b9374363f>:0
  at R2API.Utils.Reflection.GetFieldGetDelegate[TReturn] (System.Reflection.FieldInfo field) [0x0002a] in <c76c8ea2ea7548b8b89f3e6f45901cd4>:0
  at R2API.Utils.Reflection.GetFieldValue[TReturn] (System.Object instance, System.String fieldName) [0x00011] in <c76c8ea2ea7548b8b89f3e6f45901cd4>:0

Guess it wasn't a very highly used feature in that scenario then, good catch