Re: [FFmpeg-devel] [PATCH 2/8] avcodec/h264_cavlc: Remove redundant check

2021-03-12 Thread Andreas Rheinhardt
Andreas Rheinhardt: > 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 rem

[FFmpeg-devel] [PATCH 2/8] avcodec/h264_cavlc: Remove redundant check

2021-03-09 Thread Andreas Rheinhardt
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: Andrea