[FFmpeg-devel] [PATCH] avformat/mov: read stream bitrates from isml manifest

[compn]

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.

some ideas:
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.

-compn