D25070: Make KCModuleQml conform to the defaulted() signal

Kevin Ottens noreply at phabricator.kde.org
Wed Oct 30 13:52:39 GMT 2019


ervin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in kcmoduleqml.cpp:80-82
> needsSave emits the current state and then connects for changes
> representsDefaults  only connects for changes
> 
> I would expect them to match as they're doing equivalent things.
> 
> It looks to me that it's this line that's somewhat pointless - we're in a constructor so only super classes could possibly have connected and we can see it doesn't.
> Can you confirm.

Yes, I confirm this is pointless and why I didn't replicate that for defaulted.

REPOSITORY
  R295 KCMUtils

REVISION DETAIL
  https://phabricator.kde.org/D25070

To: ervin, #plasma, #frameworks, mart, davidedmundson, dfaure
Cc: 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/20191030/f969fca8/attachment.html>


More information about the Kde-frameworks-devel mailing list