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

Fix "context objects" in VirtualDesktopManager::setVirtualDesktopManagement

Browse source 
Summary: Just to make "lifetimes" more cleaner.

Reviewers: #kwin, davidedmundson

Reviewed By: #kwin, davidedmundson

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D18374
This commit is contained in:
Vlad Zagorodniy 2019-01-19 00:31:03 +02:00
parent 47e9e52df3
commit 7477a411ae
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
virtualdesktops.cpp
View file

@ -76,10 +76,10 @@ void VirtualDesktopManager::setVirtualDesktopManagement(KWayland::Server::Plasma
); );
}; };
connect(this, &VirtualDesktopManager::desktopCreated, this, createPlasmaVirtualDesktop); connect(this, &VirtualDesktopManager::desktopCreated, m_virtualDesktopManagement, createPlasmaVirtualDesktop);
//handle removed: from VirtualDesktopManager to the wayland interface //handle removed: from VirtualDesktopManager to the wayland interface
connect(this, &VirtualDesktopManager::desktopRemoved, this, connect(this, &VirtualDesktopManager::desktopRemoved, m_virtualDesktopManagement,
[this](VirtualDesktop *desktop) { [this](VirtualDesktop *desktop) {
m_virtualDesktopManagement->removeDesktop(desktop->id()); m_virtualDesktopManagement->removeDesktop(desktop->id());
} }
@ -106,7 +106,7 @@ void VirtualDesktopManager::setVirtualDesktopManagement(KWayland::Server::Plasma
//Now we are sure all ids are there //Now we are sure all ids are there
save(); save();
connect(this, &VirtualDesktopManager::currentChanged, this, connect(this, &VirtualDesktopManager::currentChanged, m_virtualDesktopManagement,
[this]() { [this]() {
for (auto *deskInt : m_virtualDesktopManagement->desktops()) { for (auto *deskInt : m_virtualDesktopManagement->desktops()) {
if (deskInt->id() == currentDesktop()->id()) { if (deskInt->id() == currentDesktop()->id()) {