From 5af1b5ab22c1b202f3faa7f3c293d725771a1ff8 Mon Sep 17 00:00:00 2001 From: Hanno Braun Date: Tue, 25 Mar 2025 21:00:53 +0100 Subject: [PATCH] Inline redundant function --- crates/fj-viewer/src/viewer.rs | 5 ++++- crates/fj-viewer/src/window.rs | 10 ---------- 2 files changed, 4 insertions(+), 11 deletions(-) diff --git a/crates/fj-viewer/src/viewer.rs b/crates/fj-viewer/src/viewer.rs index 39e296ea7..82505bc57 100644 --- a/crates/fj-viewer/src/viewer.rs +++ b/crates/fj-viewer/src/viewer.rs @@ -177,7 +177,10 @@ impl ViewerWindow { /// Draw the graphics pub fn draw(&mut self) { - let size_is_valid = self.window.size().is_valid(); + let size_is_valid = { + let size = self.window.size(); + size.width > 0 && size.height > 0 + }; if !size_is_valid { return; } diff --git a/crates/fj-viewer/src/window.rs b/crates/fj-viewer/src/window.rs index b3f9df366..456f7c5e5 100644 --- a/crates/fj-viewer/src/window.rs +++ b/crates/fj-viewer/src/window.rs @@ -38,13 +38,3 @@ pub struct WindowSize { /// The height of the screen pub height: u32, } - -impl WindowSize { - /// # Indicate whether the screen size is valid - /// - /// A screen size is valid, if neither of its dimensions is zero. But it can - /// be reported as zero by spurious screen resize events. - pub fn is_valid(&self) -> bool { - self.width > 0 && self.height > 0 - } -}