On Sat, Feb 3, 2018 at 9:29 AM, Michael Niedermayer
<mich...@niedermayer.cc> wrote:
> On Sat, Feb 03, 2018 at 01:36:37AM +0700, Muhammad Faiz wrote:
>> They don't modify AVCodec, no needs to call it at register. They will be
>> wasteful if these codecs are unused. Instead, call static data initialization
>> at codecs' init.
>>
>> Benchmark:
>> old: 51281340 decicycles in avcodec_register_all,       1 runs,      0 skips
>> new:  6738960 decicycles in avcodec_register_all,       1 runs,      0 skips
>>
>> Signed-off-by: Muhammad Faiz <mfc...@gmail.com>
>> ---
>>  libavcodec/jpeg2000dec.c | 16 +++++++++-------
>>  libavcodec/qdmc.c        |  7 +++++--
>>  libavcodec/wmavoice.c    |  7 +++++--
>>  3 files changed, 19 insertions(+), 11 deletions(-)
>
> LGTM

Applied.

>
> it would be better though if this would happen for all
> init_static_data() without the need for extra code per codec

As wm4 explained.

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

Reply via email to