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

[Iago Toral <itoral igalia com>]

On Wed, 02 Mar 2011 14:09:59 +0100, "Juan A." Suárez Romero 
<jasuarez igalia com> 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).

aha, then what about this?:

"Removes the value at position @index for @key. If there are other keys 
related to @key,
their values at position @index will also be removed."

Iago