SDDM-KCM In Review

[Vishesh Handa]

Hey David.

None of these are blockers. Just stuff I noticed.

* There is this strange ACKNOWLEDGEMENTS file

* themeconfig.cpp
** The delegate seems to have a fixed size of 128x128. This might not be
ideal for hidpi screens
** ThemeConfig::dump - Extra indendation

* thememodel.cpp - KStandardDirs and KGlobal headers are not required
* usersmodel.cpp - The coding style is quite different
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20141015/60525162/attachment.htm>