From b6b3e9b9f995abf5cc65814e143418b6f1ec7464 Mon Sep 17 00:00:00 2001
From: Héctor Ramón Jiménez <hector@hecrj.dev>
Date: Thu, 18 Jan 2024 10:42:02 +0100
Subject: Avoid stacking new primitives when undamaged

---
 tiny_skia/src/window/compositor.rs | 40 ++++++++++++++++++++------------------
 1 file changed, 21 insertions(+), 19 deletions(-)

(limited to 'tiny_skia/src')

diff --git a/tiny_skia/src/window/compositor.rs b/tiny_skia/src/window/compositor.rs
index c0aabdb6..dae57975 100644
--- a/tiny_skia/src/window/compositor.rs
+++ b/tiny_skia/src/window/compositor.rs
@@ -185,29 +185,31 @@ pub fn present<T: AsRef<str>>(
         })
         .unwrap_or_else(|| vec![Rectangle::with_size(viewport.logical_size())]);
 
+    if damage.is_empty() {
+        return Ok(());
+    }
+
     surface.primitive_stack.push_front(primitives.to_vec());
     surface.background_color = background_color;
 
-    if !damage.is_empty() {
-        let damage = damage::group(damage, scale_factor, physical_size);
+    let damage = damage::group(damage, scale_factor, physical_size);
 
-        let mut pixels = tiny_skia::PixmapMut::from_bytes(
-            bytemuck::cast_slice_mut(&mut buffer),
-            physical_size.width,
-            physical_size.height,
-        )
-        .expect("Create pixel map");
-
-        backend.draw(
-            &mut pixels,
-            &mut surface.clip_mask,
-            primitives,
-            viewport,
-            &damage,
-            background_color,
-            overlay,
-        );
-    }
+    let mut pixels = tiny_skia::PixmapMut::from_bytes(
+        bytemuck::cast_slice_mut(&mut buffer),
+        physical_size.width,
+        physical_size.height,
+    )
+    .expect("Create pixel map");
+
+    backend.draw(
+        &mut pixels,
+        &mut surface.clip_mask,
+        primitives,
+        viewport,
+        &damage,
+        background_color,
+        overlay,
+    );
 
     buffer.present().map_err(|_| compositor::SurfaceError::Lost)
 }
-- 
cgit