Quoting Andreas Rheinhardt (2020-12-10 12:16:20) > ff_mpeg12_init_vlcs() currently initializes index_run, max_level and > max_run of ff_rl_mpeg1/2; yet the only user of these fields is the > MPEG-1/2 encoder which already initializes these tables on its own. > So remove the initializations in ff_mpeg12_init_vlcs(); this also > simplifies making ff_mpeg12_init_vlcs() thread-safe. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > libavcodec/mpeg12.c | 4 ---- > libavcodec/mpeg12.h | 2 -- > libavcodec/mpeg12enc.c | 6 ++++-- > 3 files changed, 4 insertions(+), 8 deletions(-)
Looks good, thought it's really confusing how rl_vlc is initialized both by ff_rl_init() and by INIT_2D_VLC_RL() -- Anton Khirnov _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".