Re: [Multi-valued V2 (grilo) 11/12] core: Merge GrlDataMulti into GrlData

["Juan A." Suárez Romero <jasuarez igalia com>]

On Wed, 2011-03-02 at 14:09 +0100, Juan A. Suárez Romero wrote:
> On Tue, 2011-03-01 at 11:18 +0000, Iago Toral wrote:
> > > +/**
> > > + * grl_data_remove_property:
> > > + * @data: a data
> > > + * @key: a metadata key
> > > + * @index: index of key to be removed, starting at 0
> > > + *
> > > + * Removes key and related keys from position in mdata.
> > 
> >  I think this does not remove any key ro related keys, but their
> > values 
> >  for index, right? Please in that case change the description and use
> > '@' 
> >  to refer to parameters.
> > 
> 
> It removes the GrlProperty at @index position for @key in @data.
> 
> It means that if you previously had:
> 
> prop = grl_data_get_property (data, <key>, <index>)
> 
> After doing:
> 
> grl_data_remove_property (data, <key>, <index>)
> 
> 
> "prop" will not be there. Invoking again grl_data_get_property() would
> return a different property (if <key> has multiple values), or @NULL (if
> there wasn't more values).
> 
> 


BTW, I've rephrased the doc as

"Removes #GrlProperty for @key in @data at position @index"

	J.A.