Review Request 129394: [filenamesearch] Fix huge ram usage in kded module

Anthony Fieroni bvbfan at abv.bg
Mon Dec 5 20:04:38 UTC 2016


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

Review request for KDE Frameworks, Anthony Fieroni, David Faure, and Emmanuel Pescosta.


Changes
-------

NOTE: If we have more than one filenameseach process, dbus module will fail - epic


Repository: kio-extras


Description
-------

Bug is introduced in https://git.reviewboard.kde.org/r/129297/
When is fixed new kio-extras realease is needed for 16.08 branch.


Diffs (updated)
-----

  filenamesearch/CMakeLists.txt 774d79e 
  filenamesearch/filenamesearchdbusinterface.h PRE-CREATION 
  filenamesearch/filenamesearchdbusinterface.cpp PRE-CREATION 
  filenamesearch/kded/CMakeLists.txt 48160ed 
  filenamesearch/kded/filenamesearchdbusadaptor.h PRE-CREATION 
  filenamesearch/kded/filenamesearchdbusadaptor.cpp PRE-CREATION 
  filenamesearch/kded/filenamesearchmodule.h 4fa6bb1 
  filenamesearch/kded/filenamesearchmodule.cpp f12626d 
  filenamesearch/kio_filenamesearch.h 0920394 
  filenamesearch/kio_filenamesearch.cpp f124f02 

Diff: https://git.reviewboard.kde.org/r/129394/diff/


Testing
-------

No big ram usage but still not works as expected.
1. Perform search in Dolphin
2. Delete one result item
3. View must be update, but it's not


Thanks,

Anthony Fieroni

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161205/55d19f52/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list