Browse Source

Clean up method patcher implementation a bit

Bepis 3 years ago
parent
commit
f419c909bc
1 changed files with 4 additions and 6 deletions
  1. 4 6
      BepInEx.IL2CPP/Hook/IL2CPPDetourMethodPatcher.cs

+ 4 - 6
BepInEx.IL2CPP/Hook/IL2CPPDetourMethodPatcher.cs

@@ -92,13 +92,11 @@ namespace BepInEx.IL2CPP.Hook
 
 			// Apply a detour from the unmanaged implementation to the patched harmony method
 
-			var unmanagedDelegateType = DelegateTypeFactory.instance.CreateDelegateType(unmanagedTrampolineMethod.ReturnType,
-				unmanagedTrampolineMethod.GetParameters().Select(x => x.ParameterType).ToArray());
-
-
-			var detourPtr = MonoExtensions.GetFunctionPointerForDelegate(unmanagedTrampolineMethod.CreateDelegate(unmanagedDelegateType),
+			var unmanagedDelegateType = DelegateTypeFactory.instance.CreateDelegateType(unmanagedTrampolineMethod,
 				CallingConvention.Cdecl);
 
+			var detourPtr = Marshal.GetFunctionPointerForDelegate(unmanagedTrampolineMethod.CreateDelegate(unmanagedDelegateType));
+
 			nativeDetour = new FastNativeDetour(originalNativeMethodInfo->methodPointer, detourPtr);
 
 			nativeDetour.Apply();
@@ -195,7 +193,7 @@ namespace BepInEx.IL2CPP.Hook
 				// Load thisptr as arg0
 
 				il.Emit(OpCodes.Ldarg_0);
-				il.Emit(OpCodes.Newobj, AccessTools.DeclaredConstructor(Original.DeclaringType, new[] { typeof(IntPtr) }));
+				EmitConvertArgumentToManaged(il, Original.DeclaringType, out _);
 			}
 
 			for (int i = 0; i < managedParams.Length; ++i)