From b2193f370d7dffc589921e7374904236456828a5 Mon Sep 17 00:00:00 2001 From: Vlad Zahorodnii Date: Mon, 29 Apr 2024 13:41:49 +0300 Subject: [PATCH] Drop the remaining geometry updates blockers in Window While these geometry updates blockers are used in functions that can call moveResize() several times in a row, it's not critical. On the other hand, by removing these blockers, no other core code is going to rely on GeometryUpdatesBlocker, which opens us the future opportunitinies to drop it and make geometry manipulation code simpler. --- src/window.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/window.cpp b/src/window.cpp index d09338f7eb..7cb59881eb 100644 --- a/src/window.cpp +++ b/src/window.cpp @@ -1257,7 +1257,6 @@ bool Window::startInteractiveMoveResize() void Window::finishInteractiveMoveResize(bool cancel) { const bool wasMove = isInteractiveMove(); - GeometryUpdatesBlocker blocker(this); leaveInteractiveMoveResize(); doFinishInteractiveMoveResize(); @@ -3469,8 +3468,6 @@ void Window::setQuickTileMode(QuickTileMode mode, bool keyboard) workspace()->updateFocusMousePosition(Cursors::self()->mouse()->pos()); // may cause leave event - GeometryUpdatesBlocker blocker(this); - const QuickTileMode oldMode = requestedQuickTileMode(); QPointF whichScreen = keyboard ? moveResizeGeometry().center() : Cursors::self()->mouse()->pos();