Re: [PATCH 5/5] core: Add a default root media in notify_change()



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]