From 7769e6602d2d32144a3e1ca3fa7b06cca5d1c988 Mon Sep 17 00:00:00 2001 From: Ganesh Ajjanagadde Date: Tue, 10 Nov 2015 22:26:25 -0500 Subject: [PATCH] avfilter/af_dynaudnorm: use M_PI The ad-hoc pi constant has a ludicrous number of digits that offer no value whatsoever. M_PI is more consistent and readable across the codebase. Reviewed-by: Paul B Mahol Reviewed-by: Nicolas George Signed-off-by: Ganesh Ajjanagadde --- libavfilter/af_dynaudnorm.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavfilter/af_dynaudnorm.c b/libavfilter/af_dynaudnorm.c index 8f0c2d0858..62a2653ec5 100644 --- a/libavfilter/af_dynaudnorm.c +++ b/libavfilter/af_dynaudnorm.c @@ -227,8 +227,6 @@ static int cqueue_pop(cqueue *q) return 0; } -static const double s_pi = 3.1415926535897932384626433832795028841971693993751058209749445923078164062862089986280348253421170679; - static void init_gaussian_filter(DynamicAudioNormalizerContext *s) { double total_weight = 0.0; @@ -238,7 +236,7 @@ static void init_gaussian_filter(DynamicAudioNormalizerContext *s) // Pre-compute constants const int offset = s->filter_size / 2; - const double c1 = 1.0 / (sigma * sqrt(2.0 * s_pi)); + const double c1 = 1.0 / (sigma * sqrt(2.0 * M_PI)); const double c2 = 2.0 * pow(sigma, 2.0); // Compute weights