From a3764b9713befbea432a6c49b015c04fcfad622d Mon Sep 17 00:00:00 2001 From: Elena Torro Date: Wed, 4 Feb 2026 11:01:58 +0100 Subject: [PATCH] :wrench: Avoid clone in rebuild_touched_tiles Use std::mem::take instead of clone to avoid HashSet allocation. The set was cleared anyway by clean_touched(), so take() is safe. --- render-wasm/src/render.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/render-wasm/src/render.rs b/render-wasm/src/render.rs index b1968e7e99..06164bec1b 100644 --- a/render-wasm/src/render.rs +++ b/render-wasm/src/render.rs @@ -2307,7 +2307,7 @@ impl RenderState { let mut all_tiles = HashSet::::new(); - let ids = self.touched_ids.clone(); + let ids = std::mem::take(&mut self.touched_ids); for shape_id in ids.iter() { if let Some(shape) = tree.get(shape_id) { @@ -2322,8 +2322,6 @@ impl RenderState { self.remove_cached_tile(tile); } - self.clean_touched(); - performance::end_measure!("rebuild_touched_tiles"); }