[FFmpeg-cvslog] avfilter/af_dynaudnorm: use M_PI
Ganesh Ajjanagadde
git at videolan.org
Fri Nov 20 14:33:37 CET 2015
ffmpeg | branch: master | Ganesh Ajjanagadde <gajjanagadde at gmail.com> | Tue Nov 10 22:26:25 2015 -0500| [7769e6602d2d32144a3e1ca3fa7b06cca5d1c988] | committer: Ganesh Ajjanagadde
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 <onemda at gmail.com>
Reviewed-by: Nicolas George <george at nsup.org>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=7769e6602d2d32144a3e1ca3fa7b06cca5d1c988
---
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 8f0c2d0..62a2653 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
More information about the ffmpeg-cvslog
mailing list