diff options
| author | srdusr <[email protected]> | 2025-02-21 14:26:00 +0200 |
|---|---|---|
| committer | srdusr <[email protected]> | 2025-02-21 14:26:00 +0200 |
| commit | d303c276e289c9c7146cf74ccad2743246922248 (patch) | |
| tree | af79ce8f6e3cb3a2e57b34490ffa46e588899c5f /crates/wayland/src/state/lifecycle.rs | |
| parent | 681a023b04b513b3283d85fe157d6b38987bc42e (diff) | |
| download | srdwm-d303c276e289c9c7146cf74ccad2743246922248.tar.gz srdwm-d303c276e289c9c7146cf74ccad2743246922248.zip | |
Remove resolved corner-rounding diagnostics
The top-rounds-but-bottom-doesn't investigation these were tracking is
closed: live-verified via a real screenshot (pixel-level, not
eyeballed) that both corners round correctly on both a decorated
window and an undecorated/CSD one relying on content masking. Removes
three log::debug! blocks (corner-mask state, TOP/BOTTOM border strip
position dumps, and a raw alpha-byte dump of the bottom border buffer)
that were firing on every single render pass regardless of whether
anything changed, adding real per-frame overhead for output no longer
needed. Build + clippy + test (378 passing) all still green.
Diffstat (limited to 'crates/wayland/src/state/lifecycle.rs')
| -rw-r--r-- | crates/wayland/src/state/lifecycle.rs | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/crates/wayland/src/state/lifecycle.rs b/crates/wayland/src/state/lifecycle.rs index 259979b..1de8563 100644 --- a/crates/wayland/src/state/lifecycle.rs +++ b/crates/wayland/src/state/lifecycle.rs @@ -211,31 +211,6 @@ impl CompState { } if strips[1].width > 0 && strips[1].height > 0 { let data = decoration::render_border_bottom(strips[1].width, w.border_width, color, w.corner_radius); - // Temporary: chasing a live report that the bottom two - // corners render as a solid, uncurved block for the buffer's - // own "extra" rows (0..height-thickness) while the nominal - // rows (height-thickness..height) curve correctly. Dumps the - // alpha byte at x=0..11 for row 0 (should already show some - // cutting per a standalone simulation of this exact - // algorithm) and the last nominal row, straight out of the - // buffer this function just built - before it's wrapped - // into a MemoryRenderBuffer at all, so this is ground truth - // for whether `render_border_bottom` itself is the problem - // or something downstream of it is. Remove once resolved. - let w_usize = strips[1].width.max(1) as usize; - let h_usize = strip_h.max(1) as usize; - let alpha_row = |row: usize| -> Vec<u8> { - (0..12.min(w_usize)).map(|x| data.get((row * w_usize + x) * 4 + 3).copied().unwrap_or(255)).collect() - }; - log::debug!( - "udev::lifecycle: BOTTOM border buffer for {} (id {id:?}): dims={w_usize}x{h_usize} row0_alpha={:?} row3_alpha={:?} row7_alpha={:?} row8_alpha={:?} row11_alpha={:?}", - w.app_id, - alpha_row(0), - alpha_row(3.min(h_usize.saturating_sub(1))), - alpha_row(7.min(h_usize.saturating_sub(1))), - alpha_row(8.min(h_usize.saturating_sub(1))), - alpha_row(11.min(h_usize.saturating_sub(1))), - ); let buffer = MemoryRenderBuffer::from_slice(&data, Fourcc::Argb8888, (strips[1].width as i32, strip_h as i32), 1, Transform::Normal, None); self.border_bottom_decorations.insert(id, buffer); } else { |