[virt-tools-list] [libosinfo v4] API to query required user avatar format

[Christophe Fergeau]

On Fri, Nov 23, 2012 at 05:26:19AM +0200, Zeeshan Ali (Khattak) wrote:
> @@ -157,6 +165,8 @@ osinfo_install_script_finalize (GObject *object)
>      g_free(script->priv->output_prefix);
>      g_free(script->priv->output_filename);
>      g_list_free_full(script->priv->config_param_list, g_object_unref);
> +    if (script->priv->avatar != NULL)
> +        g_object_unref(script->priv->avatar);


I've heard a few times that g_*_free should be done in _finalize while
g_object_unref should be called from _dispose. I don't know how important
this is (ie we can probably live with this _unref in _finalize).

ACK.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20121123/27b338e2/attachment.sig>