[FFmpeg-devel] [PATCH 06/15] lavc/aacpsy: replace pow(10, x) by exp10(x)

Rostislav Pehlivanov atomnuker at gmail.com
Wed Dec 23 21:36:42 CET 2015


On Wed, 2015-12-23 at 10:47 -0800, Ganesh Ajjanagadde wrote:
> exp10, recently introduced, is superior for the purpose.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  libavcodec/aacpsy.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/libavcodec/aacpsy.c b/libavcodec/aacpsy.c
> index 71eeb3e..163ee43 100644
> --- a/libavcodec/aacpsy.c
> +++ b/libavcodec/aacpsy.c
> @@ -349,10 +349,10 @@ static av_cold int psy_3gpp_init(FFPsyContext
> *ctx) {
>          for (g = 0; g < ctx->num_bands[j] - 1; g++) {
>              AacPsyCoeffs *coeff = &coeffs[g];
>              float bark_width = coeffs[g+1].barks - coeffs->barks;
> -            coeff->spread_low[0] = pow(10.0, -bark_width *
> PSY_3GPP_THR_SPREAD_LOW);
> -            coeff->spread_hi [0] = pow(10.0, -bark_width *
> PSY_3GPP_THR_SPREAD_HI);
> -            coeff->spread_low[1] = pow(10.0, -bark_width *
> en_spread_low);
> -            coeff->spread_hi [1] = pow(10.0, -bark_width *
> en_spread_hi);
> +            coeff->spread_low[0] = exp10(-bark_width *
> PSY_3GPP_THR_SPREAD_LOW);
> +            coeff->spread_hi [0] = exp10(-bark_width *
> PSY_3GPP_THR_SPREAD_HI);
> +            coeff->spread_low[1] = exp10(-bark_width *
> en_spread_low);
> +            coeff->spread_hi [1] = exp10(-bark_width *
> en_spread_hi);
>              pe_min = bark_pe * bark_width;
>              minsnr = exp2(pe_min / band_sizes[g]) - 1.5f;
>              coeff->min_snr = av_clipf(1.0f / minsnr, PSY_SNR_25DB,
> PSY_SNR_1DB);


LGTM

Thanks


More information about the ffmpeg-devel mailing list