D28128: Add force save behavior to KEntryMap

Benjamin Port noreply at phabricator.kde.org
Mon Mar 23 20:06:03 GMT 2020


bport planned changes to this revision.
bport added inline comments.

INLINE COMMENTS

> dfaure wrote in kconfigtest.cpp:1965
> This would pass no matter in which file the write happened, no, due to caching?
> Doesn't this need a generalLocal.reparseConfiguration() to be meaningful?

yes indeed

> dfaure wrote in kconfigtest.cpp:1970
> Hmm, so this is what this is all about?
> 
> This contradicts the documentation for revertToDefault().
> 
> - Reverts the entry with key @p key in the current group in the
> - application specific config file to either the system wide (default)
> - value or the value specified in the global KDE config file.
> 
> The value in the global config file is 10, that's what this is supposed to revert to.

This is a global local file, not system wide and so not considered as default
cf. https://lxr.kde.org/source/frameworks/kconfig/src/core/kconfig.cpp#0702
if the entry is set system wide 
/etc/kde5rc
/etc/xdg/system.kdeglobals
/etc/xdg/kdeglobals
/etc/xdg/yourcustomfile
~/.config/system.kdeglobals

It will be reverted to the default value specified in the file

REPOSITORY
  R237 KConfig

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

To: bport, ervin, dfaure, meven, crossi, hchain
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200323/2ba723bc/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list