[FFmpeg-cvslog] avcodec/h264_cavlc: Remove redundant check

Andreas Rheinhardt git at videolan.org
Mon Mar 15 08:01:52 EET 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Tue Mar  9 20:25:06 2021 +0100| [b41e4f6187a10e729288fcf9ff5584f47a6a34e9] | committer: Andreas Rheinhardt

avcodec/h264_cavlc: Remove redundant check

The only caller to ff_h264_decode_init_vlc() already uses
ff_thread_once() for the call; ergo the check via a simple int with
static storage duration in ff_h264_decode_init_vlc() is redundant.
And if it were not redundant, it would be a potential for data races.
So remove it.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b41e4f6187a10e729288fcf9ff5584f47a6a34e9
---

 libavcodec/h264_cavlc.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/libavcodec/h264_cavlc.c b/libavcodec/h264_cavlc.c
index 6481992e58..d0548d84f6 100644
--- a/libavcodec/h264_cavlc.c
+++ b/libavcodec/h264_cavlc.c
@@ -325,12 +325,8 @@ static av_cold void init_cavlc_level_tab(void){
 }
 
 av_cold void ff_h264_decode_init_vlc(void){
-    static int done = 0;
-
-    if (!done) {
         int i;
         int offset;
-        done = 1;
 
         chroma_dc_coeff_token_vlc.table = chroma_dc_coeff_token_vlc_table;
         chroma_dc_coeff_token_vlc.table_allocated = chroma_dc_coeff_token_vlc_table_size;
@@ -410,7 +406,6 @@ av_cold void ff_h264_decode_init_vlc(void){
                  INIT_VLC_USE_NEW_STATIC);
 
         init_cavlc_level_tab();
-    }
 }
 
 static inline int get_level_prefix(GetBitContext *gb){



More information about the ffmpeg-cvslog mailing list