[Telepathy] [Bug 19248] Patch that implement Connection.Interface.SimplePresence

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jan 5 13:00:24 PST 2009


http://bugs.freedesktop.org/show_bug.cgi?id=19248





--- Comment #6 from Jonny Lamb <jonny.lamb at collabora.co.uk>  2009-01-05 13:00:24 PST ---
Thanks for the updated patch. Just three tiny comments:

 * telepathy.server.DBusProperties.__init__(self)
   This is an unnecessary addition as telepathy.server.Connection calls this in
   its constructor. To be fair, this is a little harsh on you as I told you to
   post this patch before I released tp-python.

 * "(presence_type,presence,personal_message)" needs spaces

 * There are excessive newlines between the two classes in simple_presence.py.

It's just worth noting here for my benefit that this requires an updated dep of
tp-python to 0.15.4.

Thanks again for your patch.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the Telepathy mailing list