Re: [PATCH 4/6] Serialize all properties

[Víctor M. Jáquez L. <vjaquez igalia com>]

On Fri, May 28, 2010 at 11:56:54AM +0200, Juan A. Suarez Romero wrote:
> On Fri, 2010-05-28 at 11:15 +0200, Víctor M. Jáquez L. wrote:
> > > -grl_media_serialize (GrlMedia *media)
> > > +grl_media_serialize (GrlMedia *media,
> > > +                     gboolean full)
> > 
> > Why not keep grl_media_serialize() which would be a wrapper of a
> > grl_media_serialize_ex() 
> 
> Are you meaning having two serialization functions,
> grl_media_serialize() and grl_media_serialize_extended(), instead of one
> using the "full" parameter?

No.

I meant

grl_media_serialize(GrlMedia *media) {
   grl_media_serialize_extended(media, TRUE);
}

grl_media_serialize(GrlMedia *media, gboolean full) {
   ...
}

grl_media_serialize should be the most used option by the users.

vmjl