From 8b8f0590168cfcc88c55c1eb38272a1f361a9457 Mon Sep 17 00:00:00 2001 From: Rich Harris Date: Mon, 7 Jul 2025 21:35:34 -0400 Subject: [PATCH] simplify --- .../src/internal/client/reactivity/batch.js | 28 ++++++------------- 1 file changed, 8 insertions(+), 20 deletions(-) diff --git a/packages/svelte/src/internal/client/reactivity/batch.js b/packages/svelte/src/internal/client/reactivity/batch.js index d55e388723..aaedbb7ac7 100644 --- a/packages/svelte/src/internal/client/reactivity/batch.js +++ b/packages/svelte/src/internal/client/reactivity/batch.js @@ -273,29 +273,17 @@ export class Batch { if (!skip && effect.fn !== null) { if (is_branch) { effect.f ^= CLEAN; - } else if ((flags & EFFECT_ASYNC) !== 0) { - const boundary = effect.b; - - if (check_dirtiness(effect)) { - var effects = boundary?.pending ? this.#boundary_async_effects : this.#async_effects; + } else if ((flags & EFFECT) !== 0) { + this.#effects.push(effect); + } else if (async_mode_flag && (flags & RENDER_EFFECT) !== 0) { + this.#render_effects.push(effect); + } else if (check_dirtiness(effect)) { + if ((flags & EFFECT_ASYNC) !== 0) { + var effects = effect.b?.pending ? this.#boundary_async_effects : this.#async_effects; effects.push(effect); - } - } else if ((flags & BLOCK_EFFECT) !== 0) { - if (check_dirtiness(effect)) { - update_effect(effect); - } - } else if ((flags & RENDER_EFFECT) !== 0) { - // we need to branch here because in legacy mode we run render effects - // before running block effects - if (async_mode_flag) { - this.#render_effects.push(effect); } else { - if (check_dirtiness(effect)) { - update_effect(effect); - } + update_effect(effect); } - } else if ((flags & EFFECT) !== 0) { - this.#effects.push(effect); } var child = effect.first;