diff options
| author | Ivan Molodetskikh <yalterz@gmail.com> | 2025-09-06 11:10:23 +0300 |
|---|---|---|
| committer | Ivan Molodetskikh <yalterz@gmail.com> | 2025-09-17 22:04:23 +0300 |
| commit | 82276771a5e5590d5aac5847be84d99076815501 (patch) | |
| tree | 5bd017630c261a449cb07bcb8e63ff05256da829 /src | |
| parent | cd7108ef5dc30afd7d5aefb51d5e73fbd004afc1 (diff) | |
| download | niri-82276771a5e5590d5aac5847be84d99076815501.tar.gz niri-82276771a5e5590d5aac5847be84d99076815501.tar.bz2 niri-82276771a5e5590d5aac5847be84d99076815501.zip | |
Downgrade lack of our initial configure from error! to debug!
This can happen if a surface unmaps by committing a null buffer and then
immediately does the initial commit without a sync roundtrip, while there are
pending configures from the compositor in-flight. In this case, the surface
cannot tell that the pending configures were meant for "before unmapping" and
considers them to be the new initial configure.
From our point of view, we don't get to do a proper initial configuration
sequence in this case, and receive a mapping commit without our initial
configure state. We cannot really do much about it, but it is not an error when
this specific situation happens.
Diffstat (limited to 'src')
| -rw-r--r-- | src/handlers/compositor.rs | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/handlers/compositor.rs b/src/handlers/compositor.rs index ffad0750..034ee0d0 100644 --- a/src/handlers/compositor.rs +++ b/src/handlers/compositor.rs @@ -115,7 +115,9 @@ impl CompositorHandler for State { (rules, width, height, is_full_width, output, workspace_id) } else { - error!("window map must happen after initial configure"); + // Can happen when a surface unmaps by attaching a null buffer while + // there are in-flight pending configures. + debug!("window mapped without proper initial configure"); (ResolvedWindowRules::empty(), None, None, false, None, None) }; |
