yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Drop the remaining geometry updates blockers in Window

Browse source 
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.
This commit is contained in:
Vlad Zahorodnii 2024-04-29 13:41:49 +03:00
parent e4e5c52300
commit b2193f370d
1 changed files with 0 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/window.cpp
View file

@ -1257,7 +1257,6 @@ bool Window::startInteractiveMoveResize()
void Window::finishInteractiveMoveResize(bool cancel) void Window::finishInteractiveMoveResize(bool cancel)
{ {
const bool wasMove = isInteractiveMove(); const bool wasMove = isInteractiveMove();
GeometryUpdatesBlocker blocker(this);
leaveInteractiveMoveResize(); leaveInteractiveMoveResize();
doFinishInteractiveMoveResize(); doFinishInteractiveMoveResize();
@ -3469,8 +3468,6 @@ void Window::setQuickTileMode(QuickTileMode mode, bool keyboard)
workspace()->updateFocusMousePosition(Cursors::self()->mouse()->pos()); // may cause leave event workspace()->updateFocusMousePosition(Cursors::self()->mouse()->pos()); // may cause leave event
GeometryUpdatesBlocker blocker(this);
const QuickTileMode oldMode = requestedQuickTileMode(); const QuickTileMode oldMode = requestedQuickTileMode();
QPointF whichScreen = keyboard ? moveResizeGeometry().center() : Cursors::self()->mouse()->pos(); QPointF whichScreen = keyboard ? moveResizeGeometry().center() : Cursors::self()->mouse()->pos();