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