broulik created this revision. broulik added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY We initially populate the `QQmlPropertyMap` with all keys in settings after which the properties are created for the first time. There's no need to emit value changed then. TEST PLAN Plasma starts normal, applet configs are loaded and persisted. I didn't actually find a case where `KCoreConfigSkeleton::configChanged` would be called as in `writeConfigValue` we block signals and we don't monitor outside config changes. Probably still needs more testing. Saves 60ms plasmashell startup REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D9312 AFFECTED FILES src/kdeclarative/configpropertymap.cpp src/kdeclarative/configpropertymap.h To: broulik, #plasma Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart