D26133: Enable Auto Save

[Kevin Ottens]

ervin requested changes to this revision.
ervin added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> apol wrote in test_signal.h.ref:137
> How about having `isSaveNeededChanged(bool)`? It could be in KCoreConfigSkeleton.

Damn, and I was wondering which better name we could use... it was already there all along since KCoreConfigSkeleton has isSaveNeeded indeed... Thanks Aleix! :-)

Note this means the logic will need to be adjusted though since it'll also need being emitted only when the value changes. AFAICT currently it does emit always on property change while only the first change would be necessary... also it won't emit again on load() calls when it's most likely we're rolling back.

REPOSITORY
  R237 KConfig

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

To: tcanabrava, ervin
Cc: ervin, GB_2, apol, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191226/0058119c/attachment.html>