Re: [FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-28 Thread Paul B Mahol 
On 12/28/15, Ganesh Ajjanagadde  wrote:
> Signed-off-by: Ganesh Ajjanagadde 
> ---
>  libavfilter/af_anequalizer.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavfilter/af_anequalizer.c b/libavfilter/af_anequalizer.c
> index d7b5b6c..649c0b9 100644
> --- a/libavfilter/af_anequalizer.c
> +++ b/libavfilter/af_anequalizer.c
> @@ -391,8 +391,8 @@ static void chebyshev1_bp_filter(EqualizatorFilter *f,
>
>  epsilon = sqrt((G*G - Gb*Gb) / (Gb*Gb - G0*G0));
>  g0 = pow(G0,1.0/N);
> -alfa = pow(1.0/epsilon+ sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
> -beta = pow(G/epsilon + Gb * sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
> +alfa = pow(1.0/epsilon+ sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
> +beta = pow(G/epsilon + Gb * sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
>  a = 0.5 * (alfa - 1.0/alfa);
>  b = 0.5 * (beta - g0*g0*(1/beta));
>  tetta_b = tan(wb/2);
> --
> 2.6.4
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

both patches ok
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-28 Thread Ganesh Ajjanagadde 
On Mon, Dec 28, 2015 at 12:14 AM, Paul B Mahol  wrote:
> On 12/28/15, Ganesh Ajjanagadde  wrote:
>> Signed-off-by: Ganesh Ajjanagadde 
>> ---
>>  libavfilter/af_anequalizer.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavfilter/af_anequalizer.c b/libavfilter/af_anequalizer.c
>> index d7b5b6c..649c0b9 100644
>> --- a/libavfilter/af_anequalizer.c
>> +++ b/libavfilter/af_anequalizer.c
>> @@ -391,8 +391,8 @@ static void chebyshev1_bp_filter(EqualizatorFilter *f,
>>
>>  epsilon = sqrt((G*G - Gb*Gb) / (Gb*Gb - G0*G0));
>>  g0 = pow(G0,1.0/N);
>> -alfa = pow(1.0/epsilon+ sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
>> -beta = pow(G/epsilon + Gb * sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
>> +alfa = pow(1.0/epsilon+ sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
>> +beta = pow(G/epsilon + Gb * sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
>>  a = 0.5 * (alfa - 1.0/alfa);
>>  b = 0.5 * (beta - g0*g0*(1/beta));
>>  tetta_b = tan(wb/2);
>> --
>> 2.6.4
>>
>> ___
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
> both patches ok

thanks, please also have a look at the one removing cabs/cexp:
https://ffmpeg.org/pipermail/ffmpeg-devel/2015-December/185847.html.
Pushing later today.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-28 Thread Ganesh Ajjanagadde 
On Mon, Dec 28, 2015 at 8:00 AM, Ganesh Ajjanagadde
 wrote:
> On Mon, Dec 28, 2015 at 12:14 AM, Paul B Mahol  wrote:
>> On 12/28/15, Ganesh Ajjanagadde  wrote:
>>> Signed-off-by: Ganesh Ajjanagadde 
>>> ---
>>>  libavfilter/af_anequalizer.c | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/libavfilter/af_anequalizer.c b/libavfilter/af_anequalizer.c
>>> index d7b5b6c..649c0b9 100644
>>> --- a/libavfilter/af_anequalizer.c
>>> +++ b/libavfilter/af_anequalizer.c
>>> @@ -391,8 +391,8 @@ static void chebyshev1_bp_filter(EqualizatorFilter *f,
>>>
>>>  epsilon = sqrt((G*G - Gb*Gb) / (Gb*Gb - G0*G0));
>>>  g0 = pow(G0,1.0/N);
>>> -alfa = pow(1.0/epsilon+ sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
>>> -beta = pow(G/epsilon + Gb * sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
>>> +alfa = pow(1.0/epsilon+ sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
>>> +beta = pow(G/epsilon + Gb * sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
>>>  a = 0.5 * (alfa - 1.0/alfa);
>>>  b = 0.5 * (beta - g0*g0*(1/beta));
>>>  tetta_b = tan(wb/2);
>>> --
>>> 2.6.4
>>>
>>> ___
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel@ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>
>>
>> both patches ok
>
> thanks, please also have a look at the one removing cabs/cexp:
> https://ffmpeg.org/pipermail/ffmpeg-devel/2015-December/185847.html.
> Pushing later today.

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

[FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-27 Thread Ganesh Ajjanagadde 
Signed-off-by: Ganesh Ajjanagadde 
---
 libavfilter/af_anequalizer.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavfilter/af_anequalizer.c b/libavfilter/af_anequalizer.c
index d7b5b6c..649c0b9 100644
--- a/libavfilter/af_anequalizer.c
+++ b/libavfilter/af_anequalizer.c
@@ -391,8 +391,8 @@ static void chebyshev1_bp_filter(EqualizatorFilter *f,
 
 epsilon = sqrt((G*G - Gb*Gb) / (Gb*Gb - G0*G0));
 g0 = pow(G0,1.0/N);
-alfa = pow(1.0/epsilon+ sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
-beta = pow(G/epsilon + Gb * sqrt(1 + pow(epsilon,-2.0)), 1.0/N);
+alfa = pow(1.0/epsilon+ sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
+beta = pow(G/epsilon + Gb * sqrt(1 + 1/(epsilon*epsilon)), 1.0/N);
 a = 0.5 * (alfa - 1.0/alfa);
 b = 0.5 * (beta - g0*g0*(1/beta));
 tetta_b = tan(wb/2);
-- 
2.6.4

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