Review Request 129205: [kcoredirlister] Ability to watch files changes
Anthony Fieroni
bvbfan at abv.bg
Mon Oct 31 15:53:47 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129205/
-----------------------------------------------------------
(Updated Oct. 31, 2016, 5:53 p.m.)
Status
------
This change has been discarded.
Review request for KDE Frameworks and David Faure.
Repository: kio
Description
-------
David, i will discard review if you don't like it, cause watching files changes can be *really* expensive. I try to:
1. to not break abi compability
2. to extend filenamesearch with this option
3. to fix https://git.reviewboard.kde.org/r/129141/
Diffs
-----
src/core/kcoredirlister.h e6ba2ac
src/core/kcoredirlister.cpp 508516e
src/core/kcoredirlister_p.h 9a3cc7b
tests/kdirlistertest_gui.h 8316b20
tests/kdirlistertest_gui.cpp 11e89cf
Diff: https://git.reviewboard.kde.org/r/129205/diff/
Testing
-------
For 3. i still can't figure out why in filenamesearch signal for delete item(s) is not triggered.
1. Search for file (by name) in dolphin
2. When appear in view delete him
3. Signal ItemsDeleted is not triggered, file stays in the view even if new search is performed and reload is needed. The cache look good, tests pass, works but in filenamesearch.
Thanks,
Anthony Fieroni
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161031/688eea25/attachment.html>
More information about the Kde-frameworks-devel
mailing list