vitalif
/
kwin
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove no longer relevant case in Workspace::replaceInStack() 

Unmanaged windows are always in the stack now, so the else branch is not
needed, and in fact, it can produce undesired side effects if unmapped
window is closed.

(cherry picked from commit 298d63bc71a3d9b161849259c8309f677e77f939)
icc-effect-5.27.2
Vlad Zahorodnii 2023-02-26 15:11:09 +02:00
parent 9d515d3abb
commit 98e01c4f44
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/workspace.cpp
View File

@ -857,17 +857,11 @@ void Workspace::replaceInStack(Window *original, Deleted *deleted)
const int unconstraintedIndex = unconstrained_stacking_order.indexOf(original);
if (unconstraintedIndex != -1) {
unconstrained_stacking_order.replace(unconstraintedIndex, deleted);
} else {
// This can be the case only if an override-redirect window is unmapped.
unconstrained_stacking_order.append(deleted);
}
const int index = stacking_order.indexOf(original);
if (index != -1) {
stacking_order.replace(index, deleted);
} else {
// This can be the case only if an override-redirect window is unmapped.
stacking_order.append(deleted);
}
for (Constraint *constraint : std::as_const(m_constraints)) {