2016-10-14 20:00 GMT+08:00 wm4 <nfx...@googlemail.com>: > On Thu, 13 Oct 2016 11:27:12 +0200 > Nicolas George <geo...@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.mccar...@shaw.ca> > > > Based-on: patch by Gorilla Maguila <gorilla.magu...@gmail.com> > > > Signed-off-by: Steven Liu <l...@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@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
