Skip to content

ivas_jbm_dec_get_md_map_even_spacing(): Unused parameter default_len

The function ivas_jbm_dec_get_md_map_even_spacing() has a basically unused argument default_len, which is only used in an assert(), but for nothing else:

void ivas_jbm_dec_get_md_map_even_spacing(
    const int16_t default_len,  /* i  : default frame length in metadata slots         */
    const int16_t len,          /* i  : length of the modfied frames in metadata slots */
    const int16_t subframe_len, /* i  : default length of a subframe                   */
    const int16_t offset,       /* i  : current read offset into the md buffer         */
    const int16_t buf_len,      /* i  : length of the metadata buffer                  */
    int16_t *map                /* o  : metadata index map                             */
)
{
    int16_t map_idx, sf_idx, sf_length, increment, subframes_written;
    float decimal, decimal_sum, eps;
    int16_t subframe_map_length[MAX_PARAM_SPATIAL_SUBFRAMES];
#ifdef DEBUGGING
    assert( default_len % 2 == 0 );
#endif

If DEBUGGING is inactive, it will trigger a compiler warning. Could this function argument be removed?