From d440a3992e8a0fa49857210e646b0d653dfa7675 Mon Sep 17 00:00:00 2001 From: srdusr <99972264+srdusr@users.noreply.github.com> Date: Fri, 21 Aug 2026 19:24:00 +0200 Subject: Put back the allow attribute my last commit orphaned Inserting the title-elision helper directly above render_titlebar pushed that function away from its own #[allow(clippy::too_many_arguments)], which then applied to a const and left the function warning again. Third time I have done this to an attribute in this tree; the pattern is inserting at a "just before this function" anchor without checking what sits immediately above it. Also `"x".repeat(20_000)` in the new test, which clippy asked for. Clippy is clean again - and it was not when I said it was in the previous commit: the check printed its own "ok" line unconditionally, so three real warnings scrolled past above it. --- crates/wayland/src/decoration/tests.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'crates/wayland/src/decoration/tests.rs') diff --git a/crates/wayland/src/decoration/tests.rs b/crates/wayland/src/decoration/tests.rs index 0f1a9ab..c777f06 100644 --- a/crates/wayland/src/decoration/tests.rs +++ b/crates/wayland/src/decoration/tests.rs @@ -1010,7 +1010,7 @@ fn a_titlebar_with_no_room_at_all_draws_no_title() { #[test] fn a_pathologically_long_title_is_bounded() { let Some(font) = super::font::find_system_font() else { return }; - let huge: String = std::iter::repeat('x').take(20_000).collect(); + let huge = "x".repeat(20_000); let laid_out = super::titlebar::lay_out_title(&font, &huge, 400.0); assert!(laid_out.len() < super::titlebar::MAX_TITLE_CHARS); } -- cgit v1.2.3