Small API change exception for KActivities::Info

Ivan Čukić ivan.cukic at kde.org
Tue Jan 15 11:51:38 GMT 2013


Hi all,

I'm requesting an api freeze exception for the KActivities::Info class to 
introduce a simple const method:
    bool isResourceLinked(const KUrl & url);

The implementation for the method already exists, it is just not published in 
the API.

The reason for this is that because the former doesn't exist, some* are using 
the currently existing (and deprecated**) method - linkedResources - which 
returns a list of all linked resources like this:
    linkedResources().contains(...)

This is *very* bad - it is slow - just to check linking for one specific 
resource, the above line queries nepomuk for the whole list of resources, 
those resources are then transferred via d-bus to kactivitymanagerd, then 
(again via dbus) from kactivitymanagerd to the client that uses the library.

This could have been a simple yes/no question, with a much faster sparql query 
than retrieving the whole list is.

I have Marco's +1 on this.***

Cheerio,
Ivan

* Share-Like-Connect that will be released with PA4, and for normal Plasma 
soon after 4.10 is released
** Deprecated because it is blocking, and slow
*** http://mail.kde.org/pipermail/plasma-devel/2013-January/023496.html

-- 
You know, there are many people in the country today who,
through no fault of their own, are sane. Some of them were born sane.
Some of them became sane later in their lives...
  -- Monty Python's Flying Circus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130115/98fa478f/attachment.sig>


More information about the kde-core-devel mailing list