[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