Remove the unused pointer-based attachment deletion method

Summary:
This will help towards making Attachment an implicitly shared value type
too.

Reviewers: dvratil

Reviewed By: dvratil

Subscribers: kde-pim

Tags: KDE PIM

Differential Revision: https://phabricator.kde.org/D22125