[Bug 54917] Should have GVariant helpers similar to tp_asv_

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Sat Sep 15 15:44:50 CEST 2012


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

--- Comment #3 from Xavier Claessens <xclaesse at gmail.com> 2012-09-15 13:44:50 UTC ---
(In reply to comment #1)
> http://cgit.collabora.com/git/user/xclaesse/telepathy-glib.git/commit/?h=variant&id=cb2fbab3b4079843e240035ebbf7c4f318e36f06
> + * Since: 0.UNRELEASED
> 
> These API are not new so maybe we should remove this.

Only the internal functions are not new, all the public stuff are new.

> http://cgit.collabora.com/git/user/xclaesse/telepathy-glib.git/commit/?h=variant&id=ca47ad360751210e016632bbc8a8e50bc4aaace8
> 
> +GVariantClass
> +tp_variant_type_classify (const GVariantType *type)
> +{
> +  return *g_variant_type_peek_string (type);
> 
> This is worth a comment and I'd use [0] (easier to understand).

done

> +
> +  switch (g_variant_classify (variant))
> +    {
> +    case G_VARIANT_CLASS_DOUBLE:
> 
> 'case' should be idented.
> 
> Same for the others switch.

done

> http://cgit.collabora.com/git/user/xclaesse/telepathy-glib.git/commit/?h=variant&id=c6dc12c3f6b64eb5f526ccf775df5b399ec1d4a7
> 
> Can't you link on the internal lib and use _tp_asv_to_vardict() ?

Actually I'm wondering why we don't do that for all tests... done.

All fixed and merged.

-- 
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