[FFmpeg-devel] [PATCH] add hds demuxer

Steven Liu lingjiujianke at gmail.com
Fri Oct 14 16:30:46 EEST 2016


2016-10-14 20:00 GMT+08:00 wm4 <nfxjfg at googlemail.com>:

> On Thu, 13 Oct 2016 11:27:12 +0200
> Nicolas George <george at nsup.org> wrote:
>
> > 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,
> >
>
> Rather than NIHing it, I think using libexpat would be a good choice
> (even if its API is terrible).
>
ok, i'll learn how to use it, and use it next step. :-)

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list