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

Aurelien Jacobs aurel
Tue Feb 17 01:05:46 CET 2009


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.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ape_warning.diff
Type: text/x-patch
Size: 418 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090217/2dcd4557/attachment.bin>



More information about the ffmpeg-devel mailing list