Switch from naked KConfig to KConfigXT
ClosedPublic
Actions

Authored by ervin on Oct 3 2019, 4:10 PM.

Details

Reviewers

mart
apol

Group Reviewers

Plasma

Commits

R119:6648a1a476a8: Switch from naked KConfig to KConfigXT

Summary

It provides some improvements at least: first in code reuse, but also it
cleans up the config file behind itself when we get back to the default
value.

Diff Detail

Repository

R119 Plasma Desktop

Lint

Lint Skipped

Unit

Unit Tests Skipped

Build Status

Buildable 17298
Build 17316: arc lint + arc unit

ervin created this revision.Oct 3 2019, 4:10 PM

Restricted Application added a project: Plasma. · View Herald TranscriptOct 3 2019, 4:10 PM

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

ervin requested review of this revision.Oct 3 2019, 4:10 PM

Harbormaster completed remote builds in B17298: Diff 67269.Oct 3 2019, 4:10 PM

Looks good!

This revision is now accepted and ready to land.Oct 3 2019, 6:58 PM

mart accepted this revision.Oct 4 2019, 8:17 AM

Closed by commit R119:6648a1a476a8: Switch from naked KConfig to KConfigXT (authored by ervin). · Explain WhyOct 4 2019, 8:52 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

		Path
M		kcms/desktoptheme/CMakeLists.txt (2 lines)
A	M	kcms/desktoptheme/desktopthemesettings.kcfg (13 lines)
A	M	kcms/desktoptheme/desktopthemesettings.kcfgc (5 lines)
M		kcms/desktoptheme/kcm.h (3 lines)
M		kcms/desktoptheme/kcm.cpp (25 lines)