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