ngraham added inline comments.

INLINE COMMENTS

> accountinfo.cpp:364
>      if (email.isEmpty()) {
> -        return false;
> +        return true;
>      }

This seems like the only part of the patch that's actually about fixing the 
issue. The rest seems to be unrelated code cleanup that should be probably be 
done separately.

REPOSITORY
  R128 User Manager

REVISION DETAIL
  https://phabricator.kde.org/D26824

To: meven, #plasma, ngraham
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to