Attachment 34549 Details for Bug 27379 – Patch as described

[patch] Patch as described

0001-Add-a-new-signal-allKnownContactsChanged-to-monitor-.patch (text/plain), 4.26 KB, created by Dario Freddi on 2010-03-30 13:42:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Dario Freddi

Created: 2010-03-30 13:42:54 UTC

Size: 4.26 KB

patch

obsolete

>From 78ab480f3081832e0670c2c1028052d38b994e66 Mon Sep 17 00:00:00 2001
>From: Dario Freddi <drf@kde.org>
>Date: Tue, 30 Mar 2010 22:35:59 +0200
>Subject: [PATCH] Add a new signal, allKnownContactsChanged, to monitor contact addition/removal
>
>Signed-off-by: Dario Freddi <drf@kde.org>
>---
> TelepathyQt4/contact-manager.cpp |   35 +++++++++++++++++++++++++++++++++++
> TelepathyQt4/contact-manager.h   |    2 ++
> 2 files changed, 37 insertions(+), 0 deletions(-)
>
>diff --git a/TelepathyQt4/contact-manager.cpp b/TelepathyQt4/contact-manager.cpp
>index bb52f9d..e2339a8 100644
>--- a/TelepathyQt4/contact-manager.cpp
>+++ b/TelepathyQt4/contact-manager.cpp
>@@ -56,6 +56,8 @@ struct TELEPATHY_QT4_NO_EXPORT ContactManager::Private
>     Private(ContactManager *parent, const ConnectionPtr &connection)
>         : parent(parent), connection(connection)
>     {
>+        // Make the cache start from the current known contacts
>+        cachedAllKnownContacts = allKnownContacts();
>     }
> 
>     QString addContactListGroupChannel(const ChannelPtr &contactListGroupChannel)
>@@ -91,6 +93,7 @@ struct TELEPATHY_QT4_NO_EXPORT ContactManager::Private
>     ContactManager *parent;
>     WeakPtr<Connection> connection;
>     QMap<uint, QWeakPointer<Contact> > contacts;
>+    Contacts cachedAllKnownContacts;
> 
>     QMap<Contact::Feature, bool> tracking;
>     void ensureTracking(Contact::Feature feature);
>@@ -106,6 +109,7 @@ struct TELEPATHY_QT4_NO_EXPORT ContactManager::Private
> 
>     Contacts allKnownContacts() const;
>     void updateContactsPresenceState();
>+    void computeKnownContactsChanges(const Contacts &added, const Contacts &removed);
> };
> 
> ConnectionPtr ContactManager::connection() const
>@@ -797,6 +801,13 @@ void ContactManager::onSubscribeChannelMembersChanged(
>         debug() << "Contact" << contact->id() << "removed from subscribe list";
>         contact->setSubscriptionState(Contact::PresenceStateNo);
>     }
>+
>+    // Perform the needed computation for allKnownContactsChanged
>+    Contacts allAdded;
>+    allAdded.unite(groupMembersAdded);
>+    allAdded.unite(groupLocalPendingMembersAdded);
>+    allAdded.unite(groupRemotePendingMembersAdded);
>+    mPriv->computeKnownContactsChanges(allAdded, groupMembersRemoved);
> }
> 
> void ContactManager::onPublishChannelMembersChanged(
>@@ -830,6 +841,13 @@ void ContactManager::onPublishChannelMembersChanged(
>     if (!groupLocalPendingMembersAdded.isEmpty()) {
>         emit presencePublicationRequested(groupLocalPendingMembersAdded);
>     }
>+
>+    // Perform the needed computation for allKnownContactsChanged
>+    Contacts allAdded;
>+    allAdded.unite(groupMembersAdded);
>+    allAdded.unite(groupLocalPendingMembersAdded);
>+    allAdded.unite(groupRemotePendingMembersAdded);
>+    mPriv->computeKnownContactsChanges(allAdded, groupMembersRemoved);
> }
> 
> void ContactManager::onDenyChannelMembersChanged(
>@@ -1095,6 +1113,23 @@ Contacts ContactManager::Private::allKnownContacts() const
>     return contacts;
> }
> 
>+void ContactManager::Private::computeKnownContactsChanges(const Tp::Contacts& added, const Tp::Contacts& removed)
>+{
>+    // First of all, compute the real additions/removals based upon our cache
>+    Tp::Contacts realAdded = added;
>+    realAdded.subtract(cachedAllKnownContacts);
>+    Tp::Contacts realRemoved = removed;
>+    realRemoved.intersect(cachedAllKnownContacts);
>+
>+    // Are there any real changes?
>+    if (!realAdded.isEmpty() || !realRemoved.isEmpty()) {
>+        // Yes, update our "cache" and emit the signal
>+        cachedAllKnownContacts.unite(realAdded);
>+        cachedAllKnownContacts.subtract(realRemoved);
>+        emit parent->allKnownContactsChanged(realAdded, realRemoved);
>+    }
>+}
>+
> void ContactManager::Private::updateContactsPresenceState()
> {
>     Contacts subscribeContacts;
>diff --git a/TelepathyQt4/contact-manager.h b/TelepathyQt4/contact-manager.h
>index 834916e..34c825c 100644
>--- a/TelepathyQt4/contact-manager.h
>+++ b/TelepathyQt4/contact-manager.h
>@@ -113,6 +113,8 @@ Q_SIGNALS:
>     void groupMembersChanged(const QString &group,
>             const Tp::Contacts &groupMembersAdded,
>             const Tp::Contacts &groupMembersRemoved);
>+    void allKnownContactsChanged(const Tp::Contacts &contactsAdded,
>+            const Tp::Contacts &contactsRemoved);
> 
> private Q_SLOTS:
>     void onAliasesChanged(const Tp::AliasPairList &);
>-- 
>1.7.0.3
>

Actions: View | Splinter Review

Attachments on bug 27379: 34549 | 34550 | 34560 | 34589

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.