tidy up

Author: Rich-Harris

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

@@ -97,7 +97,7 @@ export class Batch {
 			process_effects(this, root);
 		}
 
-		if (this.async_effects.length === 0 && this.settled()) {
+		if (this.async_effects.length === 0 && this.#pending === 0) {
 			var merged = false;
 
 			// if there are older batches with overlapping
@@ -191,25 +191,6 @@ export class Batch {
 
 	remove() {
 		batches.delete(this);
-
-		// for (var batch of batches) {
-		// 	/** @type {Source} */
-		// 	var source;
-
-		// 	if (batch.#id < this.#id) {
-		// 		// other batch is older than this
-		// 		for (source of this.#previous.keys()) {
-		// 			batch.#previous.delete(source);
-		// 		}
-		// 	} else {
-		// 		// other batch is newer than this
-		// 		for (source of batch.#previous.keys()) {
-		// 			if (this.#previous.has(source)) {
-		// 				batch.#previous.set(source, source.v);
-		// 			}
-		// 		}
-		// 	}
-		// }
 	}
 
 	restore() {
@@ -223,15 +204,14 @@ export class Batch {
 			return;
 		}
 
-		if (this.settled()) {
+		if (this.#pending === 0) {
 			this.remove();
 		}
 
 		current_batch = null;
 	}
 
 	commit() {
-		// commit changes
 		for (const fn of this.#callbacks) {
 			fn();
 		}
@@ -266,10 +246,6 @@ export class Batch {
 		}
 	}
 
-	settled() {
-		return this.#pending === 0;
-	}
-
 	/** @param {() => void} fn */
 	add_callback(fn) {
 		this.#callbacks.add(fn);