srdusr
aboutsummaryrefslogtreecommitdiffstats
path: root/crates/wayland/src/winit/render.rs
diff options
context:
space:
mode:
authorsrdusr <[email protected]>2024-08-03 20:07:00 +0200
committersrdusr <[email protected]>2024-08-03 20:07:00 +0200
commit35a28a257644483495ac2bd6a5fb1e109e8c1f26 (patch)
tree37944a7a2a61bf7ff977fa6349742d1a53ed5ad3 /crates/wayland/src/winit/render.rs
parentf166df4d979162ae09b439c87cdc64f2b0981308 (diff)
downloadsrdwm-35a28a257644483495ac2bd6a5fb1e109e8c1f26.tar.gz
srdwm-35a28a257644483495ac2bd6a5fb1e109e8c1f26.zip
Fix doc-comment file references left stale by today's six module splits
~17 comments across the codebase still pointed at udev.rs/winit.rs/ state.rs by their old flat-file names after those became udev/, winit/, state/ directories - found while auditing what this work rushed, since the split verification (function/struct-name diffing, full test suite) checked structural correctness but never comment accuracy. Updated each to either the specific new file (e.g. "see state.rs's REPEAT_DELAY" -> "see state/mod.rs's REPEAT_DELAY", "udev.rs's monitors()" -> "udev/platform.rs's monitors()") or the bare module name where the reference was already generic ("the udev/winit backends", not a specific location).
Diffstat (limited to 'crates/wayland/src/winit/render.rs')
-rw-r--r--crates/wayland/src/winit/render.rs26
1 files changed, 13 insertions, 13 deletions
diff --git a/crates/wayland/src/winit/render.rs b/crates/wayland/src/winit/render.rs
index 3dda278..902aed4 100644
--- a/crates/wayland/src/winit/render.rs
+++ b/crates/wayland/src/winit/render.rs
@@ -50,7 +50,7 @@ impl WaylandPlatform {
// The right-click titlebar menu, if open - pushed first so it's
// topmost over every window (this backend draws no cursor of its
// own, see this module's doc comment, so there's no "stay under
- // the pointer" ordering concern like udev.rs's matching push has).
+ // the pointer" ordering concern like udev/render.rs's matching push has).
if let (Some(menu), Some(buffer)) = (self.state.context_menu.as_ref(), self.state.context_menu_buffer.as_ref()) {
let pos = (menu.pos.0 as f64, menu.pos.1 as f64);
match MemoryRenderBufferRenderElement::from_buffer(renderer, pos, buffer, None, None, None, Kind::Unspecified) {
@@ -72,7 +72,7 @@ impl WaylandPlatform {
// instrumenting a locally vendored smithay copy directly) turned
// out to be `sync_geometry`'s `Space::map_element` call silently
// re-stacking windows to the top of `Space`'s *own* internal
- // order as a side effect of updating position - see `state.rs`'s
+ // order as a side effect of updating position - see `state/tick.rs`'s
// `resync_stacking_order` doc comment for the full story and the
// fix that landed for it (called after every `map_element` since).
// This loop never reads `Space`'s order at all: `ids` below comes
@@ -86,13 +86,13 @@ impl WaylandPlatform {
let ids: Vec<WindowId> = self.wm.borrow().visible_windows_front_to_back().map(|w| w.id).collect();
let focused = self.wm.borrow().focused_id();
// Popups next: always above every window's own content - see the
- // matching comment in `udev.rs`'s render loop for why this has to
+ // matching comment in `udev/render.rs`'s render loop for why this has to
// be pushed ahead of both the bar/dock and every window now that
// content shares this same list.
let popup_targets = crate::elements::popup_targets(&self.state);
custom_elements.extend(crate::elements::popup_render_elements(&popup_targets, renderer, (0, 0)).into_iter().map(crate::rounded_corners::WinitElement::Base));
// The bar/dock/launcher, skipped entirely for a fullscreen window --
- // see `udev.rs`'s matching push for the full reasoning.
+ // see `udev/render.rs`'s matching push for the full reasoning.
let hide_top_layers = self.wm.borrow().visible_windows_front_to_back().any(|w| w.fullscreen);
// `None` (the user's config never touched `general.rounded_corners`)
// defaults to *on* here - this backend has an actual GPU shader
@@ -111,7 +111,7 @@ impl WaylandPlatform {
// being built right now - `ids` is already front-to-back, so this
// only ever needs appending to, not recomputing. A window's own
// *content*, pushed inside this same loop below, needs no separate
- // occlusion test - see the matching comment in `udev.rs`'s render
+ // occlusion test - see the matching comment in `udev/render.rs`'s render
// loop for why ordinary front-to-back push order already occludes
// it correctly. The border strips and titlebar bitmap are
// different: outside `geometry`, so they still need `occluders`'
@@ -121,11 +121,11 @@ impl WaylandPlatform {
let Some(w) = self.wm.borrow().window(id).cloned() else { continue };
// `w.geometry` is the animation's target, not necessarily where
// the window is actually drawn this frame - see the matching
- // comment in `udev.rs`'s render loop for the full story
+ // comment in `udev/render.rs`'s render loop for the full story
// (reported live as the border "not flush" with the window
// during an animated maximize/fullscreen/open-slide transition).
let geom = self.state.window_anims.get(&id).map(crate::state::WindowAnim::current_rect).unwrap_or(w.geometry);
- // Same reasoning as udev.rs's matching push: positioned from
+ // Same reasoning as udev/render.rs's matching push: positioned from
// `geom`, not `w.geometry`, and not fragment-clipped against
// `occluders` - see that comment.
if let Some(shadow) = self.state.shadow_buffers.get(&id) {
@@ -137,7 +137,7 @@ impl WaylandPlatform {
}
}
if let Some(deco) = self.state.decorations.get(&id) {
- // Fragment-clipped, same as udev.rs's matching titlebar
+ // Fragment-clipped, same as udev/render.rs's matching titlebar
// push - see that comment for why all-or-nothing (skip
// only once *fully* covered) wasn't enough: a titlebar
// only partially covered, the common case for cascaded
@@ -172,7 +172,7 @@ impl WaylandPlatform {
// why a per-frame rebuild of either was a real, continuous
// cost, not a cosmetic one. Not fragment-clipped like the
// other three below - see the matching comment in
- // `udev.rs` for why the top strip only gets the cheaper
+ // `udev/render.rs` for why the top strip only gets the cheaper
// all-or-nothing occlusion check.
if strips[0].width > 0 && strips[0].height > 0 && !strips[0].subtract_all(&occluders).is_empty() {
if let Some(buffer) = self.state.border_top_decorations.get(&id) {
@@ -196,9 +196,9 @@ impl WaylandPlatform {
}
}
// The window's own content, at its own `opacity` - see the
- // matching push in `udev.rs`'s render loop for why. Single
+ // matching push in `udev/render.rs`'s render loop for why. Single
// output at the global origin, so no offset to subtract (see
- // `elements.rs`'s doc comment on why `udev.rs`'s per-head call
+ // `elements.rs`'s doc comment on why `udev/render.rs`'s per-head call
// does). Rounded via `rounded_corners::rounded_content_element`
// when the feature's on and the shader compiled - a decorated
// window only rounds its bottom two corners (the top two are
@@ -235,7 +235,7 @@ impl WaylandPlatform {
.map(crate::rounded_corners::WinitElement::Base),
);
- // Not `smithay::desktop::space::render_output`: see `udev.rs`'s
+ // Not `smithay::desktop::space::render_output`: see `udev/render.rs`'s
// matching call site for why (per-window opacity, fullscreen-aware
// layer-shell inclusion - `custom_elements` above already carries
// everything that wrapper would have built).
@@ -293,7 +293,7 @@ impl WaylandPlatform {
}
// Layer-shell surfaces get their callback every pass, unconditionally
// - NOT folded into the `has_damage` gate above. See the matching
- // (much longer) comment in udev.rs's `render_udev_frame`: many
+ // (much longer) comment in udev/render.rs's `render_udev_frame`: many
// layer-shell clients (GTK4/AGS among them) drive their entire
// repaint loop off frame callbacks with no independent timer
// fallback, so withholding the callback until *something* on the