dedupe

Rich-Harris · Rich-Harris · commit f92dfc158aeb · 2025-07-22T19:00:55.000-04:00
diff --git a/packages/svelte/src/internal/client/reactivity/sources.js b/packages/svelte/src/internal/client/reactivity/sources.js
@@ -324,14 +324,16 @@ export function mark_reactions(signal, status, schedule_async = true) {
 			continue;
 		}
 
+		var should_schedule = (flags & DIRTY) === 0 && (schedule_async || (flags & ASYNC) === 0);
+
 		// don't set a DIRTY reaction to MAYBE_DIRTY
-		if ((flags & DIRTY) === 0 && (schedule_async || (flags & ASYNC) === 0)) {
+		if (should_schedule) {
 			set_signal_status(reaction, status);
 		}
 
 		if ((flags & DERIVED) !== 0) {
 			mark_reactions(/** @type {Derived} */ (reaction), MAYBE_DIRTY);
-		} else if ((flags & DIRTY) === 0 && (schedule_async || (flags & ASYNC) === 0)) {
+		} else if (should_schedule) {
 			schedule_effect(/** @type {Effect} */ (reaction));
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -324,14 +324,16 @@ export function mark_reactions(signal, status, schedule_async = true) {`
`324`	`324`	`continue;`
`325`	`325`	`}`
`326`	`326`
	`327`	`+ var should_schedule = (flags & DIRTY) === 0 && (schedule_async \|\| (flags & ASYNC) === 0);`
	`328`	`+`
`327`	`329`	`// don't set a DIRTY reaction to MAYBE_DIRTY`
`328`		`- if ((flags & DIRTY) === 0 && (schedule_async \|\| (flags & ASYNC) === 0)) {`
	`330`	`+ if (should_schedule) {`
`329`	`331`	`set_signal_status(reaction, status);`
`330`	`332`	`}`
`331`	`333`
`332`	`334`	`if ((flags & DERIVED) !== 0) {`
`333`	`335`	`mark_reactions(/** @type {Derived} */ (reaction), MAYBE_DIRTY);`
`334`		`- } else if ((flags & DIRTY) === 0 && (schedule_async \|\| (flags & ASYNC) === 0)) {`
	`336`	`+ } else if (should_schedule) {`
`335`	`337`	`schedule_effect(/** @type {Effect} */ (reaction));`
`336`	`338`	`}`
`337`	`339`	`}`