D24494: Add convenience for defaults/dirty states to KCoreConfigSkeleton
David Edmundson
noreply at phabricator.kde.org
Tue Oct 8 17:20:29 BST 2019
davidedmundson added a comment.
Looks good to me
INLINE COMMENTS
> kcoreconfigskeleton.cpp:140
> +{
> + return d->mIsDefaultImpl();
> +}
Do we need to make this
if (d->mIsDefaultImpl){
return d->mIsDefaultImpl();
}
return false;
and initialize mIsDefaultImpl to nullptr
so that it doesn't crash if someone subclasses KConfigSkeletonItem directly and doesn't implement this?
REPOSITORY
R237 KConfig
REVISION DETAIL
https://phabricator.kde.org/D24494
To: ervin, #plasma, #frameworks, dfaure, mart
Cc: davidedmundson, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191008/eedb17d3/attachment.html>
More information about the Kde-frameworks-devel
mailing list