Merge lp:~dandrader/qtubuntu/ensureDBusClipboardReady into lp:qtubuntu

Proposed by Daniel d'Andrada
Status: Merged
Approved by: Gerry Boland
Approved revision: 275
Merged at revision: 275
Proposed branch: lp:~dandrader/qtubuntu/ensureDBusClipboardReady
Merge into: lp:qtubuntu
Diff against target: 14 lines (+4/-0)
1 file modified
src/ubuntumirclient/clipboard.cpp (+4/-0)
To merge this branch: bzr merge lp:~dandrader/qtubuntu/ensureDBusClipboardReady
Reviewer Review Type Date Requested Status
Gerry Boland (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+268612@code.launchpad.net

Commit message

UbuntuClipboard: ensure DBus has been set up before setting clipboard contents

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Gerry Boland (gerboland) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/ubuntumirclient/clipboard.cpp'
--- src/ubuntumirclient/clipboard.cpp 2014-09-23 11:49:03 +0000
+++ src/ubuntumirclient/clipboard.cpp 2015-08-20 15:40:22 +0000
@@ -266,6 +266,10 @@
266266
267void UbuntuClipboard::setDBusClipboardContents(const QByteArray &clipboardContents)267void UbuntuClipboard::setDBusClipboardContents(const QByteArray &clipboardContents)
268{268{
269 if (!mDBusSetupDone) {
270 setupDBus();
271 }
272
269 if (!mPendingSetContentsCall.isNull()) {273 if (!mPendingSetContentsCall.isNull()) {
270 // Ignore any previous set call as we are going to overwrite it anyway274 // Ignore any previous set call as we are going to overwrite it anyway
271 QObject::disconnect(mPendingSetContentsCall.data(), 0, this, 0);275 QObject::disconnect(mPendingSetContentsCall.data(), 0, this, 0);

Subscribers

People subscribed via source and target branches