refactor: move toNumber call out of loop

chrislone · Feb 4, 2023 · 8d637f6 · 8d637f6
1 parent c3f22c4
commit 8d637f6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/reactivity/src/effect.ts b/packages/reactivity/src/effect.ts
@@ -276,8 +276,9 @@ export function trigger(
     // trigger all effects for target
     deps = [...depsMap.values()]
   } else if (key === 'length' && isArray(target)) {
+    const newLength = toNumber(newValue)
     depsMap.forEach((dep, key) => {
-      if (key === 'length' || key >= toNumber(newValue)) {
+      if (key === 'length' || key >= newLength) {
         deps.push(dep)
       }
     })