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

Reply via email to