Review Request: When a source file for kconf_update does not exist, do not create it

Oswald Buddenhagen ossi at kde.org
Mon Jun 7 08:35:52 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4243/#review6003
-----------------------------------------------------------



/trunk/KDE/kdelibs/kconf_update/kconf_update.cpp
<http://reviewboard.kde.org/r/4243/#comment5624>

    the additional conditions will lead to memory leaks.



/trunk/KDE/kdelibs/kconf_update/kconf_update.cpp
<http://reviewboard.kde.org/r/4243/#comment5623>

    do you have an explanation why the existing test is not sufficient?



/trunk/KDE/kdelibs/kconf_update/kconf_update.cpp
<http://reviewboard.kde.org/r/4243/#comment5621>

    this is not what the test checks ...


- Oswald


On 2010-06-06 13:57:27, Kevin Krammer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4243/
> -----------------------------------------------------------
> 
> (Updated 2010-06-06 13:57:27)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> Currently, when kconf_update is being run by kdeinit4 on a new user account, it creates all config files that it has updates for, regardless whether these files exist.
> Which mean .kde/share/config gets populated with a lot of files with only kconf_update's entry in them.
> 
> Unfortunately that means that programs checking for a specific file's existance, e.g. PIM apps which are checking whether to run a migrator tool, will be fooled into assuming that there are actually valid settings.
> 
> The patch uses the already set m_skipFile flag to decide when not to sync the KConfig object, thus not creating the file if it didn't exist before
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/kconf_update/kconf_update.cpp 1135120 
> 
> Diff: http://reviewboard.kde.org/r/4243/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kevin
> 
>





More information about the kde-core-devel mailing list