Re: [PATCH 5/5] core: Add a default root media in notify_change()
- From: "Juan A." Suárez Romero <jasuarez igalia com>
- To: grilo-list gnome org
- Subject: Re: [PATCH 5/5] core: Add a default root media in notify_change()
- Date: Tue, 01 Feb 2011 15:56:07 +0100
On Tue, 2011-02-01 at 13:24 +0000, Iago Toral wrote:
>  In that case I'd still rework the code like this:
> 
>   if (!media) {
>    media = grl_media_box_new ();
>    free_media =TRUE;
>   }
>   grl_media_set_source (media,
>                         grl_metadata_source_get_id
> (GRL_METADATA_SOURCE
>   (source)));
> 
>  ...
> 
>  if (free_media) {
>    g_object_unref (media);
>  } 
Ok. I'll do as you suggest.
	J.A.
[
Date Prev][
Date Next]   [
Thread Prev][
Thread Next]   
[
Thread Index]
[
Date Index]
[
Author Index]