Commit 8757e52e authored by Archit Tamarapu's avatar Archit Tamarapu
Browse files

[fix] incorrect condition for position change check

parent 7fb489c7

lib_rend/lib_rend.c

+5 −5
Original line number Diff line number Diff line
@@ -1255,7 +1255,7 @@ static int8_t checkObjectPositionChanged( 
    IVAS_REND_AudioObjectPosition *currentPos,

    IVAS_REND_AudioObjectPosition *previousPos )

{

    return ( fabs( currentPos->azimuth - previousPos->azimuth ) < EPSILON &&

    return !( fabs( currentPos->azimuth - previousPos->azimuth ) < EPSILON &&

              fabs( currentPos->elevation - previousPos->elevation ) < EPSILON );

}

#endif
@@ -6290,7 +6290,7 @@ static ivas_error renderIsmToBinauralRoom( 
        rotatedPos.elevation = ismInput->currentPos.elevation;

    }



    position_changed = !ismInput->firstFrameRendered &&

    position_changed = !ismInput->firstFrameRendered ||

                       checkObjectPositionChanged( &rotatedPos, &rotatedPosPrev );



    /* compute gains only if position changed */
@@ -6431,7 +6431,7 @@ static ivas_error renderIsmToMc( 
    push_wmops( "renderIsmToMc" );



#ifdef FIX_518_ISM_BRIR_EXTREND

    position_changed = !ismInput->firstFrameRendered &&

    position_changed = !ismInput->firstFrameRendered ||

                       checkObjectPositionChanged( &ismInput->currentPos, &ismInput->previousPos );

#else

    /* TODO(sgi): Possible optimization: less processing needed if position didn't change */
@@ -6567,7 +6567,7 @@ static ivas_error renderIsmToSba( 
    else

    {

#else

    position_changed = !ismInput->firstFrameRendered &&

    position_changed = !ismInput->firstFrameRendered ||

                       checkObjectPositionChanged( &ismInput->currentPos, &ismInput->previousPos );



    if ( position_changed )