cfeck added inline comments.
INLINE COMMENTS
> kmessagewidget.cpp:263
> + QStringList rgb = settings->value(key).toStringList();
> + return QColor(rgb.at(0).toInt(), rgb.at(1).toInt(), rgb.at(2).toInt());
> +}
missing `if (rgb.size() >= 3)`
> kmessagewidget.cpp:281
> + const QString themeGroup = QStringLiteral("Colors:Window");
> + QSettings *settings = !kdeGlobals.isEmpty()? new QSettings(kdeGlobals,
> QSettings::IniFormat) : nullptr;
> + if (settings && settings->childGroups().contains(themeGroup)) {
missing space before `?`
REPOSITORY
R236 KWidgetsAddons
REVISION DETAIL
https://phabricator.kde.org/D13777
To: rjvbb, ngraham, #frameworks
Cc: cfeck, kde-frameworks-devel, michaelh, ngraham, bruns
