[Differential] [Request, 65 lines] D2395: [Milou] Optimize and cleanup sources model
broulik (Kai Uwe Broulik)
noreply at phabricator.kde.org
Wed Aug 10 14:30:58 UTC 2016
broulik created this revision.
broulik added reviewers: Plasma, vhanda.
broulik set the repository for this revision to rMILOU Milou.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
- Simplify queryString once, rather than during every iteration
- Foreach instead of potentially detaching range-for
- Use reverse iterators (new in Qt 5.6) instead of Java-style iterators
- Remove traces of begin/endInsert/RemoveRows feature - while it would be nice to not reset the model with all this sorting and merging going on it's pretty difficult to predict where an item will end up which is probably why it never got implemented. It still calculated tons of indices that went unused, so remove this and go a different route in the future
TEST PLAN
Seems to work
REPOSITORY
rMILOU Milou
REVISION DETAIL
https://phabricator.kde.org/D2395
AFFECTED FILES
lib/sourcesmodel.cpp
lib/sourcesmodel.h
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma, vhanda
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160810/c6d711c8/attachment.html>
More information about the Plasma-devel
mailing list