D17265: Fix changing the number of rows via the dbus protocol

Martin Flöser <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
graesslin added inline comments.

INLINE COMMENTS

> mart wrote in virtualdesktops.cpp:612-614
> uhm, where? (tough didn't work as without this saving desktop names gets corrupted and autotests fail without it

From VirtualDesktopManager::save:

  if (s_loadingDesktopSettings) {
      return;
  }

s_loadingDesktopSettings is set to true at start of VirtualDesktopManager::load and back to false at the end of the method. Given that this was identical to the newly introduced m_isLoading it got removed in 92b5ddf94c6b472d928c20e4d4bc5407b086e44b <https://phabricator.kde.org/R108:92b5ddf94c6b472d928c20e4d4bc5407b086e44b>

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D17265

To: mart, #plasma, #kwin, hein, davidedmundson, graesslin
Cc: graesslin, davidedmundson, ngraham, kwin, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.