D25698: New query mechanism for applications: KApplicationTrader

[Albert Astals Cid]

aacid added inline comments.

INLINE COMMENTS

> kapplicationtrader.h:56
> +     * true for all services, this would return the complete list of all
> +     * installed applications (slow).
> +     *

why is it slow? Looking at the code we have to go trhough all apps anyway since what we do is erase if returning false, so wouldn't returning true actually be faster?

REPOSITORY
  R309 KService

REVISION DETAIL
  https://phabricator.kde.org/D25698

To: dfaure, broulik, mart, vkrause, nicolasfella, aacid, davidedmundson
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200119/fa12a90c/attachment.html>