D16989: [kded] Introduce Config wrapper class
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R104:9e34fd814f90: [kded] Introduce Config wrapper class (authored by romangg). REPOSITORY R104 KScreen CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16989?vs=59653&id=59753 REVISION DETAIL https://phabricator.kde.org/D16989 AFFECTED FILES kded/CMakeLists.txt kded/config.cpp kded/config.h kded/daemon.cpp kded/daemon.h kded/serializer.cpp kded/serializer.h tests/kded/CMakeLists.txt tests/kded/configtest.cpp tests/kded/serializertest.cpp To: romangg, #plasma Cc: plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16989: [kded] Introduce Config wrapper class
romangg updated this revision to Diff 59653. romangg added a comment. Rebase on master. REPOSITORY R104 KScreen CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16989?vs=57655&id=59653 BRANCH 1configClass REVISION DETAIL https://phabricator.kde.org/D16989 AFFECTED FILES kded/CMakeLists.txt kded/config.cpp kded/config.h kded/daemon.cpp kded/daemon.h kded/serializer.cpp kded/serializer.h tests/kded/CMakeLists.txt tests/kded/configtest.cpp tests/kded/serializertest.cpp To: romangg, #plasma Cc: plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16989: [kded] Introduce Config wrapper class
romangg updated this revision to Diff 57655. romangg added a comment. Rebase on master. REPOSITORY R104 KScreen CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D16989?vs=45756&id=57655 BRANCH 1configClass REVISION DETAIL https://phabricator.kde.org/D16989 AFFECTED FILES kded/CMakeLists.txt kded/config.cpp kded/config.h kded/daemon.cpp kded/daemon.h kded/serializer.cpp kded/serializer.h tests/kded/CMakeLists.txt tests/kded/configtest.cpp tests/kded/serializertest.cpp To: romangg, #plasma Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D16989: [kded] Introduce Config wrapper class
romangg created this revision. romangg added a reviewer: Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. romangg requested review of this revision. REVISION SUMMARY In order to consolidate the code and as a preparatory step for larger changes introduce a wrapper class for KScreen::Config types. This class combines information about the type and the previously static Serializer methods, such that the Serializer class can be removed. TEST PLAN Autotest updated. REPOSITORY R104 KScreen BRANCH configClass REVISION DETAIL https://phabricator.kde.org/D16989 AFFECTED FILES kded/CMakeLists.txt kded/config.cpp kded/config.h kded/daemon.cpp kded/daemon.h kded/serializer.cpp kded/serializer.h tests/kded/CMakeLists.txt tests/kded/configtest.cpp tests/kded/serializertest.cpp To: romangg, #plasma Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
