apol added a comment.
Review without knowing very well the codebase.
INLINE COMMENTS
> accountmodel.cpp:181
> {
> -if(!index.isValid()) {
> -return false;
> +qDebug() << "AccountModel::setDataWithResult";
> +if (!index.isValid()) {
Remove?
> accountmodel.h:64
> +
meven created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
meven requested review of this revision.
REVISION SUMMARY
BUG: 417882
Not tested yet !
REPOSITORY
R128 User Manager
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D27517
