From d2648c6c77858bb38dafc0b4748f6832240b0664 Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Wed, 7 Nov 2018 16:32:56 +0000 Subject: [PATCH] [wayland] Fix arguments to clientMaximizedStateChanged(bool, bool) Summary: It should be indicating the current state of horizontal and vertical maximisation not whether it changed or not. Reviewers: #kwin, zzag Reviewed By: #kwin, zzag Subscribers: kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D16740 --- shell_client.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/shell_client.cpp b/shell_client.cpp index f0ddf76373..f758e2e7f3 100644 --- a/shell_client.cpp +++ b/shell_client.cpp @@ -1504,12 +1504,10 @@ void ShellClient::updateMaximizeMode(MaximizeMode maximizeMode) return; } - bool horizontalChanged = (maximizeMode & MaximizeHorizontal) != (m_maximizeMode & MaximizeHorizontal); - bool verticalChanged = (maximizeMode & MaximizeVertical) != (m_maximizeMode & MaximizeVertical); m_maximizeMode = maximizeMode; emit clientMaximizedStateChanged(this, m_maximizeMode); - emit clientMaximizedStateChanged(this, horizontalChanged, verticalChanged); + emit clientMaximizedStateChanged(this, m_maximizeMode & MaximizeHorizontal, m_maximizeMode & MaximizeVertical); } bool ShellClient::hasStrut() const