+ QMimeData *mMimeData;
Using a QScopedPointer here would mean you don't have to worry about deleting the contents before you change it.
Similar for mPasteReply, that I don't see ever being deleted here - leak?
=== modified file 'src/platforms/mirserver/clipboard.cpp'
+ QDBusPendingCallWatcher *watcher = new QDBusPendingCallWatcher(reply, this);
one of those times "auto" is nicer, since you state the type twice.
Everything else quite sensible. I need to test the silo though
=== modified file 'src/platforms/ mirserver/ clipboard. h'
+ * Copyright (C) 2014,2016 Canonical, Ltd.
2014-2016
+ QMimeData *mMimeData;
Using a QScopedPointer here would mean you don't have to worry about deleting the contents before you change it.
Similar for mPasteReply, that I don't see ever being deleted here - leak?
=== modified file 'src/platforms/ mirserver/ clipboard. cpp' lWatcher *watcher = new QDBusPendingCal lWatcher( reply, this);
+ QDBusPendingCal
one of those times "auto" is nicer, since you state the type twice.
Everything else quite sensible. I need to test the silo though