From 681a023b04b513b3283d85fe157d6b38987bc42e Mon Sep 17 00:00:00 2001 From: srdusr <99972264+srdusr@users.noreply.github.com> Date: Tue, 18 Feb 2025 21:15:00 +0200 Subject: Fix clippy warnings surfaced by the merge Pre-existing issues in the uncommitted rust-rewrite work (unused imports, over-arity glyph-drawing functions, a couple of complex inline types, or_insert_with(T::default) instead of or_default(), and one unsimplified test-only arithmetic expression), plus two imports left unused by switching to or_default(). None of these are behavior changes. Full workspace build + clippy -D warnings + test suite (378 tests) all green after this. --- crates/wayland/src/udev/render.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'crates/wayland/src/udev/render.rs') diff --git a/crates/wayland/src/udev/render.rs b/crates/wayland/src/udev/render.rs index 3a62292..bbabc40 100644 --- a/crates/wayland/src/udev/render.rs +++ b/crates/wayland/src/udev/render.rs @@ -154,6 +154,7 @@ impl CompState { // frame-callback loop below (after `udev` is no longer borrowed) // can notify only the windows that damage actually overlapped -- // see `windows_touched_by_damage`'s doc comment in elements.rs. + #[allow(clippy::type_complexity)] let mut presented: Vec<(Output, Point, Vec>)> = Vec::new(); for (index, output) in ready { let lock_surface = self.lock_surface_for(&output).cloned(); @@ -193,7 +194,7 @@ impl CompState { // pointer itself. See `color_filter::render_element` for // why an overlay rather than a true per-pixel shader. let color_filter = self.wm.borrow().color_filter; - let buf = self.color_filter_buffers.entry(output.name()).or_insert_with(SolidColorBuffer::default); + let buf = self.color_filter_buffers.entry(output.name()).or_default(); if let Some(elem) = crate::color_filter::render_element(buf, color_filter, hsize) { custom_elements.push(crate::elements::OverlayElement::Solid(elem)); } -- cgit v1.2.3