diff mbox series

[FFmpeg-devel,11/39] avcodec/ituh263dec: Don't initialize unused RL VLCs

Message ID 20201210111657.2276739-12-andreas.rheinhardt@gmail.com
State Accepted
Commit a3831e81ff368f6c8f098b5c3dad80cae91bacae
Headers show
Series Make mpegvideo encoders init-threadsafe
Related show


Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished

Commit Message

Andreas Rheinhardt Dec. 10, 2020, 11:16 a.m. UTC
The ff_rl_intra_aic RLTable is only used by ituh263dec and ituh263enc;
the former is the only user of its RL VLC tables. It uses only the very
first one of these VLC tables, but up until now all 32 are initialized,
wasting 68696B from the .bss segment (or that amount of memory if this
decoder has actually been used). This commit changes this.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
 libavcodec/ituh263dec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series


diff --git a/libavcodec/ituh263dec.c b/libavcodec/ituh263dec.c
index ced7fa52ee..7ddb33b795 100644
--- a/libavcodec/ituh263dec.c
+++ b/libavcodec/ituh263dec.c
@@ -125,7 +125,7 @@  av_cold void ff_h263_decode_init_vlc(void)
         ff_rl_init(&ff_h263_rl_inter, ff_h263_static_rl_table_store[0]);
         ff_rl_init(&ff_rl_intra_aic, ff_h263_static_rl_table_store[1]);
         INIT_VLC_RL(ff_h263_rl_inter, 554);
-        INIT_VLC_RL(ff_rl_intra_aic, 554);
+        INIT_FIRST_VLC_RL(ff_rl_intra_aic, 554);
         INIT_VLC_STATIC(&h263_mbtype_b_vlc, H263_MBTYPE_B_VLC_BITS, 15,
                  &ff_h263_mbtype_b_tab[0][1], 2, 1,
                  &ff_h263_mbtype_b_tab[0][0], 2, 1, 80);