D25677: [KColorScheme/KStatefulBrush] Switch hardcoded numbers for enum items
Noah Davis
noreply at phabricator.kde.org
Mon Dec 2 07:04:35 GMT 2019
ndavis created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ndavis requested review of this revision.
REVISION SUMMARY
This also replaces some for-loops with C++11 range based for-loops and switches for simpler if/else control blocks.
TEST PLAN
Open colorscheme editor in the Colors KCM to see if any colors or effects look broken.
REPOSITORY
R265 KConfigWidgets
BRANCH
remove-hardcoded-values (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D25677
AFFECTED FILES
src/kcolorscheme.cpp
To: ndavis
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191202/b18eee67/attachment.html>
More information about the Kde-frameworks-devel
mailing list