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]