[Differential] [Request, 10 lines] D1026: [FolderModel] Simplify deleting existing drag image
broulik (Kai Uwe Broulik)
noreply at phabricator.kde.org
Fri Feb 26 14:11:25 UTC 2016
broulik created this revision.
broulik added reviewers: Plasma, hein.
broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.
broulik added a project: Plasma.
Herald added a subscriber: plasma-devel.
REVISION SUMMARY
Instead of looking up three times (contains, value, remove) we just use take which removes the item and returns it and delete that. If there is no such item Qt promises to return a null pointer which is safe to delete.
REPOSITORY
rPLASMADESKTOP Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D1026
AFFECTED FILES
containments/desktop/plugins/folder/foldermodel.cpp
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, Plasma, hein
Cc: plasma-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160226/9df0378f/attachment.html>
More information about the Plasma-devel
mailing list