Review Request: update the data when the predicate files change

Aaron Seigo aseigo at kde.org
Mon Jan 4 10:53:13 CET 2010


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

Ship it!


looks good; pls commit.

- Aaron


On 2009-12-07 17:38:58, Giulio Camuffo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2331/
> -----------------------------------------------------------
> 
> (Updated 2009-12-07 17:38:58)
> 
> 
> Review request for Plasma and Aaron Seigo.
> 
> 
> Summary
> -------
> 
> This patch uses a KDirWatch to watch the folders where the predicate files are in. If one of them gets created, modified, deleted, it updates its data so the applets using it receive a dataUpdated() and can adjust their things.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/hotplug/CMakeLists.txt 1059391 
>   /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/hotplug/hotplugengine.h 1059391 
>   /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/hotplug/hotplugengine.cpp 1059391 
> 
> Diff: http://reviewboard.kde.org/r/2331/diff
> 
> 
> Testing
> -------
> 
> tested, works. the notifier changes its "x actions for this device" instantly. (the notifier needs a fix for it anyway, since the actions themselves aren't changed. i'll do it once this is committed)
> 
> However, maybe would be a goog idea to cache somehow the value of the function predicatesForDevice() in order to avoid to parse everytime all the predicates and so parse only the new or changed ones, cause it seems to have quite a big impact on performances.
> 
> 
> Thanks,
> 
> Giulio
> 
>



More information about the Plasma-devel mailing list