D27833: Add an accessor to get the last loaded value for KConfigSkeletonItem
Kevin Ottens
noreply at phabricator.kde.org
Thu Mar 5 15:45:20 GMT 2020
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200305/ff2a2328/attachment.html>
More information about the Kde-frameworks-devel
mailing list