refactor: one-time cleanups use for instead of foreach

Cumcord · Aug 20, 2024 · a929b5e · a929b5e
1 parent 3c2b5fc
commit a929b5e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "spitroast",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "A simple JavaScript function patcher.",
   "main": "dist/cjs.js",
   "module": "dist/esm/index.js",

diff --git a/src/hook.ts b/src/hook.ts
@@ -31,16 +31,16 @@ export default function (
     // This calls the original function
     (...args: unknown[]) =>
       isConstruct
-        ? Reflect.construct(patch.o, args, ctxt)
-        : patch.o.apply(ctxt, args)
+        ? Reflect.construct(origFunc, args, ctxt)
+        : origFunc.apply(ctxt, args)
   )(...funcArgs);
 
   // After patches
   for (const hook of patch.a.values())
     workingRetVal = hook.call(ctxt, funcArgs, workingRetVal) ?? workingRetVal;
 
   // Cleanups (one-times)
-  patch.c.forEach((c) => c());
+  for (const cleanup of patch.c) cleanup()
   patch.c = [];
 
   return workingRetVal;