romangg added a comment.
Yes, please put the m_timer removal in a separate patch. And if you removed the m_timer the QTimer include in the header file can be removed as well I assume. Thanks. REPOSITORY R104 KScreen REVISION DETAIL https://phabricator.kde.org/D28817 To: bport, #plasma, romangg, davidedmundson Cc: romangg, davidedmundson, anthonyfieroni, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart