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, -- Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
