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