broulik accepted this revision.
broulik added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> kcmsmserver.cpp:86
> + auto sm = new SessionManagement(this);
> + auto doShutdown=[sm]() {
> + sm->requestReboot();
Coding style.
> kcmsmserver.cpp:88
> + sm->requestReboot();
> + delete sm;
> + };
Is this safe, or should it do a `deleteLater()`?
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D21763
To: davidedmundson, #plasma, broulik
Cc: broulik, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen,
Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts,
sebas, apol, mart
