[FFmpeg-devel] [PATCH] avformat/mov: read stream bitrates from isml manifest
tempn at twmi.rr.com
Thu Jun 27 15:14:46 CEST 2013
On Thu, 27 Jun 2013 12:16:07 +0200, Alexandre Sicard wrote:
>Le jeudi 27 juin 2013 10:36:11 Michael Niedermayer a écrit :
>> > +#include <regex.h>
>> This is unavaliable on at least windows AFAIK
>Oops, this is a problem. Parsing XML would require regex or an XML parsing
>library. Actually, I first wrote this patch using libexpat, but I thought it
>was not worth adding a dependency when I could just use regex.
>Now, I think it could be useful: I have a colleague working on adding Smooth
>Streaming playback support to FFmpeg, and for now he uses libexpat to parse
>the manifest. Other adaptive streaming formats such as DASH and HDS have XML
>manifests too, so it could definitively be of use.
>What do you think about this? Should I send a patch adding libexpat to FFmpeg
>and the libexpat version of my patch?
i dont think we ever came to a consensus on xml parsing in ffmpeg.
1. dependency on xml lib like libexpat
2. rewrite small xml parser for libavutil
3. parse xml as text with a bunch of hacks like mplayer does.
4. include regex in ffmpeg
so yes , please submit a patch with libexpat support.
we'll go from there.
More information about the ffmpeg-devel