Review Request 129205: [kcoredirlister] Ability to watch files changes

Anthony Fieroni bvbfan at abv.bg
Mon Oct 24 07:05:40 UTC 2016


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

(Updated Oct. 24, 2016, 10:05 a.m.)


Review request for KDE Frameworks and David Faure.


Changes
-------

Test case


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 (updated)
-----

  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/20161024/2668ddda/attachment.html>


More information about the Kde-frameworks-devel mailing list