[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [PATCH] BUG 167388 inconsistency in ~/.kde4/share/config/trashrc
From: David Faure <faure () kde ! org>
Date: 2009-09-14 14:16:43
Message-ID: 200909141616.44037.faure () kde ! org
[Download RAW message or body]
[Too much cross-posting, removing kde-devel from the CC list]
On Wednesday 02 September 2009, 潘卫平(Peter Pan) wrote:
===================================================================
> --- trashimpl.cpp (revision 1018673)
> +++ trashimpl.cpp (working copy)
> @@ -771,6 +771,7 @@
>
> void TrashImpl::fileAdded()
> {
> + m_config.reparseConfiguration();
> KConfigGroup group = m_config.group( "Status" );
> if ( group.readEntry( "Empty", true) == true ) {
> group.writeEntry( "Empty", false );
This one makes sense indeed. Before reading we have to re-read from
disk. Well, ideally only if it changed, so another fix would be to use
KDirWatch on ktrashrc.
> @@ -784,6 +785,7 @@
> void TrashImpl::fileRemoved()
> {
> if ( isEmpty() ) {
> + m_config.reparseConfiguration();
> KConfigGroup group = m_config.group( "Status" );
> group.writeEntry( "Empty", true );
> m_config.sync();
This one is not necessary. There's no point in reparsing if we're only writing
and not reading, is there? We'll overwrite any changes anyway.
> But if more than one kio_trash are created, inconsistency may occur,
This is indeed true, and Tobias' answer is valid for that part.
I don't think KInterProcessLock removes the need for the reparseConfiguration
before reading, though, if that's what Tobias implied -- but I don't think it
was what he implied ;).
> reparseConfiguration is expensive,
Yes, because of kdeglobals. We should turn m_config into a SimpleConfig, in
fact, so that it only reads the local trashrc and not any kdeglobals or global
trashrc. I'll do that, you commit the reparseConfiguration and close the bug?
Thanks!
Tobias: hmm I just noticed that you used "ktrashrc" for the user settings,
while the empty/not-empty flag is in "trashrc", for a reason I cannot remember.
On one hand we should probably merge that so that it's less confusing, on the
other hand it would make reparseConfiguration more expensive again ;-)
--
David Faure, faure@kde.org, sponsored by Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic