Avoids unnecessary copy of patches in applyPatches

immerjs · Nov 18, 2021 · fe80e02 · fe80e02
1 parent abba680
commit fe80e02
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/core/immerClass.ts b/src/core/immerClass.ts
@@ -196,6 +196,11 @@ export class Immer implements ProducersFns {
 				break
 			}
 		}
+		// If there was a patch that replaced the entire state, start from the
+		// patch after that.
+		if (i > -1) {
+			patches = patches.slice(i + 1)
+		}
 
 		const applyPatchesImpl = getPlugin("Patches").applyPatches_
 		if (isDraft(base)) {
@@ -204,7 +209,7 @@ export class Immer implements ProducersFns {
 		}
 		// Otherwise, produce a copy of the base state.
 		return this.produce(base, (draft: Drafted) =>
-			applyPatchesImpl(draft, patches.slice(i + 1))
+			applyPatchesImpl(draft, patches)
 		)
 	}
 }