KGuiAddons Review

John Layt jlayt at kde.org
Tue Mar 4 17:46:53 UTC 2014


Hi,

Here's my first pass through KGuiAddons, focussing on the public api.

KColorCollection
- Should probably become a QSharedDataPointer

KWorkdWrap
- "// KDE5 TODO: return a value, not a pointer, and use QSharedDataPointer."

KModifierKeyInfo
 - Generally looks OK
 - Has lots of "bool isKeyPressed(Qt::Key)" style methods and 
keyPressed(Qt::Key, bool) style signals when perhaps a KeyState enum would be 
better?
 - Uses X11 / XKB / XCB, will need Wayland backend eventually?
 - Perhaps really belongs in Qt, or somewhere else?

KImageCache
KSharedPixmapCacheMixin
KLocalImageCacheImplementation
 - Looks OK
 - KImageCache not exported, but KLocalImageCacheImplementation is, some CMake 
magic involved?

KColorMimeData
KFontUtils
KIconUtils
 - Looks OK

KColorUtils
 - Looks OK
 - Qt should probably get some of these methods?

KDateValidator
 - Looks OK
 - Qt should probably have one, QTime as well?

And the usual documentation improvements of course.  Otherwise looks in good 
shape.

Cheers!

John.


More information about the Kde-frameworks-devel mailing list