broulik requested changes to this revision. broulik added a comment. This revision now requires changes to proceed.
There seem to be a bunch of additional `m_helper.isNull()` that could be ported to be consistent? INLINE COMMENTS > powerdevildpmsaction.cpp:196 > // Inhibition triggered: disable DPMS > - if (!m_helper.isNull()) { > + if (isSupported())) { > m_helper->inhibited(); There's a superfluous parenthesis. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D27278 To: alnikiforov, apol, davidedmundson, broulik Cc: broulik, plasma-devel, mlcarr, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart