Commit 999850a6 authored by Archit Tamarapu's avatar Archit Tamarapu
Browse files

[cleanup] accept FIX_993_REMOVE_SBA_GET_ORDER

parent aab1bf4c
Loading
Loading
Loading
Loading
+0 −8
Original line number Diff line number Diff line
@@ -3506,14 +3506,6 @@ ivas_error ivas_cldfb_dec_reconfig(
    int16_t numCldfbAnalyses_old,                               /* i  : number of CLDFB analysis instances in previous frame  */
    const int16_t numCldfbSyntheses_old                         /* i  : number of CLDFB synthesis instances in previous frame */
);
#ifndef FIX_993_REMOVE_SBA_GET_ORDER
/*! r: Ambisonic (SBA) order */
int16_t ivas_sba_get_order(
    const int16_t nb_channels,                                  /* i  : Number of ambisonic channels            */
    const int16_t sba_planar                                    /* i  : SBA planar flag                         */
);

#endif
/*! r: Ambisonic (SBA) order used for analysis and coding */
int16_t ivas_sba_get_analysis_order(
    const int32_t ivas_total_brate,                             /* i  : IVAS total bitrate                      */
+0 −47
Original line number Diff line number Diff line
@@ -64,23 +64,7 @@ void ivas_sba_config(
    int16_t *element_mode          /* o  : element mode of the core coder           */
)
{
#ifndef FIX_993_REMOVE_SBA_GET_ORDER
    if ( ( sba_order < 0 ) && ( nb_channels < 0 ) )
    {
        assert( 0 && "Either order or number of channels must be positive" );
    }
    else if ( sba_order < 0 )
    {
        sba_order = ivas_sba_get_order( nb_channels, sba_planar );
    }
    else if ( nb_channels < 0 )
    {
        nb_channels = ivas_sba_get_nchan( sba_order, sba_planar );
    }
    else
#else
    if ( nb_channels > 0 )
#endif
    {
        if ( sba_planar )
        {
@@ -121,37 +105,6 @@ void ivas_sba_config(
    return;
}

#ifndef FIX_993_REMOVE_SBA_GET_ORDER
/*-------------------------------------------------------------------*
 * ivas_sba_get_order()
 *
 * Get Ambisonic order from number of ambisonic channels
 *-------------------------------------------------------------------*/

/*! r: Ambisonic (SBA) order */
int16_t ivas_sba_get_order(
    const int16_t nb_channels, /* i  : Number of ambisonic channels     */
    const int16_t sba_planar   /* i  : SBA Planar flag                  */
)
{
    int16_t sba_order;

    if ( sba_planar )
    {
        sba_order = ( nb_channels - 1 ) / 2;
        assert( ( 2 * sba_order + 1 == nb_channels ) && "Number of channels not supported in Planar SBA!" );
    }
    else
    {
        sba_order = (int16_t) sqrtf( (float) nb_channels ) - 1;
        assert( ( ( sba_order + 1 ) * ( sba_order + 1 ) == nb_channels ) && "Number of channels not supported in SBA!" );
    }

    assert( ( sba_order <= 3 ) && "Error: SBA order must be <= 3!" );

    return ( sba_order );
}
#endif

/*-------------------------------------------------------------------*
 * ivas_sba_get_analysis_order()
+0 −1
Original line number Diff line number Diff line
@@ -180,7 +180,6 @@
#ifdef NONBE_FIX_1058_DECODER_ERROR_WITH_REVERB_ROOM
#define NONBE_FIX_AVG_IAC_CLDFB_REVERB
#endif
#define FIX_993_REMOVE_SBA_GET_ORDER                    /* VA: issue 993: remove unused function ivas_sba_get_order() */
#define FIX_960_SYN_OUTPUT                              /* VA: issue 960: unused function syn_output() is removed */

#define USE_NEW_HRTF_BINARY_FILE_FORMAT                 /* Orange: to activate when decided to change the hrtf binary file format */