tidy

Author: dummdidumm

packages/svelte/src/internal/client/reactivity/batch.js

@@ -607,24 +607,19 @@ function flush_queued_effects(effects) {
 				/** @type {Effect[]} */
 				const filtered_effects = [];
 
-				for (const e of eager_block_effects) {
+				outer: for (const e of eager_block_effects) {
 					// Skip eager effects that have already been unmounted
 					if ((e.f & (DESTROYED | INERT)) !== 0) continue;
 
 					// Skip effects that have related parent effects in the current flush,
 					// as the inner ones will be run if its parent triggers it (eg. in a guard).
-					let skip = false;
 					let ancestor = e.parent;
-					while (!skip && ancestor !== null) {
+					while (ancestor !== null) {
 						if (eager_block_effects.has(ancestor)) {
-							skip = true;
-							break;
+							continue outer;
 						}
 						ancestor = ancestor.parent;
 					}
-					if (skip) {
-						continue;
-					}
 
 					filtered_effects.push(e);
 				}

packages/svelte/tests/runtime-runes/samples/guard-else-effect/main.svelte

@@ -1,21 +1,16 @@
 <script lang="ts">
-	import {untrack} from 'svelte';
-
-	const {trackBranch}:{trackBranch: (branch:string)=>void} = $props();
+	const { trackBranch }:{ trackBranch: (branch:string)=>void } = $props();
 
 	let b = $state(false);
 	let v = $state("two");
 
-
 	$effect(() => {
 		v = b ? "one" : "two";
 	})
 </script>
 
-
 <button onclick={() => b = !b}>Trigger</button>
 
-
 {#if v === "one"}
 	<div>if1 matched! {trackBranch('one')}</div>
 {:else if v === "two"}

packages/svelte/tests/runtime-runes/samples/guard-if-nested/main.svelte

@@ -1,19 +1,18 @@
 <script>
-	let centerRow = $state({ nested: { optional: 2, required: 3 } },
-		);
+	let centerRow = $state({ nested: { optional: 2, required: 3 } });
 
 	let someChange = $state(false);
 	$effect(() => {
 		if (someChange) centerRow = undefined;
 	});
 </script>
 
-	{#if centerRow?.nested}
-		{#if centerRow?.nested?.optional != undefined && centerRow.nested.optional > 0}
-			op: {centerRow.nested.optional}<br />
-		{:else}
-			req: {centerRow.nested.required}<br />
-		{/if}
+{#if centerRow?.nested}
+	{#if centerRow?.nested?.optional != undefined && centerRow.nested.optional > 0}
+		op: {centerRow.nested.optional}<br />
+	{:else}
+		req: {centerRow.nested.required}<br />
 	{/if}
+{/if}
 
 <button onclick={() => (someChange = true)}>Trigger</button>