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

Reply via email to