From 42ffd06d1ee14b5c36d4798f8a43c977ef067ad1 Mon Sep 17 00:00:00 2001 From: Shawn Wallace Date: Tue, 7 Jan 2025 21:41:28 -0500 Subject: [PATCH] xstate: avoid crashing if setting fullscreen state fails Could be triggered by spamming alt+z with gpu-screen-recorder-ui. --- src/xstate/mod.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/xstate/mod.rs b/src/xstate/mod.rs index d430ee8..219927b 100644 --- a/src/xstate/mod.rs +++ b/src/xstate/mod.rs @@ -897,15 +897,17 @@ impl XConnection for RealConnection { } else { &[] }; - self.connection + + if let Err(e) = self.connection .send_and_check_request(&x::ChangeProperty:: { mode: x::PropMode::Replace, window, property: atoms.net_wm_state, r#type: x::ATOM_ATOM, data, - }) - .unwrap(); + }) { + warn!("Failed to set fullscreen state on {window:?} ({e})"); + } } fn focus_window(