From ca6459145c7153fb4f14c9b3301519a2a6945925 Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Wed, 1 Nov 2017 15:45:12 +0000 Subject: [PATCH] Remove code duplication Summary: setGlobalPos sets these, we don't need to call it twice Test Plan: Looked at setGlobalPos Reviewers: #plasma, graesslin Reviewed By: #plasma, graesslin Subscribers: plasma-devel, kwin, #kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D8581 --- plugins/platforms/drm/drm_output.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/plugins/platforms/drm/drm_output.cpp b/plugins/platforms/drm/drm_output.cpp index fc7491aa51..bd16fbfd14 100644 --- a/plugins/platforms/drm/drm_output.cpp +++ b/plugins/platforms/drm/drm_output.cpp @@ -706,8 +706,6 @@ bool DrmOutput::commitChanges() } if (m_changeset->positionChanged()) { qCDebug(KWIN_DRM) << "Server setting position: " << m_changeset->position(); - m_waylandOutput->setGlobalPosition(m_changeset->position()); - m_waylandOutputDevice->setGlobalPosition(m_changeset->position()); setGlobalPos(m_changeset->position()); // may just work already! }