volkov added a comment.
Looks good to me. powerdevilpowermanagement.{h,cpp} should be extracted into a library, but it can be done in a separate commit. Since they deal with org.freedesktop.PowerManagement interface which is obsolete and implemented only by PowerDevil, I suppose such a library should be a part of PowerDevil. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D4939 To: denisshienkov, volkov, afiestas Cc: graesslin, davidedmundson, broulik, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol