fvogt requested changes to this revision. fvogt added a comment. This revision now requires changes to proceed.
There should be a comment somewhere to explain what's happening and why. INLINE COMMENTS > mprisplugin.h:153 > > + QTimer *m_propertyChangeSignalTimer; > + // interface - <property name, value> Why a pointer? REPOSITORY R856 Plasma Browser Integration REVISION DETAIL https://phabricator.kde.org/D21998 To: broulik, #plasma, fvogt Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
