[Bug 55108] GVariant-based tp_protocol_new, and test coverage

[bugzilla-daemon at freedesktop.org]

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

Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #9 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> ---
(In reply to comment #6)
> Comment on attachment 94898 [details] [review]
> protocol: add TpProtocol:protocol-properties-vardict
> 
> Review of attachment 94898 [details] [review]:
> -----------------------------------------------------------------
> 
> ::: telepathy-glib/protocol.c
> @@ +2387,5 @@
> > + */
> > +GVariant *
> > +tp_protocol_dup_immutable_properties (TpProtocol *self)
> > +{
> > +  return _tp_asv_to_vardict (self->priv->protocol_properties);
> 
> This will need sinking when you merge this to next.

Done that.

Merged to next as well.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.