D28617: Remove QSessionManager usage

Aleix Pol Gonzalez <[email protected]> Mon, 6 Apr 2020 12:35:50 +0000

Newsgroups	gmane.comp.kde.devel.kwin
Message-ID	<[email protected]>

apol added a comment.


  +1 LGTM

INLINE COMMENTS

> sm.cpp:357
>  
> +// TODO should we rethink this now that we have dedicated start end end save methods?
>  void SessionManager::setState(SessionState state)

Care to elaborate? what do you have in mind here?

Feels like it could end up being one of these TODO that stay forever

REPOSITORY
  R108 KWin

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

To: broulik, #kwin
Cc: apol, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, ahiemstra, mart