From 6e705f603415e5694377852491f5b8ee81091cca Mon Sep 17 00:00:00 2001
From: Vlad Zahorodnii <vlad.zahorodnii@kde.org>
Date: Fri, 24 Feb 2023 17:08:55 +0200
Subject: [PATCH] Simplify Window::setDesktop()

workspace()->updateOnAllDesktopsOfTransients() is not needed because we
call setDesktops() for transients immediately after it.
---
 src/window.cpp | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/window.cpp b/src/window.cpp
index bf6da445a4..8b75bbd363 100644
--- a/src/window.cpp
+++ b/src/window.cpp
@@ -1047,7 +1047,6 @@ void Window::setDesktops(QVector<VirtualDesktop *> desktops)
         return;
     }
 
-    const bool wasOnAllDesktops = isOnAllDesktops();
     m_desktops = desktops;
 
     if (windowManagementInterface()) {
@@ -1069,10 +1068,6 @@ void Window::setDesktops(QVector<VirtualDesktop *> desktops)
         }
     }
 
-    if (isOnAllDesktops() != wasOnAllDesktops) {
-        workspace()->updateOnAllDesktopsOfTransients(this);
-    }
-
     auto transients_stacking_order = workspace()->ensureStackingOrder(transients());
     for (auto it = transients_stacking_order.constBegin(); it != transients_stacking_order.constEnd(); ++it) {
         (*it)->setDesktops(desktops);