Re: [FFmpeg-devel] [PATCH v2] avfilter: sort filter list

[Muhammad Faiz]

On Fri, Feb 2, 2018 at 5:36 PM, Nicolas George  wrote:
> Muhammad Faiz (2018-02-02):
>> Move REGISTER_FILTER to FILTER_TABLE in configure.
>> Replace linked list with static table, and sort it.
>> Use bsearch() on avfilter_get_by_name().
>> Deprecate avfilter_register_all(), avfilter_register(), and
>> avfilter_next().
>> Add avfilter_iterate() as a replacement for avfilter_next().
>
> I'll be explicit: unless discussed otherwise, I oppose this patch as
> long as it is not part of a series that applies the same treatment to
> all components (codecs, parsers, formats).

Yeah, probably no needs to rush on this.

Thank's.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2] avfilter: sort filter list

[Nicolas George]

Muhammad Faiz (2018-02-02):
> Move REGISTER_FILTER to FILTER_TABLE in configure.
> Replace linked list with static table, and sort it.
> Use bsearch() on avfilter_get_by_name().
> Deprecate avfilter_register_all(), avfilter_register(), and
> avfilter_next().
> Add avfilter_iterate() as a replacement for avfilter_next().

I'll be explicit: unless discussed otherwise, I oppose this patch as
long as it is not part of a series that applies the same treatment to
all components (codecs, parsers, formats).

Regards,

-- 
  Nicolas George


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel