[Nepomuk] Review Request 109078: Add a base class for the file watcher back-ends
Gabriel Poesia
gabriel.poesia at gmail.com
Mon Feb 25 17:08:54 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109078/
-----------------------------------------------------------
(Updated Feb. 25, 2013, 5:08 p.m.)
Review request for Nepomuk and Vishesh Handa.
Changes
-------
Added Trueg's copyright statement (from kinotify.h) to filesystemmonitor.h, as most code there is his (taken from kinotify.h).
So, should I follow with this, implementing the two subclasses that use inotify and kDirNotify?
Description
-------
This is about this task: http://community.kde.org/Projects/Nepomuk/4.11#File_Watcher
This abstract FileSystemMonitor class is basically the generic interface extracted from KInotify.
It also has a method to ignore paths based on a RegExpCache filter list, just like IgnoringKInotify.
Diffs (updated)
-----
services/filewatch/filesystemmonitor.h PRE-CREATION
services/filewatch/filesystemmonitor.cpp PRE-CREATION
services/filewatch/test/CMakeLists.txt 8b6ebe6
services/filewatch/test/filesystemmonitortest.h PRE-CREATION
services/filewatch/test/filesystemmonitortest.cpp PRE-CREATION
Diff: http://git.reviewboard.kde.org/r/109078/diff/
Testing
-------
The empty test that includes it compiles.
Thanks,
Gabriel Poesia
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20130225/4edf2d13/attachment.html>
More information about the Nepomuk
mailing list