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

Review request for Nepomuk, Vishesh Handa and Sebastian Trueg.


Description
-------

Some fairly trivial misc improvements to the filewatch service. Probably don't 
make a big difference, but probably a good idea. 

- Use QReadWriteLock instead of QMutex in FileIndexerConfig, thus allowing 
multiple threads to call shouldFolderBeIndexed at once (not that we really do 
that right now).

- Add the IN_EXCL_UNLINK inotify flag. On recent (2.6.36) kernels, this means 
we don't generate events for files once
they have been unlinked from the directory we are watching. Prevents waking up 
for some already-deleted temporary files.


Diffs
-----

  services/fileindexer/fileindexerconfig.h 7debaf3 
  services/fileindexer/fileindexerconfig.cpp 5226a79 
  services/filewatch/kinotify.h 6e3f1c0 
  services/filewatch/kinotify.cpp 9868b90 

Diff: http://git.reviewboard.kde.org/r/107082/diff/


Testing
-------

Compiled, ran for a bit, didn't seem to break anything. 


Thanks,

Simeon Bird

_______________________________________________
Nepomuk mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/nepomuk

Reply via email to