D25677: [KColorScheme/KStatefulBrush] Switch hardcoded numbers for enum items
David Faure
noreply at phabricator.kde.org
Tue Dec 3 08:29:38 GMT 2019
dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> kcolorscheme.cpp:289
> struct {
> - QBrush fg[8], bg[8], deco[2];
> + QBrush fg[KColorScheme::NForegroundRoles], bg[KColorScheme::NBackgroundRoles], deco[KColorScheme::NDecorationRoles];
> } _brushes;
(I would split this into 3 lines for more readability)
> kcolorscheme.cpp:742
> +
> +
extra empty lines added at the end?
REPOSITORY
R265 KConfigWidgets
BRANCH
arcpatch-D25677 (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D25677
To: ndavis, #frameworks, dfaure
Cc: ngraham, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191203/9a43aa3d/attachment.html>
More information about the Kde-frameworks-devel
mailing list