[FFmpeg-devel] [PATCH] use new metadata API in realmedia

[Aurelien Jacobs]

Michael Niedermayer wrote:

> On Tue, Feb 17, 2009 at 05:00:53PM +0100, Aurelien Jacobs wrote:
> > On Tue, 17 Feb 2009 16:38:33 +0100
> > Michael Niedermayer <michaelni at gmx.at> wrote:
> > 
> > > On Tue, Feb 17, 2009 at 04:30:44PM +0100, Aurelien Jacobs wrote:
> > > > On Tue, 17 Feb 2009 16:20:35 +0100
> > > > Aurelien Jacobs <aurel at gnuage.org> wrote:
> > > > 
> > > > > [...]
> > > > > Not as is, due to the get_byte() or get_be16(), but OK. Done with
> > > > > a slightly modified get_tag().
> > > > 
> > > > Stupid me... Wrong patch !
> > > > Attached the correct one.
> > > 
> > > [...]
> > > > Index: libavformat/rmdec.c
> > > > ===================================================================
> > > > --- libavformat/rmdec.c	(r?vision 17390)
> > > > +++ libavformat/rmdec.c	(copie de travail)
> > > > @@ -72,6 +72,17 @@
> > > >      get_strl(pb, buf, buf_size, get_byte(pb));
> > > >  }
> > > >  
> > > > +static void get_tag(AVFormatContext *s, int wide)
> > > > +{
> > > > +    char buf[1024];
> > > > +    int i;
> > > > +    for (i=0; i<FF_ARRAY_ELEMS(ff_rm_metadata); i++) {
> > > > +        int len = wide ? get_be16(s->pb) : get_byte(s->pb);
> > > > +        get_strl(s->pb, buf, sizeof(buf), len);
> > > > +        av_metadata_set(&s->metadata, ff_rm_metadata[i], buf);
> > > > +    }
> > > > +}
> > > 
> > > the function name seems inappropriate now
> > 
> > Right. Patch updated.
> 
> ok

Applied.

Aurel