Review Request: Make KDirWatch inotify threaded.

Sebastian Sauer mail at dipe.org
Mon Oct 26 22:15:16 GMT 2009


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

(Updated 2009-10-26 22:15:16.054805)


Review request for kdelibs.


Changes
-------

Updated
-  client->watchingStopped = d->_isStopped;
+  client->watchingStopped = instance->d->_isStopped;
to use the instance like before.


Summary
-------

Problem;
Under heavy hd load (e.g. compiling KDE) KDirWatch may freeze for fractions of seconds or even longer.

Solution;
Do the same QFileSystemWatcher (qt/src/corelib/io/qfilesystemwatcher_inotify.cpp) does and make the QSocketNotifier+inotifyEventReceived logic threaded.

Warning;
This *needs* review.


Diffs (updated)
-----

  /trunk/KDE/kdelibs/kio/kio/kdirwatch.h 1038809 
  /trunk/KDE/kdelibs/kio/kio/kdirwatch.cpp 1038809 
  /trunk/KDE/kdelibs/kio/kio/kdirwatch_p.h 1038809 

Diff: http://reviewboard.kde.org/r/1963/diff


Testing
-------

kdelibs/kio/tests/kdirwatch, kdelibs/kio/tests/kdirwatch_gui


Thanks,

Sebastian





More information about the kde-core-devel mailing list