[FFmpeg-devel] [PATCH] add hds demuxer

[Steven Liu]

2016-10-13 17:27 GMT+08:00 Nicolas George <george at nsup.org>:

> Le duodi 22 vendémiaire, an CCXXV, Steven Liu a écrit :
> > init add hds demuxer
> >
> > Based-on: patch by CORY MCCARTHY <cory.mccarthy at shaw.ca>
> > Based-on: patch by Gorilla Maguila <gorilla.maguila at gmail.com>
> > Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
>
> Thanks for the update.
>
> I think I already said it when it was first submitted: I am not really
> happy
> about the use of libxml2. I remember a time when libxml2 was a "security
> advisory of the week" kind of library, and looking at the changelog, it
> does
> not seem to have progressed a lot since.
>
> To be fair, implementing a full-featured XML parser is very hard work and
> time-consuming.
>
> But really, almost nobody needs that, and especially not FFmpeg. What we
> need is a parser capable of reading elements, attributes, text in three
> standard encodings, and that is all. Implementing this should be reasonably
> short work for a decent developer.
>
> Of course, since I can not volunteer to do it myself soon, I will not
> consider it grounds for rejecting the patch. But please consider it. If
> libxml2 is enabled by default, that makes it a security concern; if it is
> not, that gives this code much less usefulness and testing.
>
> Regards,
>
> --
>   Nicolas George
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
Hi Nicolas George,


      I cannot sure if i misunderstand your mean. But i think your suggest
is very useful, maybe this can do at TODO.  Is your mean i need add a
simple parser for f4m manifest?

Thanks