[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [RFC] Use QFileSystemWatcher as another method for KDirWatch?
From:       David Faure <faure () kde ! org>
Date:       2007-12-05 23:12:21
Message-ID: 200712060012.22841.faure () kde ! org
[Download RAW message or body]

On Wednesday 05 December 2007, Andreas Pakulat wrote:
> On 05.12.07 23:11:53, Christian Ehrlicher wrote:
> > Andreas Pakulat schrieb:
> > > On 05.12.07 20:24:27, Christian Ehrlicher wrote:
> > > > Christian Ehrlicher schrieb:
> > > > > Hi,
> > > > > 
> > > > > Using kdirwatch on windows is very cpu consuming (kded sometimes use 40% \
> > > > > cpu on my duron 1300). Therefore I want to implement QFileSystemWatcher as \
> > > > > an optional method. Is there a reason why this should not work? Should it \
> > > > > be an optional or a hard dependency? 
> > > > Ok, here the patch.
> > > > 
> > > > I enabled it only for windows to not get in trouble with 4.0.0. There's
> > > > a bug (task-tracker id 170021) which prevents watching directories on a
> > > > fat32 partition. It's fixed in 4.4 but it would be nice when this patch
> > > > could be applied to qt-copy.
> > > 
> > > Does this support all features of KDirWatch? Last time I checked
> > > QFSWatcher had some limitations.
> > > 
> > There's one major difference between KDirWatch and QFileSystemWatcher.
> > KDirWatch can watch files/dirs which do not exist yet.
> > Any other difference you're aware of?
> 
> No, and now that you mention it, I think thats exactly what I was
> thinking about. I'm not sure though how much that feature is used...

I think it's quite necessary in kded, which has to notice directories created
in the usual places (e.g. $KDEHOME/share/services). But I guess this can always
be emulated by watching the parent dir instead - at the expense of making the
code more complex.

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic