D25872: Refactor GTK KCM code
Mikhail Zolotukhin
noreply at phabricator.kde.org
Wed Dec 11 08:46:20 GMT 2019
gikari created this revision.
gikari added a reviewer: apol.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
gikari requested review of this revision.
REVISION SUMMARY
This diff introduces purely cosmetic changes.
1. KCM now compiled with no signals/slot keywords to avoid clash with gtk
2. Remove linking of XCursors library, because it is no longer needed
3. Replace some for-cycles with range-based ones
4. Wrap some string literals with QStringLiteral
5. Rearrange include headers
6. Some other changes to better match code style
TEST PLAN
Check, if there is no regressions
REPOSITORY
R99 KDE Gtk Configuration Tool
BRANCH
refactor-kcm-code (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D25872
AFFECTED FILES
CMakeLists.txt
src/abstractappearance.cpp
src/abstractappearance.h
src/appearancegtk2.cpp
src/appearancegtk2.h
src/appearancegtk3.cpp
src/appearancegtk3.h
src/appearencegtk.cpp
src/appearencegtk.h
src/dialog_installer.cpp
src/dialog_installer.h
src/dialog_uninstaller.cpp
src/dialog_uninstaller.h
src/gtkconfigkcmodule.cpp
src/gtkconfigkcmodule.h
src/installer.cpp
src/thread.cpp
src/thread.h
tests/CMakeLists.txt
tests/configsavetest.h
To: gikari, apol
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191211/53ceeac5/attachment.html>
More information about the Plasma-devel
mailing list