[Bug 42281] Doesn't advertise ContactBlocking when supported

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Wed Oct 26 16:01:18 CEST 2011


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

--- Comment #2 from Will Thompson <will.thompson at collabora.co.uk> 2011-10-26 07:01:18 PDT ---
Comment on attachment 52797
  --> https://bugs.freedesktop.org/attachment.cgi?id=52797
Advertise ContactBlocking if supported

Review of attachment 52797:
 --> (https://bugs.freedesktop.org/page.cgi?id=splinter.html&bug=42281&attachment=52797)
-----------------------------------------------------------------

Functionally, looks fine.

::: src/connection.c
@@ +2819,5 @@
> +    {
> +       const gchar *ifaces[] =
> +         { TP_IFACE_CONNECTION_INTERFACE_CONTACT_BLOCKING, NULL };
> +
> +      tp_base_connection_add_interfaces ((TpBaseConnection *) conn, ifaces);

Something is funky with the indentation here. The declaration of 'ifaces' is
one space further in than the function call.

::: tests/twisted/roster/test-google-roster.py
@@ +296,5 @@
>          )
>  
> +    assertContains(cs.CONN_IFACE_CONTACT_BLOCKING,
> +                  conn.Get(cs.CONN, "Interfaces",
> +                           dbus_interface=cs.PROPERTIES_IFACE))

conn.Properties.Get(cs.CONN, "Interfaces")

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



More information about the telepathy-bugs mailing list