D25070: Make KCModuleQml conform to the defaulted() signal
David Edmundson
noreply at phabricator.kde.org
Wed Oct 30 12:57:06 GMT 2019
davidedmundson added inline comments.
INLINE COMMENTS
> kcmoduleqml.cpp:80-82
> if (d->configModule->needsSave()) {
> emit changed(true);
> }
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.
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/7c681216/attachment.html>
More information about the Kde-frameworks-devel
mailing list