[FFmpeg-devel] [PATCH 20/39] avcodec/h261dec: Don't initialize unused part of RLTable

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Thu Dec 10 13:16:38 EET 2020


The H.261 decoder only uses an RLTable's VLC table, yet it also
initializes its index_run, max_level and max_run. This commit stops
doing so; it will also simplify making this decoder init-threadsafe,
as the H.261 decoder and encoder now initialize disjoint parts of their
common RLTable.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
---
The earlier patchset instead guarded ff_h261_common_init() by an AVOnce.

 libavcodec/h261dec.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index 2dc2fb30f2..6b680a862d 100644
--- a/libavcodec/h261dec.c
+++ b/libavcodec/h261dec.c
@@ -81,7 +81,6 @@ static av_cold int h261_decode_init(AVCodecContext *avctx)
     s->low_delay   = 1;
     avctx->pix_fmt = AV_PIX_FMT_YUV420P;
 
-    ff_h261_common_init();
     h261_decode_init_vlc(h);
 
     h->gob_start_code_skipped = 0;
-- 
2.25.1



More information about the ffmpeg-devel mailing list