<div>ivan marked 2 inline comments as done.</div><br /><div><strong>INLINE COMMENTS</strong><div><div>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.</div><div>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).<br />
<br />
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.</div><div>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.</div><div>imports/activitymanager/backport/switcheractivitiesmodel.cpp:372 Heh, switching from boost classes...</div><div>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.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D986" rel="noreferrer">https://phabricator.kde.org/D986</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>ivan, davidedmundson, sebas, mart<br /><strong>Cc: </strong>plasma-devel<br /></div>