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