Commit 11260329 authored by Jonas Svedberg's avatar Jonas Svedberg
Browse files

removed += to avoid int to int16_t truncatio warning when we do deliberate 16bit border wrapping

parent cb27453d

lib_dec/FEC_HQ_phase_ecu.c

+4 −4
Original line number Diff line number Diff line
@@ -2121,7 +2121,7 @@ static void hq_phase_ecu( 
            }

            else

            {

                *time_offs += (int16_t)output_frame; /* EVS_MONO BE compatible, but EVS CR needed as wrap will cause burst length muting envelope instability issues */

                *time_offs = (int16_t) (*time_offs + output_frame );/* EVS_MONO BE compatible, but EVS CR needed as wrap will cause burst length muting envelope instability issues */

            }

#else

            *time_offs += output_frame;
@@ -2131,7 +2131,7 @@ static void hq_phase_ecu( 
    else

    {

#ifdef FIX_1179_USAN_PHASEECU

        *time_offs += (int16_t)output_frame; /* cast added for USAN */

        *time_offs = (int16_t)(*time_offs + output_frame) ; /* cast added for USAN, "+=" avoided as it may creat a truncation from int to int16_t  */ 

#else

        *time_offs += output_frame;

#endif
@@ -2155,7 +2155,7 @@ static void hq_phase_ecu( 
    if ( *num_p > 0 )

    {

#ifdef FIX_1179_USAN_PHASEECU 

        seed += (int16_t) ( plocs[*num_p - 1] ); /* explicit cast,  i.e. the seed value may wrap  */

        seed =  ( int16_t )( seed + plocs[*num_p - 1] ); /* explicit cast and "+="  not used,  as it "+="  may create a cast from 32 bit to 16 bit, triggering USAN   */

#else

        seed += plocs[*num_p - 1];

#endif