telepathy-qt: Add Q_UNUSED as Q_ASSERT is a no-op in release mode

David Edmundson davidedmundson at kemper.freedesktop.org
Wed Apr 2 09:53:45 PDT 2014


Module: telepathy-qt
Branch: master
Commit: ff169cde751d0bb0309109514dd10b9eff7de4e3
URL:    http://cgit.freedesktop.org/telepathy/telepathy-qt/commit/?id=ff169cde751d0bb0309109514dd10b9eff7de4e3

Author: Uwe L. Korn <uwelk at xhochy.com>
Date:   Fri Mar 28 10:28:16 2014 +0000

Add Q_UNUSED as Q_ASSERT is a no-op in release mode

Reviewed-by: David Edmundson

---

 TelepathyQt/connection-manager.cpp     |    1 +
 examples/file-transfer/file-sender.cpp |    2 ++
 2 files changed, 3 insertions(+)

diff --git a/TelepathyQt/connection-manager.cpp b/TelepathyQt/connection-manager.cpp
index 96b3234..e6e32e0 100644
--- a/TelepathyQt/connection-manager.cpp
+++ b/TelepathyQt/connection-manager.cpp
@@ -1084,6 +1084,7 @@ void ConnectionManager::gotParametersLegacy(QDBusPendingCallWatcher *watcher)
         ++pos;
     }
     Q_ASSERT(found);
+    Q_UNUSED(found);
 
     if (!reply.isError()) {
         debug() << QString(QLatin1String("Got reply to ConnectionManager.GetParameters(%1)")).arg(protocolName);
diff --git a/examples/file-transfer/file-sender.cpp b/examples/file-transfer/file-sender.cpp
index a747af1..50d50c2 100644
--- a/examples/file-transfer/file-sender.cpp
+++ b/examples/file-transfer/file-sender.cpp
@@ -91,6 +91,7 @@ void FileSender::onAMReady(PendingOperation *op)
 
     PendingReady *pr = qobject_cast<PendingReady*>(op);
     Q_ASSERT(pr != NULL);
+    Q_UNUSED(pr);
     qDebug() << "AccountManager ready";
 
     mAccount = mAM->accountForObjectPath(
@@ -116,6 +117,7 @@ void FileSender::onAccountReady(PendingOperation *op)
 
     PendingReady *pr = qobject_cast<PendingReady*>(op);
     Q_ASSERT(pr != NULL);
+    Q_UNUSED(pr);
     qDebug() << "Account ready";
 
     qDebug() << "Checking if account is online...";



More information about the telepathy-commits mailing list