On 2015-07-09 18:55, Paul B Mahol wrote:
> +    { "n", "enable channels coupling", OFFSET(channels_coupled), 
> AV_OPT_TYPE_INT, {.i64 = 1}, 0, 1, FLAGS },

This should be "enable channel coupling"

> +static const double s_pi = 
> 3.1415926535897932384626433832795028841971693993751058209749445923078164062862089986280348253421170679;

Why not use the standard M_PI define?

> +static void init_gaussian_filter(DynamicAudioNormalizerContext *s)
> +{
> +    double total_weight = 0.0;
> +    const double sigma = (((s->filter_size / 2.0) - 1.0) / 3.0) + (1.0 / 
> 3.0);
> +    double adjust;
> +    int i;
> +
> +    // Pre-computer constants

"Pre-compute constants"


Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to