Commit 29b183a6 authored by Jan Kiene's avatar Jan Kiene
Browse files

remove obsolete comments after they were clarified

parent 117cfdb2

lib_dec/fd_cng_dec_fx.c

+2 −4
Original line number Diff line number Diff line
@@ -2940,7 +2940,6 @@ void generate_comfort_noise_dec_ivas_fx( 
    temp = 0;

    move16();

#ifdef NONBE_MDCT_ST_DTX_FIX_SUBOPT_SPATIAL_CNG

    // TODO: float is "sqrt(1 - coherence)". Here MAX_16 is used for 1. coherence_fx is in Q15. Why isn't ONE_IN_Q15 used here? (which is different from MAX_16)

    c2 = Sqrt16( sub( MAX_16, hFdCngCom->coherence_fx[0] ), &temp ); /*Q15 - temp*/

#else

    c2 = Sqrt16( sub( MAX_16, hFdCngCom->coherence_fx ), &temp ); /*Q15 - temp*/
@@ -3050,7 +3049,6 @@ void generate_comfort_noise_dec_ivas_fx( 
            c1 = shl( c1, temp );                             /*Q15*/

            temp = 0;

            move16();

            // TODO: float is "sqrt(1 - coherence)". Here MAX_16 is used for 1. coherence_fx is in Q15. Why isn't ONE_IN_Q15 used here? (which is different from MAX_16)

            c2 = Sqrt16( sub( MAX_16, hFdCngCom->coherence_fx[b] ), &temp ); /*Q15 - temp*/

            c2 = shl( c2, temp );                                            /*Q15*/

lib_dec/ivas_ism_metadata_dec_fx.c

+0 −1
Original line number Diff line number Diff line
@@ -1251,7 +1251,6 @@ void ivas_ism_metadata_sid_dec_fx( 
#ifdef NONBE_MDCT_ST_DTX_FIX_SUBOPT_SPATIAL_CNG

                    FOR( i = 0; i < MDCT_ST_DTX_NUM_COHERENCE_BANDS; i++ )

                    {

                        // TODO: why not ONE_IN_Q15 ? (value there is different)

                        hSCE[ch]->hCoreCoder[0]->hFdCngDec->hFdCngCom->coherence_fx[i] = 32767;

                    }