oliverhenshaw added inline comments. INLINE COMMENTS
> powerdevilcore.cpp:151 > + // to an idle session by removing all idle timeouts > + KIdleTime::instance()->removeAllIdleTimeouts(); > + m_registeredActionTimeouts.clear(); Like you said, other kidletime users might not care whether the session is active or not. This will remove all timeouts set by other kded services, right? Just off the top of my head, Is ActionPool::unloadAllActiveActions() better here, or is that not appropriate? It's too late in the day here for me to look into that today. REPOSITORY rPOWERDEVIL Powerdevil REVISION DETAIL https://phabricator.kde.org/D2033 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma Cc: oliverhenshaw, graesslin, plasma-devel, jensreuterberg, sebas
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel