[FFmpeg-cvslog] avcodec/intrax8: Avoid superfluous VLC structures

Andreas Rheinhardt git at videolan.org
Tue Oct 31 22:15:14 EET 2023


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Wed Sep 20 22:24:18 2023 +0200| [c95e123e8c7664598be10b0306d2a4d60ad2cf12] | committer: Andreas Rheinhardt

avcodec/intrax8: Avoid superfluous VLC structures

Of all these VLCs here, only VLC.table was really used
after init, so use the ff_vlc_init_tables API
to get rid of them.

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

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

 libavcodec/intrax8.c | 46 ++++++++++++++++++++--------------------------
 1 file changed, 20 insertions(+), 26 deletions(-)

diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c
index 0d90e37007..b6aacb170a 100644
--- a/libavcodec/intrax8.c
+++ b/libavcodec/intrax8.c
@@ -45,49 +45,43 @@
 #define AC_VLC_MTD MAX_TABLE_DEPTH(AC_VLC_BITS, MAX_AC_VLC_BITS)
 #define OR_VLC_MTD MAX_TABLE_DEPTH(OR_VLC_BITS, MAX_OR_VLC_BITS)
 
-static VLC j_ac_vlc[2][2][8];  // [quant < 13], [intra / inter], [select]
-static VLC j_dc_vlc[2][8];     // [quant], [select]
-static VLC j_orient_vlc[2][4]; // [quant], [select]
+static const VLCElem *j_ac_vlc[2][2][8];  // [quant < 13], [intra / inter], [select]
+static const VLCElem *j_dc_vlc[2][8];     // [quant], [select]
+static const VLCElem *j_orient_vlc[2][4]; // [quant], [select]
 
-static av_cold void x8_init_vlc(VLC *vlc, int nb_bits, int nb_codes,
-                                int *offset, const uint8_t table[][2])
+static av_cold const VLCElem *x8_init_vlc(VLCInitState *state, int nb_bits,
+                                          int nb_codes, const uint8_t table[][2])
 {
-    static VLCElem vlc_buf[VLC_BUFFER_SIZE];
-
-    vlc->table           = &vlc_buf[*offset];
-    vlc->table_allocated = VLC_BUFFER_SIZE - *offset;
-    ff_vlc_init_from_lengths(vlc, nb_bits, nb_codes, &table[0][1], 2,
-                             &table[0][0], 2, 1, 0, VLC_INIT_STATIC_OVERLONG, NULL);
-    *offset += vlc->table_size;
+    return ff_vlc_init_tables_from_lengths(state, nb_bits, nb_codes, &table[0][1], 2,
+                                           &table[0][0], 2, 1, 0, 0);
 }
 
 static av_cold void x8_vlc_init(void)
 {
+    static VLCElem vlc_buf[VLC_BUFFER_SIZE];
+    VLCInitState state = VLC_INIT_STATE(vlc_buf);
     int i;
-    int offset = 0;
 
 // set ac tables
     for (int i = 0; i < 2; i++)
         for (int j = 0; j < 2; j++)
             for (int k = 0; k < 8; k++)
-                x8_init_vlc(&j_ac_vlc[i][j][k], AC_VLC_BITS, 77,
-                            &offset, x8_ac_quant_table[i][j][k]);
+                j_ac_vlc[i][j][k] = x8_init_vlc(&state, AC_VLC_BITS, 77,
+                                                x8_ac_quant_table[i][j][k]);
 
 // set dc tables
     for (int i = 0; i < 2; i++)
         for (int j = 0; j < 8; j++)
-            x8_init_vlc(&j_dc_vlc[i][j], DC_VLC_BITS, 34, &offset,
-                        x8_dc_quant_table[i][j]);
+            j_dc_vlc[i][j] = x8_init_vlc(&state, DC_VLC_BITS, 34,
+                                         x8_dc_quant_table[i][j]);
 
 // set orient tables
     for (i = 0; i < 2; i++)
-        x8_init_vlc(&j_orient_vlc[0][i], OR_VLC_BITS, 12,
-                    &offset, x8_orient_highquant_table[i]);
+        j_orient_vlc[0][i] = x8_init_vlc(&state, OR_VLC_BITS, 12,
+                                         x8_orient_highquant_table[i]);
     for (i = 0; i < 4; i++)
-        x8_init_vlc(&j_orient_vlc[1][i], OR_VLC_BITS, 12,
-                    &offset, x8_orient_lowquant_table[i]);
-
-    av_assert2(offset == VLC_BUFFER_SIZE);
+        j_orient_vlc[1][i] = x8_init_vlc(&state, OR_VLC_BITS, 12,
+                                         x8_orient_lowquant_table[i]);
 }
 
 static void x8_reset_vlc_tables(IntraX8Context *w)
@@ -108,7 +102,7 @@ static inline void x8_select_ac_table(IntraX8Context *const w, int mode)
 
     table_index       = get_bits(w->gb, 3);
     // 2 modes use same tables
-    w->j_ac_vlc_table[mode] = j_ac_vlc[w->quant < 13][mode >> 1][table_index].table;
+    w->j_ac_vlc_table[mode] = j_ac_vlc[w->quant < 13][mode >> 1][table_index];
     av_assert2(j_ac_vlc[mode]);
 }
 
@@ -116,7 +110,7 @@ static inline int x8_get_orient_vlc(IntraX8Context *w)
 {
     if (!w->j_orient_vlc_table) {
         int table_index = get_bits(w->gb, 1 + (w->quant < 13));
-        w->j_orient_vlc_table = j_orient_vlc[w->quant < 13][table_index].table;
+        w->j_orient_vlc_table = j_orient_vlc[w->quant < 13][table_index];
     }
 
     return get_vlc2(w->gb, w->j_orient_vlc_table, OR_VLC_BITS, OR_VLC_MTD);
@@ -258,7 +252,7 @@ static int x8_get_dc_rlf(IntraX8Context *const w, const int mode,
     if (!w->j_dc_vlc_table[mode]) {
         int table_index = get_bits(w->gb, 3);
         // 4 modes, same table
-        w->j_dc_vlc_table[mode] = j_dc_vlc[w->quant < 13][table_index].table;
+        w->j_dc_vlc_table[mode] = j_dc_vlc[w->quant < 13][table_index];
     }
 
     i = get_vlc2(w->gb, w->j_dc_vlc_table[mode], DC_VLC_BITS, DC_VLC_MTD);



More information about the ffmpeg-cvslog mailing list