[FFmpeg-devel] [PATCH] use new metadata API in APE demuxer
Aurelien Jacobs
aurel
Tue Feb 17 16:45:44 CET 2009
On Tue, 17 Feb 2009 08:03:33 +0200
Kostya <kostya.shishkov at gmail.com> wrote:
> On Tue, Feb 17, 2009 at 01:05:46AM +0100, Aurelien Jacobs wrote:
> > Kostya wrote:
> >
> > > On Sun, Feb 15, 2009 at 06:02:59PM +0100, Aurelien Jacobs wrote:
> > > > Hi,
> > > >
> > > > Attached patch convert APE demuxer to the new metadata API.
> > >
> > > Almost ok, feel free to commit with the only one question:
> > > maybe it's better to do something when string is longer than
> > > 1024 (show a warning, skip the rest of string, whatever)?
> >
> > Current patch just skip the rest of string. I think it's enough
> > so applied.
> > If you think a warning is useful, I can apply attached patch.
>
> Yes, I think that would be better (and rare).
OK. Applied.
Aurel
More information about the ffmpeg-devel
mailing list