patrickelectric added inline comments.

INLINE COMMENTS

> kcolorcombo.cpp:67
>      // inner color
> +    QVariant tv = index.data(Qt::DisplayRole);
>      QVariant cv = index.data(ColorRole);

Missing const.

> kcolorcombo.cpp:69
>      QVariant cv = index.data(ColorRole);
> +    QColor innerColor = ((cv.type() == QVariant::Color) ? cv.value<QColor>() 
> : QPalette::Base);
> +    

missing const.

REPOSITORY
  R236 KWidgetsAddons

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

To: araujoluis, tcanabrava, patrickelectric, hindenburg, ngraham
Cc: cblack, broulik, cfeck, kde-frameworks-devel, LeGast00n, michaelh, ngraham, 
bruns

Reply via email to