summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--TelepathyQt4/connection.cpp23
-rw-r--r--TelepathyQt4/connection.h9
-rw-r--r--TelepathyQt4/pending-contact-attributes.cpp4
-rw-r--r--TelepathyQt4/pending-contacts.cpp2
-rw-r--r--tests/dbus/chan-basics.cpp15
-rw-r--r--tests/dbus/chan-group.cpp15
-rw-r--r--tests/dbus/conn-basics.cpp15
-rw-r--r--tests/dbus/conn-capabilities.cpp2
-rw-r--r--tests/dbus/conn-requests.cpp2
-rw-r--r--tests/dbus/conn-roster-groups.cpp3
-rw-r--r--tests/dbus/conn-roster.cpp2
-rw-r--r--tests/dbus/contacts-capabilities.cpp2
-rw-r--r--tests/dbus/contacts.cpp15
-rw-r--r--tests/dbus/handles.cpp15
14 files changed, 65 insertions, 59 deletions
diff --git a/TelepathyQt4/connection.cpp b/TelepathyQt4/connection.cpp
index 752ea623..84cb3c2c 100644
--- a/TelepathyQt4/connection.cpp
+++ b/TelepathyQt4/connection.cpp
@@ -594,22 +594,22 @@ Connection::~Connection()
* The returned value may have changed whenever statusChanged() is
* emitted.
*
- * \return The status, as defined in Status.
+ * \return The status, as defined in Connection::Status.
*/
-uint Connection::status() const
+Connection::Status Connection::status() const
{
- return mPriv->status;
+ return (Connection::Status) mPriv->status;
}
/**
* Return the reason for the connection's status (which is returned by
* status()). The validity and change rules are the same as for status().
*
- * \return The reason, as defined in Status.
+ * \return The reason, as defined in ConnectionStatusReason.
*/
-uint Connection::statusReason() const
+ConnectionStatusReason Connection::statusReason() const
{
- return mPriv->statusReason;
+ return (ConnectionStatusReason) mPriv->statusReason;
}
/**
@@ -806,7 +806,8 @@ void Connection::onStatusReady(uint status)
mPriv->status = status;
mPriv->statusReason = mPriv->pendingStatusReason;
- emit statusChanged(mPriv->status, mPriv->statusReason);
+ emit statusChanged((Connection::Status) mPriv->status,
+ (ConnectionStatusReason) mPriv->statusReason);
}
void Connection::onStatusChanged(uint status, uint reason)
@@ -1508,7 +1509,7 @@ PendingOperation *Connection::requestDisconnect()
* \return Pointer to a newly constructed PendingContactAttributes, tracking the progress of the
* request.
*/
-PendingContactAttributes *Connection::getContactAttributes(const UIntList &handles,
+PendingContactAttributes *Connection::contactAttributes(const UIntList &handles,
const QStringList &interfaces, bool reference)
{
debug() << "Request for attributes for" << handles.size() << "contacts";
@@ -1517,16 +1518,16 @@ PendingContactAttributes *Connection::getContactAttributes(const UIntList &handl
new PendingContactAttributes(ConnectionPtr(this),
handles, interfaces, reference);
if (!isReady()) {
- warning() << "Connection::getContactAttributes() used when not ready";
+ warning() << "Connection::contactAttributes() used when not ready";
pending->failImmediately(TELEPATHY_ERROR_NOT_AVAILABLE, "The connection isn't ready");
return pending;
} /* FIXME: readd this check when Connection isn't FSCKING broken anymore: else if (status() != StatusConnected) {
- warning() << "Connection::getContactAttributes() used with status" << status() << "!= StatusConnected";
+ warning() << "Connection::contactAttributes() used with status" << status() << "!= StatusConnected";
pending->failImmediately(TELEPATHY_ERROR_NOT_AVAILABLE,
"The connection isn't Connected");
return pending;
} */else if (!this->interfaces().contains(TELEPATHY_INTERFACE_CONNECTION_INTERFACE_CONTACTS)) {
- warning() << "Connection::getContactAttributes() used without the remote object supporting"
+ warning() << "Connection::contactAttributes() used without the remote object supporting"
<< "the Contacts interface";
pending->failImmediately(TELEPATHY_ERROR_NOT_IMPLEMENTED,
"The connection doesn't support the Contacts interface");
diff --git a/TelepathyQt4/connection.h b/TelepathyQt4/connection.h
index 1cfae648..c188d287 100644
--- a/TelepathyQt4/connection.h
+++ b/TelepathyQt4/connection.h
@@ -87,8 +87,8 @@ public:
virtual ~Connection();
- uint status() const;
- uint statusReason() const;
+ Status status() const;
+ ConnectionStatusReason statusReason() const;
uint selfHandle() const;
ContactPtr selfContact() const;
@@ -107,7 +107,7 @@ public:
PendingHandles *requestHandles(uint handleType, const QStringList &names);
PendingHandles *referenceHandles(uint handleType, const UIntList &handles);
- PendingContactAttributes *getContactAttributes(const UIntList &handles,
+ PendingContactAttributes *contactAttributes(const UIntList &handles,
const QStringList &interfaces, bool reference = true);
QStringList contactAttributeInterfaces() const;
ContactManager *contactManager() const;
@@ -160,7 +160,8 @@ public:
}
Q_SIGNALS:
- void statusChanged(uint newStatus, uint newStatusReason);
+ void statusChanged(Tp::Connection::Status newStatus,
+ Tp::ConnectionStatusReason newStatusReason);
void selfHandleChanged(uint newHandle);
// FIXME: might not need this when Renaming is fixed and mapped to Contacts
void selfContactChanged();
diff --git a/TelepathyQt4/pending-contact-attributes.cpp b/TelepathyQt4/pending-contact-attributes.cpp
index b1d032ad..62a9bb66 100644
--- a/TelepathyQt4/pending-contact-attributes.cpp
+++ b/TelepathyQt4/pending-contact-attributes.cpp
@@ -56,7 +56,7 @@ namespace Tp
* \headerfile <TelepathyQt4/pending-contact-attributes.h> <TelepathyQt4/PendingContactAttributes>
*
* Class containing the parameters of and the reply to an asynchronous request for raw contact
- * attributes, as used in the Connection::getContactAttributes() low-level convenience method
+ * attributes, as used in the Connection::contactAttributes() low-level convenience method
* wrapping the ConnectionInterfaceContactsInterface::GetContactAttributes() D-Bus method.
*/
@@ -125,7 +125,7 @@ const QStringList &PendingContactAttributes::interfacesRequested() const
/**
* Returns whether it was requested that the contact handles should be referenced in addition to
* fetching their attributes. This corresponds to the <code>reference</code> argument to
- * Connection::getContactAttributes().
+ * Connection::contactAttributes().
*
* \return Whether the handles should be referenced or not.
*/
diff --git a/TelepathyQt4/pending-contacts.cpp b/TelepathyQt4/pending-contacts.cpp
index 6e14583e..7b602b62 100644
--- a/TelepathyQt4/pending-contacts.cpp
+++ b/TelepathyQt4/pending-contacts.cpp
@@ -363,7 +363,7 @@ PendingContacts::PendingContacts(ContactManager *manager,
if (conn->interfaces().contains(TELEPATHY_INTERFACE_CONNECTION_INTERFACE_CONTACTS)) {
debug() << " Building contacts using contact attributes";
PendingContactAttributes *attributes =
- conn->getContactAttributes(otherContacts.toList(),
+ conn->contactAttributes(otherContacts.toList(),
interfaces, true);
connect(attributes,
diff --git a/tests/dbus/chan-basics.cpp b/tests/dbus/chan-basics.cpp
index 879e06f3..6f36c95d 100644
--- a/tests/dbus/chan-basics.cpp
+++ b/tests/dbus/chan-basics.cpp
@@ -30,7 +30,7 @@ public:
{ }
protected Q_SLOTS:
- void expectConnReady(uint, uint);
+ void expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason);
void expectConnInvalidated();
void expectPendingHandleFinished(Tp::PendingOperation*);
void expectCreateChannelFinished(Tp::PendingOperation *);
@@ -56,7 +56,8 @@ private:
uint mHandle;
};
-void TestChanBasics::expectConnReady(uint newStatus, uint newStatusReason)
+void TestChanBasics::expectConnReady(Tp::Connection::Status newStatus,
+ Tp::ConnectionStatusReason newStatusReason)
{
qDebug() << "connection changed to status" << newStatus;
switch (newStatus) {
@@ -212,14 +213,14 @@ void TestChanBasics::initTestCase()
if (mConn->status() != Connection::StatusConnected) {
QVERIFY(connect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
- SLOT(expectConnReady(uint, uint))));
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
QCOMPARE(mLoop->exec(), 0);
QVERIFY(disconnect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
this,
- SLOT(expectConnReady(uint, uint))));
- QCOMPARE(mConn->status(), (uint) Connection::StatusConnected);
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
QVERIFY(mConn->requestsInterface() != 0);
diff --git a/tests/dbus/chan-group.cpp b/tests/dbus/chan-group.cpp
index d8505ff2..097c661d 100644
--- a/tests/dbus/chan-group.cpp
+++ b/tests/dbus/chan-group.cpp
@@ -33,7 +33,7 @@ public:
{ }
protected Q_SLOTS:
- void expectConnReady(uint, uint);
+ void expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason);
void expectConnInvalidated();
void expectChanInvalidated(Tp::DBusProxy*,const QString &, const QString &);
void expectPendingRoomHandlesFinished(Tp::PendingOperation*);
@@ -87,7 +87,8 @@ private:
QString mChanInvalidatedErrorMessage;
};
-void TestChanGroup::expectConnReady(uint newStatus, uint newStatusReason)
+void TestChanGroup::expectConnReady(Tp::Connection::Status newStatus,
+ Tp::ConnectionStatusReason newStatusReason)
{
qDebug() << "connection changed to status" << newStatus;
switch (newStatus) {
@@ -331,14 +332,14 @@ void TestChanGroup::initTestCase()
if (mConn->status() != Connection::StatusConnected) {
QVERIFY(connect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
- SLOT(expectConnReady(uint, uint))));
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
QCOMPARE(mLoop->exec(), 0);
QVERIFY(disconnect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
this,
- SLOT(expectConnReady(uint, uint))));
- QCOMPARE(mConn->status(), (uint) Connection::StatusConnected);
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
QVERIFY(mConn->requestsInterface() != 0);
diff --git a/tests/dbus/conn-basics.cpp b/tests/dbus/conn-basics.cpp
index 947b82a1..f2c32f1f 100644
--- a/tests/dbus/conn-basics.cpp
+++ b/tests/dbus/conn-basics.cpp
@@ -27,7 +27,7 @@ public:
{ }
protected Q_SLOTS:
- void expectConnReady(uint, uint);
+ void expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason);
void expectConnInvalidated();
void expectPresenceAvailable(const Tp::SimplePresence &);
@@ -46,7 +46,8 @@ private:
ConnectionPtr mConn;
};
-void TestConnBasics::expectConnReady(uint newStatus, uint newStatusReason)
+void TestConnBasics::expectConnReady(Tp::Connection::Status newStatus,
+ Tp::ConnectionStatusReason newStatusReason)
{
qDebug() << "connection changed to status" << newStatus;
switch (newStatus) {
@@ -135,14 +136,14 @@ void TestConnBasics::init()
if (mConn->status() != Connection::StatusConnected) {
QVERIFY(connect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
- SLOT(expectConnReady(uint, uint))));
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
QCOMPARE(mLoop->exec(), 0);
QVERIFY(disconnect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
this,
- SLOT(expectConnReady(uint, uint))));
- QCOMPARE(mConn->status(), (uint) Connection::StatusConnected);
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
}
diff --git a/tests/dbus/conn-capabilities.cpp b/tests/dbus/conn-capabilities.cpp
index b2227843..21374f25 100644
--- a/tests/dbus/conn-capabilities.cpp
+++ b/tests/dbus/conn-capabilities.cpp
@@ -89,7 +89,7 @@ void TestConnCapabilities::testCapabilities()
SLOT(expectSuccessfulCall(Tp::PendingOperation*))));
QCOMPARE(mLoop->exec(), 0);
QCOMPARE(mConn->isReady(), true);
- QCOMPARE(mConn->status(), static_cast<uint>(Connection::StatusConnected));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
QVERIFY(mConn->requestsInterface() != 0);
QVERIFY(mConn->capabilities() != 0);
QCOMPARE(mConn->capabilities()->supportsTextChats(), true);
diff --git a/tests/dbus/conn-requests.cpp b/tests/dbus/conn-requests.cpp
index 71f9ce23..a5ac7fd1 100644
--- a/tests/dbus/conn-requests.cpp
+++ b/tests/dbus/conn-requests.cpp
@@ -182,7 +182,7 @@ void TestConnRequests::initTestCase()
QCOMPARE(mLoop->exec(), 0);
QCOMPARE(mConn->isReady(), true);
- QCOMPARE(mConn->status(), static_cast<uint>(Connection::StatusConnected));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
QVERIFY(mConn->requestsInterface() != 0);
}
diff --git a/tests/dbus/conn-roster-groups.cpp b/tests/dbus/conn-roster-groups.cpp
index 06c4a248..fb53dd66 100644
--- a/tests/dbus/conn-roster-groups.cpp
+++ b/tests/dbus/conn-roster-groups.cpp
@@ -129,8 +129,7 @@ void TestConnRosterGroups::init()
SLOT(expectSuccessfulCall(Tp::PendingOperation*))));
QCOMPARE(mLoop->exec(), 0);
QCOMPARE(mConn->isReady(), true);
- QCOMPARE(mConn->status(), static_cast<uint>(Connection::StatusConnected));
-
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
void TestConnRosterGroups::testRosterGroups()
diff --git a/tests/dbus/conn-roster.cpp b/tests/dbus/conn-roster.cpp
index 43433a50..830b9064 100644
--- a/tests/dbus/conn-roster.cpp
+++ b/tests/dbus/conn-roster.cpp
@@ -132,7 +132,7 @@ void TestConnRoster::init()
SLOT(expectSuccessfulCall(Tp::PendingOperation*))));
QCOMPARE(mLoop->exec(), 0);
QCOMPARE(mConn->isReady(), true);
- QCOMPARE(mConn->status(), static_cast<uint>(Connection::StatusConnected));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
void TestConnRoster::testRoster()
diff --git a/tests/dbus/contacts-capabilities.cpp b/tests/dbus/contacts-capabilities.cpp
index 11f042a6..f4f43554 100644
--- a/tests/dbus/contacts-capabilities.cpp
+++ b/tests/dbus/contacts-capabilities.cpp
@@ -114,7 +114,7 @@ void TestContactsCapabilities::initTestCase()
QCOMPARE(mLoop->exec(), 0);
QCOMPARE(mConn->isReady(), true);
- QCOMPARE(mConn->status(), static_cast<uint>(Connection::StatusConnected));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
QVERIFY(mConn->contactManager()->supportedFeatures().contains(Contact::FeatureCapabilities));
}
diff --git a/tests/dbus/contacts.cpp b/tests/dbus/contacts.cpp
index 40f99e49..dcb8e44c 100644
--- a/tests/dbus/contacts.cpp
+++ b/tests/dbus/contacts.cpp
@@ -34,7 +34,7 @@ public:
}
protected Q_SLOTS:
- void expectConnReady(uint newStatus, uint newStatusReason);
+ void expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason);
void expectConnInvalidated();
void expectPendingContactsFinished(Tp::PendingOperation *);
@@ -62,7 +62,8 @@ private:
Tp::UIntList mInvalidHandles;
};
-void TestContacts::expectConnReady(uint newStatus, uint newStatusReason)
+void TestContacts::expectConnReady(Tp::Connection::Status newStatus,
+ Tp::ConnectionStatusReason newStatusReason)
{
switch (newStatus) {
case Connection::StatusDisconnected:
@@ -167,14 +168,14 @@ void TestContacts::initTestCase()
if (mConn->status() != Connection::StatusConnected) {
QVERIFY(connect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
- SLOT(expectConnReady(uint, uint))));
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
QCOMPARE(mLoop->exec(), 0);
QVERIFY(disconnect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
this,
- SLOT(expectConnReady(uint, uint))));
- QCOMPARE(mConn->status(), static_cast<uint>(Connection::StatusConnected));
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
}
diff --git a/tests/dbus/handles.cpp b/tests/dbus/handles.cpp
index 8c885230..a5cd9ef5 100644
--- a/tests/dbus/handles.cpp
+++ b/tests/dbus/handles.cpp
@@ -29,7 +29,7 @@ public:
{ }
protected Q_SLOTS:
- void expectConnReady(uint newStatus, uint newStatusReason);
+ void expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason);
void expectConnInvalidated();
void expectPendingHandlesFinished(Tp::PendingOperation*);
@@ -49,7 +49,8 @@ private:
ReferencedHandles mHandles;
};
-void TestHandles::expectConnReady(uint newStatus, uint newStatusReason)
+void TestHandles::expectConnReady(Tp::Connection::Status newStatus,
+ Tp::ConnectionStatusReason newStatusReason)
{
switch (newStatus) {
case Connection::StatusDisconnected:
@@ -148,14 +149,14 @@ void TestHandles::initTestCase()
if (mConn->status() != Connection::StatusConnected) {
QVERIFY(connect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
- SLOT(expectConnReady(uint, uint))));
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
QCOMPARE(mLoop->exec(), 0);
QVERIFY(disconnect(mConn.data(),
- SIGNAL(statusChanged(uint, uint)),
+ SIGNAL(statusChanged(Tp::Connection::Status, Tp::ConnectionStatusReason)),
this,
- SLOT(expectConnReady(uint, uint))));
- QCOMPARE(mConn->status(), (uint) Connection::StatusConnected);
+ SLOT(expectConnReady(Tp::Connection::Status, Tp::ConnectionStatusReason))));
+ QCOMPARE(mConn->status(), Connection::StatusConnected);
}
}