summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOlli Salli <ollisal@gmail.com>2011-12-30 18:47:44 (GMT)
committerOlli Salli <ollisal@gmail.com>2011-12-30 18:47:44 (GMT)
commita015ae27e7a9ca727a06488c843f4716e596968b (patch)
tree2a91c026277d7ce4c3b5f454d99647f0228a42d4
parent6fe5e533ef7694770df652886a5c5ea18888a1a4 (diff)
downloadtelepathy-qt4-a015ae27e7a9ca727a06488c843f4716e596968b.tar.gz
telepathy-qt4-a015ae27e7a9ca727a06488c843f4716e596968b.tar.xz
Add explanation to Presence::setStatus
To be translated to proper documentation when the class is documented in general
-rw-r--r--TelepathyQt/presence.cpp6
1 files changed, 6 insertions, 0 deletions
diff --git a/TelepathyQt/presence.cpp b/TelepathyQt/presence.cpp
index 32fd91c..3cb8756 100644
--- a/TelepathyQt/presence.cpp
+++ b/TelepathyQt/presence.cpp
@@ -167,6 +167,7 @@ QString Presence::statusMessage() const
return mPriv->sp.statusMessage;
}
+// Sets all fields
void Presence::setStatus(const SimplePresence &value)
{
if (!isValid()) {
@@ -177,6 +178,11 @@ void Presence::setStatus(const SimplePresence &value)
mPriv->sp = value;
}
+// TODO: explain in proper docs that we don't have setStatusType and setStatus(QString status)
+// separately, because:
+// 1) type and status are tightly related with each other
+// 2) all statuses can't have status message so changing the status alone might make the presence
+// illegal if a message was left around
void Presence::setStatus(ConnectionPresenceType type, const QString &status,
const QString &statusMessage)
{