Swap Before/After for legacy (IL)Hooks

EverestAPI · Aug 8, 2023 · 155656d · 155656d
1 parent c86c893
commit 155656d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/Celeste.Mod.mm/Mod/Helpers/LegacyMonoMod/Detour.cs b/Celeste.Mod.mm/Mod/Helpers/LegacyMonoMod/Detour.cs
@@ -242,7 +242,8 @@ private void _Refresh() {
 
             actualHook?.Dispose();
             if (IsApplied) {
-                actualHook = new Hook(Method, (MethodInfo) Target, new DetourConfig(ID, Priority, Before, After));
+                // Before/After are swapped on reorg ._.
+                actualHook = new Hook(Method, (MethodInfo) Target, new DetourConfig(ID, Priority, After, Before));
                 GC.SuppressFinalize(actualHook);
 
                 // Update the trampoline detour

diff --git a/Celeste.Mod.mm/Mod/Helpers/LegacyMonoMod/ILHook.cs b/Celeste.Mod.mm/Mod/Helpers/LegacyMonoMod/ILHook.cs
@@ -177,7 +177,8 @@ private void _Refresh() {
 
             actualHook?.Dispose();
             if (IsApplied) {
-                actualHook = new ILHook(Method, Manipulator, new DetourConfig(ID, Priority, Before, After));
+                // Before/After are swapped on reorg ._.
+                actualHook = new ILHook(Method, Manipulator, new DetourConfig(ID, Priority, After, Before));
                 GC.SuppressFinalize(actualHook);
             } else
                 actualHook = null;