crossi added inline comments. INLINE COMMENTS
> accountmodel.cpp:545 > +void AccountModel::setDpr(qreal dpr) { > + this->dpr = dpr; > +} See my previous comment, 'this->' prefix won't be necessary. > accountmodel.h:95 > AutomaticLoginSettings m_autoLoginSettings; > + qreal dpr = 1; > }; Style related, attribute member are prefixed with 'm_', see above declarations. REPOSITORY R128 User Manager REVISION DETAIL https://phabricator.kde.org/D25395 To: meven, #plasma Cc: crossi, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart