[FFmpeg-devel] [PATCH] use new metadata API in APE demuxer

Kostya kostya.shishkov
Tue Feb 17 07:03:33 CET 2009


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).
 
> Aurel




More information about the ffmpeg-devel mailing list