[FFmpeg-devel] [PATCH] avcodec/hevc_cabac: Let compiler count offsets

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Fri Mar 8 04:28:25 EET 2024


This is easily possible with an X macro.
Using an enum for the offsets also allows to remove
two arrays which are not really needed and will typically
be optimized away by the compiler: The first just exists
to count the number of syntax elements*, the second one
exists to get offset[CONSTANT]. These constants were
of type enum SyntaxElement and this enum was only used
in hevc_cabac.c (although it was declared in hevcdec.h);
it is now no longer needed at all and has therefore been
removed.

The first of these arrays led to a warning from Clang
which is fixed by this commit:
warning: variable 'num_bins_in_se' is not needed and will
not be emitted [-Wunneeded-internal-declaration]

*: One could also just added a trailing SYNTAX_ELEMENT_NB
to the SyntaxElement enum for this purpose.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
 libavcodec/hevc_cabac.c | 248 ++++++++++++++++------------------------
 libavcodec/hevcdec.h    |  52 ---------
 2 files changed, 100 insertions(+), 200 deletions(-)

diff --git a/libavcodec/hevc_cabac.c b/libavcodec/hevc_cabac.c
index 6b38da84bd..63ffb3d37c 100644
--- a/libavcodec/hevc_cabac.c
+++ b/libavcodec/hevc_cabac.c
@@ -31,114 +31,66 @@
 
 #define CABAC_MAX_BIN 31
 
-/**
- * number of bin by SyntaxElement.
- */
-static const int8_t num_bins_in_se[] = {
-     1, // sao_merge_flag
-     1, // sao_type_idx
-     0, // sao_eo_class
-     0, // sao_band_position
-     0, // sao_offset_abs
-     0, // sao_offset_sign
-     0, // end_of_slice_flag
-     3, // split_coding_unit_flag
-     1, // cu_transquant_bypass_flag
-     3, // skip_flag
-     3, // cu_qp_delta
-     1, // pred_mode
-     4, // part_mode
-     0, // pcm_flag
-     1, // prev_intra_luma_pred_mode
-     0, // mpm_idx
-     0, // rem_intra_luma_pred_mode
-     2, // intra_chroma_pred_mode
-     1, // merge_flag
-     1, // merge_idx
-     5, // inter_pred_idc
-     2, // ref_idx_l0
-     2, // ref_idx_l1
-     2, // abs_mvd_greater0_flag
-     2, // abs_mvd_greater1_flag
-     0, // abs_mvd_minus2
-     0, // mvd_sign_flag
-     1, // mvp_lx_flag
-     1, // no_residual_data_flag
-     3, // split_transform_flag
-     2, // cbf_luma
-     5, // cbf_cb, cbf_cr
-     2, // transform_skip_flag[][]
-     2, // explicit_rdpcm_flag[][]
-     2, // explicit_rdpcm_dir_flag[][]
-    18, // last_significant_coeff_x_prefix
-    18, // last_significant_coeff_y_prefix
-     0, // last_significant_coeff_x_suffix
-     0, // last_significant_coeff_y_suffix
-     4, // significant_coeff_group_flag
-    44, // significant_coeff_flag
-    24, // coeff_abs_level_greater1_flag
-     6, // coeff_abs_level_greater2_flag
-     0, // coeff_abs_level_remaining
-     0, // coeff_sign_flag
-     8, // log2_res_scale_abs
-     2, // res_scale_sign_flag
-     1, // cu_chroma_qp_offset_flag
-     1, // cu_chroma_qp_offset_idx
-};
+// ELEM(NAME, NUM_BINS)
+#define CABAC_ELEMS(ELEM)                     \
+    ELEM(SAO_MERGE_FLAG, 1)                   \
+    ELEM(SAO_TYPE_IDX, 1)                     \
+    ELEM(SAO_EO_CLASS, 0)                     \
+    ELEM(SAO_BAND_POSITION, 0)                \
+    ELEM(SAO_OFFSET_ABS, 0)                   \
+    ELEM(SAO_OFFSET_SIGN, 0)                  \
+    ELEM(END_OF_SLICE_FLAG, 0)                \
+    ELEM(SPLIT_CODING_UNIT_FLAG, 3)           \
+    ELEM(CU_TRANSQUANT_BYPASS_FLAG, 1)        \
+    ELEM(SKIP_FLAG, 3)                        \
+    ELEM(CU_QP_DELTA, 3)                      \
+    ELEM(PRED_MODE_FLAG, 1)                   \
+    ELEM(PART_MODE, 4)                        \
+    ELEM(PCM_FLAG, 0)                         \
+    ELEM(PREV_INTRA_LUMA_PRED_FLAG, 1)        \
+    ELEM(MPM_IDX, 0)                          \
+    ELEM(REM_INTRA_LUMA_PRED_MODE, 0)         \
+    ELEM(INTRA_CHROMA_PRED_MODE, 2)           \
+    ELEM(MERGE_FLAG, 1)                       \
+    ELEM(MERGE_IDX, 1)                        \
+    ELEM(INTER_PRED_IDC, 5)                   \
+    ELEM(REF_IDX_L0, 2)                       \
+    ELEM(REF_IDX_L1, 2)                       \
+    ELEM(ABS_MVD_GREATER0_FLAG, 2)            \
+    ELEM(ABS_MVD_GREATER1_FLAG, 2)            \
+    ELEM(ABS_MVD_MINUS2, 0)                   \
+    ELEM(MVD_SIGN_FLAG, 0)                    \
+    ELEM(MVP_LX_FLAG, 1)                      \
+    ELEM(NO_RESIDUAL_DATA_FLAG, 1)            \
+    ELEM(SPLIT_TRANSFORM_FLAG, 3)             \
+    ELEM(CBF_LUMA, 2)                         \
+    ELEM(CBF_CB_CR, 5)                        \
+    ELEM(TRANSFORM_SKIP_FLAG, 2)              \
+    ELEM(EXPLICIT_RDPCM_FLAG, 2)              \
+    ELEM(EXPLICIT_RDPCM_DIR_FLAG, 2)          \
+    ELEM(LAST_SIGNIFICANT_COEFF_X_PREFIX, 18) \
+    ELEM(LAST_SIGNIFICANT_COEFF_Y_PREFIX, 18) \
+    ELEM(LAST_SIGNIFICANT_COEFF_X_SUFFIX, 0)  \
+    ELEM(LAST_SIGNIFICANT_COEFF_Y_SUFFIX, 0)  \
+    ELEM(SIGNIFICANT_COEFF_GROUP_FLAG, 4)     \
+    ELEM(SIGNIFICANT_COEFF_FLAG, 44)          \
+    ELEM(COEFF_ABS_LEVEL_GREATER1_FLAG, 24)   \
+    ELEM(COEFF_ABS_LEVEL_GREATER2_FLAG, 6)    \
+    ELEM(COEFF_ABS_LEVEL_REMAINING, 0)        \
+    ELEM(COEFF_SIGN_FLAG, 0)                  \
+    ELEM(LOG2_RES_SCALE_ABS, 8)               \
+    ELEM(RES_SCALE_SIGN_FLAG, 2)              \
+    ELEM(CU_CHROMA_QP_OFFSET_FLAG, 1)         \
+    ELEM(CU_CHROMA_QP_OFFSET_IDX, 1)          \
 
 /**
- * Offset to ctxIdx 0 in init_values and states, indexed by SyntaxElement.
+ * Offset to ctxIdx 0 in init_values and states.
  */
-static const int elem_offset[sizeof(num_bins_in_se)] = {
-    0, // sao_merge_flag
-    1, // sao_type_idx
-    2, // sao_eo_class
-    2, // sao_band_position
-    2, // sao_offset_abs
-    2, // sao_offset_sign
-    2, // end_of_slice_flag
-    2, // split_coding_unit_flag
-    5, // cu_transquant_bypass_flag
-    6, // skip_flag
-    9, // cu_qp_delta
-    12, // pred_mode
-    13, // part_mode
-    17, // pcm_flag
-    17, // prev_intra_luma_pred_mode
-    18, // mpm_idx
-    18, // rem_intra_luma_pred_mode
-    18, // intra_chroma_pred_mode
-    20, // merge_flag
-    21, // merge_idx
-    22, // inter_pred_idc
-    27, // ref_idx_l0
-    29, // ref_idx_l1
-    31, // abs_mvd_greater0_flag
-    33, // abs_mvd_greater1_flag
-    35, // abs_mvd_minus2
-    35, // mvd_sign_flag
-    35, // mvp_lx_flag
-    36, // no_residual_data_flag
-    37, // split_transform_flag
-    40, // cbf_luma
-    42, // cbf_cb, cbf_cr
-    47, // transform_skip_flag[][]
-    49, // explicit_rdpcm_flag[][]
-    51, // explicit_rdpcm_dir_flag[][]
-    53, // last_significant_coeff_x_prefix
-    71, // last_significant_coeff_y_prefix
-    89, // last_significant_coeff_x_suffix
-    89, // last_significant_coeff_y_suffix
-    89, // significant_coeff_group_flag
-    93, // significant_coeff_flag
-    137, // coeff_abs_level_greater1_flag
-    161, // coeff_abs_level_greater2_flag
-    167, // coeff_abs_level_remaining
-    167, // coeff_sign_flag
-    167, // log2_res_scale_abs
-    175, // res_scale_sign_flag
-    177, // cu_chroma_qp_offset_flag
-    178, // cu_chroma_qp_offset_idx
+enum {
+#define OFFSET(NAME, NUM_BINS)                     \
+    NAME ## _OFFSET,                               \
+    NAME ## _END = NAME ## _OFFSET + NUM_BINS - 1,
+CABAC_ELEMS(OFFSET)
 };
 
 #define CNU 154
@@ -570,12 +522,12 @@ int ff_hevc_cabac_init(HEVCLocalContext *lc, int ctb_addr_ts)
 
 int ff_hevc_sao_merge_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[SAO_MERGE_FLAG]);
+    return GET_CABAC(SAO_MERGE_FLAG_OFFSET);
 }
 
 int ff_hevc_sao_type_idx_decode(HEVCLocalContext *lc)
 {
-    if (!GET_CABAC(elem_offset[SAO_TYPE_IDX]))
+    if (!GET_CABAC(SAO_TYPE_IDX_OFFSET))
         return 0;
 
     if (!get_cabac_bypass(&lc->cc))
@@ -622,7 +574,7 @@ int ff_hevc_end_of_slice_flag_decode(HEVCLocalContext *lc)
 
 int ff_hevc_cu_transquant_bypass_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[CU_TRANSQUANT_BYPASS_FLAG]);
+    return GET_CABAC(CU_TRANSQUANT_BYPASS_FLAG_OFFSET);
 }
 
 int ff_hevc_skip_flag_decode(HEVCLocalContext *lc, int x0, int y0, int x_cb, int y_cb)
@@ -638,7 +590,7 @@ int ff_hevc_skip_flag_decode(HEVCLocalContext *lc, int x0, int y0, int x_cb, int
     if (lc->ctb_up_flag || y0b)
         inc += !!SAMPLE_CTB(s->skip_flag, x_cb, y_cb - 1);
 
-    return GET_CABAC(elem_offset[SKIP_FLAG] + inc);
+    return GET_CABAC(SKIP_FLAG_OFFSET + inc);
 }
 
 int ff_hevc_cu_qp_delta_abs(HEVCLocalContext *lc)
@@ -647,7 +599,7 @@ int ff_hevc_cu_qp_delta_abs(HEVCLocalContext *lc)
     int suffix_val = 0;
     int inc = 0;
 
-    while (prefix_val < 5 && GET_CABAC(elem_offset[CU_QP_DELTA] + inc)) {
+    while (prefix_val < 5 && GET_CABAC(CU_QP_DELTA_OFFSET + inc)) {
         prefix_val++;
         inc = 1;
     }
@@ -675,7 +627,7 @@ int ff_hevc_cu_qp_delta_sign_flag(HEVCLocalContext *lc)
 
 int ff_hevc_cu_chroma_qp_offset_flag(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[CU_CHROMA_QP_OFFSET_FLAG]);
+    return GET_CABAC(CU_CHROMA_QP_OFFSET_FLAG_OFFSET);
 }
 
 int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc)
@@ -683,7 +635,7 @@ int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc)
     int c_max= FFMAX(5, lc->parent->ps.pps->chroma_qp_offset_list_len_minus1);
     int i = 0;
 
-    while (i < c_max && GET_CABAC(elem_offset[CU_CHROMA_QP_OFFSET_IDX]))
+    while (i < c_max && GET_CABAC(CU_CHROMA_QP_OFFSET_IDX_OFFSET))
         i++;
 
     return i;
@@ -691,7 +643,7 @@ int ff_hevc_cu_chroma_qp_offset_idx(HEVCLocalContext *lc)
 
 int ff_hevc_pred_mode_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[PRED_MODE_FLAG]);
+    return GET_CABAC(PRED_MODE_FLAG_OFFSET);
 }
 
 int ff_hevc_split_coding_unit_flag_decode(HEVCLocalContext *lc, int ct_depth, int x0, int y0)
@@ -712,40 +664,40 @@ int ff_hevc_split_coding_unit_flag_decode(HEVCLocalContext *lc, int ct_depth, in
     inc += (depth_left > ct_depth);
     inc += (depth_top  > ct_depth);
 
-    return GET_CABAC(elem_offset[SPLIT_CODING_UNIT_FLAG] + inc);
+    return GET_CABAC(SPLIT_CODING_UNIT_FLAG_OFFSET + inc);
 }
 
 int ff_hevc_part_mode_decode(HEVCLocalContext *lc, int log2_cb_size)
 {
-    if (GET_CABAC(elem_offset[PART_MODE])) // 1
+    if (GET_CABAC(PART_MODE_OFFSET)) // 1
         return PART_2Nx2N;
     if (log2_cb_size == lc->parent->ps.sps->log2_min_cb_size) {
         if (lc->cu.pred_mode == MODE_INTRA) // 0
             return PART_NxN;
-        if (GET_CABAC(elem_offset[PART_MODE] + 1)) // 01
+        if (GET_CABAC(PART_MODE_OFFSET + 1)) // 01
             return PART_2NxN;
         if (log2_cb_size == 3) // 00
             return PART_Nx2N;
-        if (GET_CABAC(elem_offset[PART_MODE] + 2)) // 001
+        if (GET_CABAC(PART_MODE_OFFSET + 2)) // 001
             return PART_Nx2N;
         return PART_NxN; // 000
     }
 
     if (!lc->parent->ps.sps->amp_enabled_flag) {
-        if (GET_CABAC(elem_offset[PART_MODE] + 1)) // 01
+        if (GET_CABAC(PART_MODE_OFFSET + 1)) // 01
             return PART_2NxN;
         return PART_Nx2N;
     }
 
-    if (GET_CABAC(elem_offset[PART_MODE] + 1)) { // 01X, 01XX
-        if (GET_CABAC(elem_offset[PART_MODE] + 3)) // 011
+    if (GET_CABAC(PART_MODE_OFFSET + 1)) { // 01X, 01XX
+        if (GET_CABAC(PART_MODE_OFFSET + 3)) // 011
             return PART_2NxN;
         if (get_cabac_bypass(&lc->cc)) // 0101
             return PART_2NxnD;
         return PART_2NxnU; // 0100
     }
 
-    if (GET_CABAC(elem_offset[PART_MODE] + 3)) // 001
+    if (GET_CABAC(PART_MODE_OFFSET + 3)) // 001
         return PART_Nx2N;
     if (get_cabac_bypass(&lc->cc)) // 0001
         return PART_nRx2N;
@@ -759,7 +711,7 @@ int ff_hevc_pcm_flag_decode(HEVCLocalContext *lc)
 
 int ff_hevc_prev_intra_luma_pred_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[PREV_INTRA_LUMA_PRED_FLAG]);
+    return GET_CABAC(PREV_INTRA_LUMA_PRED_FLAG_OFFSET);
 }
 
 int ff_hevc_mpm_idx_decode(HEVCLocalContext *lc)
@@ -783,7 +735,7 @@ int ff_hevc_rem_intra_luma_pred_mode_decode(HEVCLocalContext *lc)
 int ff_hevc_intra_chroma_pred_mode_decode(HEVCLocalContext *lc)
 {
     int ret;
-    if (!GET_CABAC(elem_offset[INTRA_CHROMA_PRED_MODE]))
+    if (!GET_CABAC(INTRA_CHROMA_PRED_MODE_OFFSET))
         return 4;
 
     ret  = get_cabac_bypass(&lc->cc) << 1;
@@ -793,7 +745,7 @@ int ff_hevc_intra_chroma_pred_mode_decode(HEVCLocalContext *lc)
 
 int ff_hevc_merge_idx_decode(HEVCLocalContext *lc)
 {
-    int i = GET_CABAC(elem_offset[MERGE_IDX]);
+    int i = GET_CABAC(MERGE_IDX_OFFSET);
 
     if (i != 0) {
         while (i < lc->parent->sh.max_num_merge_cand-1 && get_cabac_bypass(&lc->cc))
@@ -804,17 +756,17 @@ int ff_hevc_merge_idx_decode(HEVCLocalContext *lc)
 
 int ff_hevc_merge_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[MERGE_FLAG]);
+    return GET_CABAC(MERGE_FLAG_OFFSET);
 }
 
 int ff_hevc_inter_pred_idc_decode(HEVCLocalContext *lc, int nPbW, int nPbH)
 {
     if (nPbW + nPbH == 12)
-        return GET_CABAC(elem_offset[INTER_PRED_IDC] + 4);
-    if (GET_CABAC(elem_offset[INTER_PRED_IDC] + lc->ct_depth))
+        return GET_CABAC(INTER_PRED_IDC_OFFSET + 4);
+    if (GET_CABAC(INTER_PRED_IDC_OFFSET + lc->ct_depth))
         return PRED_BI;
 
-    return GET_CABAC(elem_offset[INTER_PRED_IDC] + 4);
+    return GET_CABAC(INTER_PRED_IDC_OFFSET + 4);
 }
 
 int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx)
@@ -823,7 +775,7 @@ int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx)
     int max = num_ref_idx_lx - 1;
     int max_ctx = FFMIN(max, 2);
 
-    while (i < max_ctx && GET_CABAC(elem_offset[REF_IDX_L0] + i))
+    while (i < max_ctx && GET_CABAC(REF_IDX_L0_OFFSET + i))
         i++;
     if (i == 2) {
         while (i < max && get_cabac_bypass(&lc->cc))
@@ -835,22 +787,22 @@ int ff_hevc_ref_idx_lx_decode(HEVCLocalContext *lc, int num_ref_idx_lx)
 
 int ff_hevc_mvp_lx_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[MVP_LX_FLAG]);
+    return GET_CABAC(MVP_LX_FLAG_OFFSET);
 }
 
 int ff_hevc_no_residual_syntax_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[NO_RESIDUAL_DATA_FLAG]);
+    return GET_CABAC(NO_RESIDUAL_DATA_FLAG_OFFSET);
 }
 
 static av_always_inline int abs_mvd_greater0_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[ABS_MVD_GREATER0_FLAG]);
+    return GET_CABAC(ABS_MVD_GREATER0_FLAG_OFFSET);
 }
 
 static av_always_inline int abs_mvd_greater1_flag_decode(HEVCLocalContext *lc)
 {
-    return GET_CABAC(elem_offset[ABS_MVD_GREATER1_FLAG] + 1);
+    return GET_CABAC(ABS_MVD_GREATER1_FLAG_OFFSET + 1);
 }
 
 static av_always_inline int mvd_decode(HEVCLocalContext *lc)
@@ -878,39 +830,39 @@ static av_always_inline int mvd_sign_flag_decode(HEVCLocalContext *lc)
 
 int ff_hevc_split_transform_flag_decode(HEVCLocalContext *lc, int log2_trafo_size)
 {
-    return GET_CABAC(elem_offset[SPLIT_TRANSFORM_FLAG] + 5 - log2_trafo_size);
+    return GET_CABAC(SPLIT_TRANSFORM_FLAG_OFFSET + 5 - log2_trafo_size);
 }
 
 int ff_hevc_cbf_cb_cr_decode(HEVCLocalContext *lc, int trafo_depth)
 {
-    return GET_CABAC(elem_offset[CBF_CB_CR] + trafo_depth);
+    return GET_CABAC(CBF_CB_CR_OFFSET + trafo_depth);
 }
 
 int ff_hevc_cbf_luma_decode(HEVCLocalContext *lc, int trafo_depth)
 {
-    return GET_CABAC(elem_offset[CBF_LUMA] + !trafo_depth);
+    return GET_CABAC(CBF_LUMA_OFFSET + !trafo_depth);
 }
 
 static int hevc_transform_skip_flag_decode(HEVCLocalContext *lc, int c_idx)
 {
-    return GET_CABAC(elem_offset[TRANSFORM_SKIP_FLAG] + !!c_idx);
+    return GET_CABAC(TRANSFORM_SKIP_FLAG_OFFSET + !!c_idx);
 }
 
 static int explicit_rdpcm_flag_decode(HEVCLocalContext *lc, int c_idx)
 {
-    return GET_CABAC(elem_offset[EXPLICIT_RDPCM_FLAG] + !!c_idx);
+    return GET_CABAC(EXPLICIT_RDPCM_FLAG_OFFSET + !!c_idx);
 }
 
 static int explicit_rdpcm_dir_flag_decode(HEVCLocalContext *lc, int c_idx)
 {
-    return GET_CABAC(elem_offset[EXPLICIT_RDPCM_DIR_FLAG] + !!c_idx);
+    return GET_CABAC(EXPLICIT_RDPCM_DIR_FLAG_OFFSET + !!c_idx);
 }
 
 int ff_hevc_log2_res_scale_abs(HEVCLocalContext *lc, int idx)
 {
     int i =0;
 
-    while (i < 4 && GET_CABAC(elem_offset[LOG2_RES_SCALE_ABS] + 4 * idx + i))
+    while (i < 4 && GET_CABAC(LOG2_RES_SCALE_ABS_OFFSET + 4 * idx + i))
         i++;
 
     return i;
@@ -918,7 +870,7 @@ int ff_hevc_log2_res_scale_abs(HEVCLocalContext *lc, int idx)
 
 int ff_hevc_res_scale_sign_flag(HEVCLocalContext *lc, int idx)
 {
-    return GET_CABAC(elem_offset[RES_SCALE_SIGN_FLAG] + idx);
+    return GET_CABAC(RES_SCALE_SIGN_FLAG_OFFSET + idx);
 }
 
 static av_always_inline void last_significant_coeff_xy_prefix_decode(HEVCLocalContext *lc, int c_idx,
@@ -936,13 +888,13 @@ static av_always_inline void last_significant_coeff_xy_prefix_decode(HEVCLocalCo
         ctx_shift = log2_size - 2;
     }
     while (i < max &&
-           GET_CABAC(elem_offset[LAST_SIGNIFICANT_COEFF_X_PREFIX] + (i >> ctx_shift) + ctx_offset))
+           GET_CABAC(LAST_SIGNIFICANT_COEFF_X_PREFIX_OFFSET + (i >> ctx_shift) + ctx_offset))
         i++;
     *last_scx_prefix = i;
 
     i = 0;
     while (i < max &&
-           GET_CABAC(elem_offset[LAST_SIGNIFICANT_COEFF_Y_PREFIX] + (i >> ctx_shift) + ctx_offset))
+           GET_CABAC(LAST_SIGNIFICANT_COEFF_Y_PREFIX_OFFSET + (i >> ctx_shift) + ctx_offset))
         i++;
     *last_scy_prefix = i;
 }
@@ -965,18 +917,18 @@ static av_always_inline int significant_coeff_group_flag_decode(HEVCLocalContext
 
     inc = FFMIN(ctx_cg, 1) + (c_idx>0 ? 2 : 0);
 
-    return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_GROUP_FLAG] + inc);
+    return GET_CABAC(SIGNIFICANT_COEFF_GROUP_FLAG_OFFSET + inc);
 }
 static av_always_inline int significant_coeff_flag_decode(HEVCLocalContext *lc, int x_c, int y_c,
                                            int offset, const uint8_t *ctx_idx_map)
 {
     int inc = ctx_idx_map[(y_c << 2) + x_c] + offset;
-    return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_FLAG] + inc);
+    return GET_CABAC(SIGNIFICANT_COEFF_FLAG_OFFSET + inc);
 }
 
 static av_always_inline int significant_coeff_flag_decode_0(HEVCLocalContext *lc, int c_idx, int offset)
 {
-    return GET_CABAC(elem_offset[SIGNIFICANT_COEFF_FLAG] + offset);
+    return GET_CABAC(SIGNIFICANT_COEFF_FLAG_OFFSET + offset);
 }
 
 static av_always_inline int coeff_abs_level_greater1_flag_decode(HEVCLocalContext *lc, int c_idx, int inc)
@@ -985,7 +937,7 @@ static av_always_inline int coeff_abs_level_greater1_flag_decode(HEVCLocalContex
     if (c_idx > 0)
         inc += 16;
 
-    return GET_CABAC(elem_offset[COEFF_ABS_LEVEL_GREATER1_FLAG] + inc);
+    return GET_CABAC(COEFF_ABS_LEVEL_GREATER1_FLAG_OFFSET + inc);
 }
 
 static av_always_inline int coeff_abs_level_greater2_flag_decode(HEVCLocalContext *lc, int c_idx, int inc)
@@ -993,7 +945,7 @@ static av_always_inline int coeff_abs_level_greater2_flag_decode(HEVCLocalContex
     if (c_idx > 0)
         inc += 4;
 
-    return GET_CABAC(elem_offset[COEFF_ABS_LEVEL_GREATER2_FLAG] + inc);
+    return GET_CABAC(COEFF_ABS_LEVEL_GREATER2_FLAG_OFFSET + inc);
 }
 
 static av_always_inline int coeff_abs_level_remaining_decode(HEVCLocalContext *lc, int rc_rice_param)
diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h
index c697119891..e65a6180ca 100644
--- a/libavcodec/hevcdec.h
+++ b/libavcodec/hevcdec.h
@@ -86,58 +86,6 @@ enum RPSType {
     NB_RPS_TYPE,
 };
 
-enum SyntaxElement {
-    SAO_MERGE_FLAG = 0,
-    SAO_TYPE_IDX,
-    SAO_EO_CLASS,
-    SAO_BAND_POSITION,
-    SAO_OFFSET_ABS,
-    SAO_OFFSET_SIGN,
-    END_OF_SLICE_FLAG,
-    SPLIT_CODING_UNIT_FLAG,
-    CU_TRANSQUANT_BYPASS_FLAG,
-    SKIP_FLAG,
-    CU_QP_DELTA,
-    PRED_MODE_FLAG,
-    PART_MODE,
-    PCM_FLAG,
-    PREV_INTRA_LUMA_PRED_FLAG,
-    MPM_IDX,
-    REM_INTRA_LUMA_PRED_MODE,
-    INTRA_CHROMA_PRED_MODE,
-    MERGE_FLAG,
-    MERGE_IDX,
-    INTER_PRED_IDC,
-    REF_IDX_L0,
-    REF_IDX_L1,
-    ABS_MVD_GREATER0_FLAG,
-    ABS_MVD_GREATER1_FLAG,
-    ABS_MVD_MINUS2,
-    MVD_SIGN_FLAG,
-    MVP_LX_FLAG,
-    NO_RESIDUAL_DATA_FLAG,
-    SPLIT_TRANSFORM_FLAG,
-    CBF_LUMA,
-    CBF_CB_CR,
-    TRANSFORM_SKIP_FLAG,
-    EXPLICIT_RDPCM_FLAG,
-    EXPLICIT_RDPCM_DIR_FLAG,
-    LAST_SIGNIFICANT_COEFF_X_PREFIX,
-    LAST_SIGNIFICANT_COEFF_Y_PREFIX,
-    LAST_SIGNIFICANT_COEFF_X_SUFFIX,
-    LAST_SIGNIFICANT_COEFF_Y_SUFFIX,
-    SIGNIFICANT_COEFF_GROUP_FLAG,
-    SIGNIFICANT_COEFF_FLAG,
-    COEFF_ABS_LEVEL_GREATER1_FLAG,
-    COEFF_ABS_LEVEL_GREATER2_FLAG,
-    COEFF_ABS_LEVEL_REMAINING,
-    COEFF_SIGN_FLAG,
-    LOG2_RES_SCALE_ABS,
-    RES_SCALE_SIGN_FLAG,
-    CU_CHROMA_QP_OFFSET_FLAG,
-    CU_CHROMA_QP_OFFSET_IDX,
-};
-
 enum PartMode {
     PART_2Nx2N = 0,
     PART_2NxN  = 1,
-- 
2.40.1



More information about the ffmpeg-devel mailing list