ervin requested changes to this revision. ervin added a comment. This revision now requires changes to proceed.
In D27833#622353 <https://phabricator.kde.org/D27833#622353>, @apol wrote: > What's the use-case? Having an idea of the patch you want to build on top of this would indeed help. INLINE COMMENTS > kcoreconfigskeleton.cpp:210 > + Q_D(KConfigSkeletonItem); > + d->mLoadedValueImpl= impl; > +} Missing space before = > kcoreconfigskeleton.cpp:226 > + Q_D(const KPropertySkeletonItem); > + return QVariant::fromValue(d->mLoadedValue); > + }); This is already a QVariant REPOSITORY R237 KConfig REVISION DETAIL https://phabricator.kde.org/D27833 To: meven, ervin, bport, crossi, #frameworks Cc: apol, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns