diff options
author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-10-22 18:55:04 +0200 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-10-27 15:44:38 +0200 |
commit | 4190019ff4672f38835fc0d03206bdf4278a8cb4 (patch) | |
tree | 19e9cd8c39f56d998841ca2bcaca52516522ceab /libavcodec | |
parent | 92b81a7c99ef82d8d8810f4432c46fc6ccffb59d (diff) | |
download | ffmpeg-4190019ff4672f38835fc0d03206bdf4278a8cb4.tar.gz |
avcodec/speedhqenc: Don't initialize unused parts of RLTable
ff_rl_init() initializes RLTable.(max_level|max_run|index_run);
max_run is unused by the SpeedHQ encoder (as well as MPEG-1/2).
Furthermore, it initializes these things twice (for two passes),
but the second half of this is never used.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'libavcodec')
-rw-r--r-- | libavcodec/speedhqenc.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/libavcodec/speedhqenc.c b/libavcodec/speedhqenc.c index 44ee62b9c2..65e66afae4 100644 --- a/libavcodec/speedhqenc.c +++ b/libavcodec/speedhqenc.c @@ -36,10 +36,12 @@ #include "mpegvideo.h" #include "mpegvideodata.h" #include "mpegvideoenc.h" +#include "rl.h" #include "speedhq.h" #include "speedhqenc.h" -static uint8_t speedhq_static_rl_table_store[2][2*MAX_RUN + MAX_LEVEL + 3]; +static uint8_t speedhq_max_level[MAX_LEVEL + 1]; +static uint8_t speedhq_index_run[MAX_RUN + 1]; /* Exactly the same as MPEG-2, except little-endian. */ static const uint16_t mpeg12_vlc_dc_lum_code_reversed[12] = { @@ -64,7 +66,8 @@ typedef struct SpeedHQEncContext { static av_cold void speedhq_init_static_data(void) { - ff_rl_init(&ff_rl_speedhq, speedhq_static_rl_table_store); + ff_rl_init_level_run(speedhq_max_level, speedhq_index_run, + ff_speedhq_run, ff_speedhq_level, SPEEDHQ_RL_NB_ELEMS); /* build unified dc encoding tables */ for (int i = -255; i < 256; i++) { @@ -88,7 +91,7 @@ static av_cold void speedhq_init_static_data(void) speedhq_chr_dc_uni[i + 255] = bits + (code << 8); } - ff_mpeg1_init_uni_ac_vlc(ff_rl_speedhq.max_level[0], ff_rl_speedhq.index_run[0], + ff_mpeg1_init_uni_ac_vlc(speedhq_max_level, speedhq_index_run, ff_speedhq_vlc_table, uni_speedhq_ac_vlc_len); } @@ -220,8 +223,8 @@ static void encode_block(MpegEncContext *s, int16_t *block, int n) MASK_ABS(sign, alevel); sign &= 1; - if (alevel <= ff_rl_speedhq.max_level[0][run]) { - code = ff_rl_speedhq.index_run[0][run] + alevel - 1; + if (alevel <= speedhq_max_level[run]) { + code = speedhq_index_run[run] + alevel - 1; /* store the VLC & sign at once */ put_bits_le(&s->pb, ff_speedhq_vlc_table[code][1] + 1, ff_speedhq_vlc_table[code][0] | (sign << ff_speedhq_vlc_table[code][1])); |