Review Request: Add removeMatch(es) methods to runnerContext

Jacopo De Simoi wilderkde at gmail.com
Sun Nov 22 10:15:50 CET 2009


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

Review request for Plasma and Aaron Seigo.


Summary
-------

This patch adds removeMatch and removeMatches methods to RunnerContext; these methods are needed to update results in krunner in case the runner knows that something changed and the previously reported results are not valid anymore (e.g. a window listed by the windows runner is closed [this currently leads to a crash] or a device is removed / changed its state (solid runner), a file has been deleted (places runner, nepomuk))

Both methods fire matchesChanged; in case the match is to be modified the runner can exploit the event compression done by runnermanager by removing the match and adding it back right away with the required modifications. 


Diffs
-----

  trunk/KDE/kdelibs/plasma/querymatch.h 1052445 
  trunk/KDE/kdelibs/plasma/querymatch.cpp 1052445 
  trunk/KDE/kdelibs/plasma/runnercontext.h 1052445 
  trunk/KDE/kdelibs/plasma/runnercontext.cpp 1052445 

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


Testing
-------

tested (in PoC stage) with the solidrunner and it works


Thanks,

Jacopo



More information about the Plasma-devel mailing list