[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