D28617: Remove QSessionManager usage

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Apr 6 13:35:50 BST 2020


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20200406/3e2fed06/attachment.html>


More information about the kwin mailing list