Re: [PATCH 11/15] core: Use constructors to create GrlMetadataSourceFooSpec

["Juan A. Suarez Romero" <jasuarez igalia com>]

On Tue, 2010-08-10 at 08:56 +0200, Iago Toral wrote:
> think I am missing something, this is what I see in the patches (from
> brose_spec_unref):
> 
> +  if (g_atomic_int_dec_and_test (&bs->ref_count)) {
> +    if (bs->source) {
> +      g_object_unref (bs->source);
> +    }
> +    if (bs->container) {
> +      g_object_unref (bs->container);
> +    }
> +    if (bs->keys) {
> +      g_list_free (bs->keys);
> +    }
> +    g_free (bs);
>  
> I don't see where the keymap is freed :-? 


It is done in free_set_metadata_ctl_cb_info (): it frees all
SetMetadataCtlCb content, which does an unref() of spec.


	J.A.