[Differential] [Updated] D986: Revamp of the activity switcher backend

[ivan (Ivan Čukić)]

ivan marked 2 inline comments as done.

INLINE COMMENTS
  imports/activitymanager/backport/model_updaters.h:3 It is a really old file from KActivities. I didn't want to change the year just because it was copied here.
  imports/activitymanager/backport/switcheractivitiesmodel.cpp:281 .contains would search for the pointer value, and not for any pointer that represents the same activity like it does now (the collection is sorted by activity name and id).
  
  Now, this might be faster with std::find_if since it is a small collection, and serial search is faster than binary for small ranges, but I don't want to optimize prematurely.
  imports/activitymanager/backport/switcheractivitiesmodel.cpp:323 The state change influences whether the activity is shown in the model or not, it has a special handler.
  imports/activitymanager/backport/switcheractivitiesmodel.cpp:372 Heh, switching from boost classes...
  imports/activitymanager/sortedactivitiesmodel.cpp:239 The source model calls dataChanged because it changes the current property of activities. And it works. Now, this might be made more robust by calling sort when lasttimeused is changed.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: ivan, davidedmundson, sebas, mart
Cc: plasma-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160218/fad4b7be/attachment-0001.html>