[PATCH] kdirwatch.cpp

[Main user of this system]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Mon, 23 Sep 2002, Josef Weidendorfer wrote:

Sorry for the somwhat late reply, my machine was totally corrupted, had to
reinstall everything :( Luckily i could keep my /home and /cvs partitions.

> Hi Otto,
>
> thanks for implementing this.
> Unfortunately their are some problems with your patch:
> - when deleting the recursive/watchFiles watch again, this only deletes the
> given directory. It should delete all other added files watches, too.

Yikes, i never bothered to check for this to be honest, I just added the
recursive and watchFiles in the adding of things to be watched because i
wanted that in Kompare, didnt bother to see what else was needed. But i'll
look into this.

> - the "recursive" and "watchFiles" option has to be dynamic, otherwise
> their is no use of implementing this in KDirWatch; your approach is static and
> only looks for files existing when adding the watch.
> E.g. if a file is created in directory of a watch entry with option
> "watchFiles", it should automatically be watched, too. And of course deleted
> with the original entry. Same holds for the "recursive" flag.

I understand. I'll see if i can implement this the right way. I assume
(without looking at the code, so i could be wrong) that it has to the new
directory entry (file or dir) to itself before it emits one of the 3
signals ? (BTW the docu is wrong imo, it talks about 3 additional signals
in the addDir docu that i cant find anywhere in the class).

> Before release of KDE 3.0, I added these flags, and started implementing
> something, but I didn't finished...
>
> BTW: Main use of the "recursive" flag would be in KDED, where recursive dirs
> are watched. Every time KDED detects a change, it deletes ALL watches and
> creates them again. This is much overhead for nothing. In KDirWatch, this
> could be done VERY efficient...

Ouch, that is indeed very inefficient. I can also use such a beast for
Kompare when one compares 2 directories :) I dont want to add every
file/directory manually (even files/dirs that currently dont have changes
should be watched if something changes in them).

> I'm waiting for your next patch :-)

I will try to mail something to you this weekend if time permits :)

> Josef

Otto
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: Made with pgp4pine 1.75-6

iD8DBQE9kO5GDc93sltYd+ERAqH7AJ9ez8S5k4vysi/cYzhPF/lO7gfhLwCeItH0
jscEmYh3BhZEJhq6bpt6TW4=
=JdtV
-----END PGP SIGNATURE-----