[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