Commit 1153441f authored by multrus's avatar multrus
Browse files

[cleanup] accept FIX_1494_SET_SPLITBFI_UNUSED

parent ead89d58

apps/renderer.c

+0 −36
Original line number Diff line number Diff line
@@ -167,9 +167,6 @@ typedef struct 
    int16_t numInMetadataFiles;

    char outMetadataFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

    char headRotationFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    char splitRendBFIFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

#endif

    char referenceVectorFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

    char referenceRotationFilePath[RENDERER_MAX_CLI_ARG_LENGTH];

    char externalOrientationFilePath[RENDERER_MAX_CLI_ARG_LENGTH];
@@ -220,9 +217,6 @@ typedef enum 
    CmdLnOptionId_listFormats,

    CmdLnOptionId_inputGain,

    CmdLnOptionId_outputMetadata,

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    CmdLnOptionId_SplitRendBFIFile,

#endif

    CmdLnOptionId_referenceVectorFile,

    CmdLnOptionId_exteriorOrientationFile,

    CmdLnOptionId_framing,
@@ -281,14 +275,6 @@ static const CmdLnParser_Option cliOptions[] = { 
        .matchShort = "om",

        .description = "coded metadata file for BINAURAL_SPLIT_PCM output mode",

    },

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    {

        .id = CmdLnOptionId_SplitRendBFIFile,

        .match = "post_rend_bfi_file",

        .matchShort = "prbfi",

        .description = "Split rendering option: bfi file",

    },

#endif

    {

        .id = CmdLnOptionId_refRotFile,

        .match = "reference_rotation_file",
@@ -764,9 +750,6 @@ int main( 
    IVAS_CLDFB_FILTER_BANK_HANDLE cldfbAna[RENDERER_MAX_INPUT_CHANNELS];

    IVAS_CLDFB_FILTER_BANK_HANDLE cldfbSyn[RENDERER_MAX_INPUT_CHANNELS];

    int16_t cldfb_in_flag, CLDFBframeSize_smpls;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    SplitRendBFIFileReader *splitRendBFIReader = NULL;

#endif

    Vector3PairFileReader *referenceVectorReader = NULL;

    hrtfFileReader *hrtfFileReader = NULL;

    IVAS_DEC_HRTF_CREND_HANDLE *hHrtfCrend = NULL;
@@ -896,13 +879,6 @@ int main( 
        }

    }



#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    if ( !isEmptyString( args.splitRendBFIFilePath ) )

    {

        convert_backslash( args.splitRendBFIFilePath );

        SplitRendBFIFileReader_open( args.splitRendBFIFilePath, &splitRendBFIReader );

    }

#endif

    if ( !isEmptyString( args.externalOrientationFilePath ) )

    {

        if ( RotationFileReader_open( args.externalOrientationFilePath, &externalOrientationFileReader ) != IVAS_ERR_OK )
@@ -2184,9 +2160,6 @@ cleanup: 
    }



    split_rend_reader_writer_close( &hSplitRendFileReadWrite );

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    SplitRendBFIFileReader_close( &splitRendBFIReader );

#endif



    for ( i = 0; i < RENDERER_MAX_MC_INPUTS; ++i )

    {
@@ -2798,9 +2771,6 @@ static CmdlnArgs defaultArgs( 


    clearString( args.headRotationFilePath );

    clearString( args.outMetadataFilePath );

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    clearString( args.splitRendBFIFilePath );

#endif

    clearString( args.referenceVectorFilePath );

    clearString( args.referenceRotationFilePath );

    clearString( args.customHrtfFilePath );
@@ -2907,12 +2877,6 @@ static void parseOption( 
            assert( numOptionValues == 1 );

            strncpy( args->outMetadataFilePath, optionValues[0], RENDERER_MAX_CLI_ARG_LENGTH - 1 );

            break;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

        case CmdLnOptionId_SplitRendBFIFile:

            assert( numOptionValues == 1 );

            strncpy( args->splitRendBFIFilePath, optionValues[0], RENDERER_MAX_CLI_ARG_LENGTH - 1 );

            break;

#endif

        case CmdLnOptionId_referenceVectorFile:

            assert( numOptionValues == 1 );

            strncpy( args->referenceVectorFilePath, optionValues[0], RENDERER_MAX_CLI_ARG_LENGTH - 1 );

lib_com/options.h

+0 −1
Original line number Diff line number Diff line
@@ -87,7 +87,6 @@ 
#define NONBE_1122_KEEP_EVS_MODE_UNCHANGED              /* FhG: Disables fix for issue 1122 in EVS mode to keep BE tests green. This switch should be removed once the 1122 fix is added to EVS via a CR.  */

#define FIX_1904_HARM_GSC_ENC                           /* VA: #1904 Harmonization of EVS and IVAS GSC code */

#define FIX_1478_UNINIT_ON_BFI                          /* VA: Fix issue 1478 where a vector is partly un-initialized during bfi */

#define FIX_1494_SET_SPLITBFI_UNUSED                    /* Dolby: Fix issue 1494, remove unused function setting BFI flag in ISAR renderer */

#define FIX_1479_MSAN_SPAR_UNINITIALIZED_VALUE          /* Dolby: Fix for issue 1479, MSAN error due to uninitialized value in SPAR */

#define HARM_LSF_ENC                                    /* VA: basop issue 2301: Remove duplicated code in LSF quantizer */

#define CLEANUP_HQ_CORE                                 /* VA: basop issue 2300: Remove unused code in core-coder: HQ core */

lib_rend/lib_rend.h

+0 −5
Original line number Diff line number Diff line
@@ -344,11 +344,6 @@ ivas_error IVAS_REND_SetReferenceVector( 
    const IVAS_VECTOR3 refPos                       /* i  : Reference position                                  */

);



#ifndef FIX_1494_SET_SPLITBFI_UNUSED

ivas_error IVAS_REND_SetSplitRendBFI(

    IVAS_REND_HANDLE hIvasRend,

    const Word16 bfi);

#endif



ivas_error IVAS_REND_SetExternalOrientation(

    IVAS_REND_HANDLE hIvasRend,                     /* i/o: Renderer handle                                                 */

lib_rend/lib_rend_fx.c

+0 −28
Original line number Diff line number Diff line
@@ -94,9 +94,6 @@ typedef struct 
    const EFAP_WRAPPER *pEfapOutWrapper;

    IVAS_REND_HeadRotData *pHeadRotData; // for now removing the const qualifier TODO: will modify later

    const RENDER_CONFIG_HANDLE *hhRendererConfig;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    const Word16 *pSplitRendBFI;

#endif

    const SPLIT_REND_WRAPPER *pSplitRendWrapper;

    const COMBINED_ORIENTATION_HANDLE *pCombinedOrientationData;

    const IVAS_DefaultReverbSize *pSelectedRoomReverbSize;
@@ -218,9 +215,6 @@ struct IVAS_REND 
    EFAP_WRAPPER efapOutWrapper;

    IVAS_LSSETUP_CUSTOM_STRUCT customLsOut;



#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    Word16 splitRendBFI;

#endif

    SPLIT_REND_WRAPPER *splitRendWrapper;

    IVAS_REND_AudioBuffer splitRendEncBuffer;

    IVAS_REND_HeadRotData headRotData;
@@ -1369,9 +1363,6 @@ static rendering_context getRendCtx( 
    ctx.pEfapOutWrapper = &hIvasRend->efapOutWrapper;

    ctx.pHeadRotData = &hIvasRend->headRotData;

    ctx.hhRendererConfig = &hIvasRend->hRendererConfig;

#ifndef FIX_1494_SET_SPLITBFI_UNUSED

    ctx.pSplitRendBFI = &hIvasRend->splitRendBFI;

#endif

    ctx.pSplitRendWrapper = hIvasRend->splitRendWrapper;

    ctx.pCombinedOrientationData = &hIvasRend->hCombinedOrientationData;

    ctx.pSelectedRoomReverbSize = &hIvasRend->selectedRoomReverbSize;
@@ -5729,25 +5720,6 @@ ivas_error IVAS_REND_DisableHeadRotation( 
    return IVAS_ERR_OK;

}



#ifndef FIX_1494_SET_SPLITBFI_UNUSED



/*-------------------------------------------------------------------*

 * IVAS_REND_SetSplitRendBFI()

 *

 *

 *-------------------------------------------------------------------*/



ivas_error IVAS_REND_SetSplitRendBFI(

    IVAS_REND_HANDLE hIvasRend, /* i/o: Renderer handle      */

    const Word16 bfi            /* i  : Bad Frame Indicator  */

)

{

    hIvasRend->splitRendBFI = bfi;



    return IVAS_ERR_OK;

}



#endif





/*-------------------------------------------------------------------*