[Bug 45410] Telepathy 1.0: start the 'next' branch

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jan 30 20:57:54 CET 2012


https://bugs.freedesktop.org/show_bug.cgi?id=45410

--- Comment #1 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2012-01-30 11:57:54 PST ---
I reviewed up to e62456367481018 - first pass, looking at the diff only, not
reading the HTML.

Ultra-low-hanging fruit:

git grep Simple and consider:

s/\<Simple_Presence\>/Presence/
s/\<Simple_Status_Spec/Status_Spec/
s/\<Simple_Contact_Presences\>/Contact_Presence_Map/ (?)

git grep telepathy/ and replace with telepathy1 where it appears in object
paths (at least one occurrence, in Debug)

Call the Debug object .../Debug instead of .../debug?

Remove the Detailed from HandleOwnersChangedDetailed

Hold should probably have an xor-requires on Call and Content, not a requires

Maybe rename Protocol (the typed string) to Protocol_Name to avoid confusion
with Protocol objects?

Set fire to Rich_Presence_Access_Control and just use Access_Control in
Location

> +    <p>The namespace <code>im.telepathy1.Qt4.Error.</code>

should be Qt.Error now?

DTMF should lose all references to stream IDs

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



More information about the telepathy-bugs mailing list