On Tue, Jan 30, 2018 at 02:24:12PM +0700, Muhammad Faiz wrote: > Move REGISTER_FILTER to FILTER_TABLE in configure. > Auto generate filter extern and filter table. > Sort filter table, use bsearch on avfilter_get_by_name. > Define next pointer at filter extern, no need to initialize > next pointer at run time, so AVFilter can be set to const.
> Make avfilter_register always return error. That breaks API Its also a step away from supporting plugins. Why plugins matter ? Because having plugin support is a big advantage, it allows a much wider community to work on, write and maintain filters. With plugins, people can write filters that are written in languages other than C. Or filters which some developer in FFmpeg doesnt want. Or they can be maintained externally by people who just do not like us. Or by people who perfer a FOSS license different from LGPL/GPL/BSD. Iam sure others can come up with more reasons ... Of course avfilter_register() isnt enough for plugins but it or something equivalent is needed for plugins. So i would prefer if avfilter_register() stays supported indefinitly or in case a different system is written for plugins then until that system is in place. That is just a preferrance, not an objection to the patch. [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just because microsoft did it is even more stupid. If everything ms did were stupid they would be bankrupt already.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
