Re: [FFmpeg-devel] [PATCH 3/9] avfilter/af_stereotools: use lrint instead of floor hack

[Paul B Mahol]

On 12/2/15, Paul B Mahol  wrote:
> On 12/2/15, Ganesh Ajjanagadde  wrote:
>> Signed-off-by: Ganesh Ajjanagadde 
>> ---
>>  libavfilter/af_stereotools.c | 8 
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/libavfilter/af_stereotools.c b/libavfilter/af_stereotools.c
>> index a22efb0..e878824 100644
>> --- a/libavfilter/af_stereotools.c
>> +++ b/libavfilter/af_stereotools.c
>> @@ -139,10 +139,10 @@ static int filter_frame(AVFilterLink *inlink,
>> AVFrame
>> *in)
>>  const double sc_level = s->sc_level;
>>  const double delay = s->delay;
>>  const int length = s->length;
>> -const int mute_l = floor(s->mute_l + 0.5);
>> -const int mute_r = floor(s->mute_r + 0.5);
>> -const int phase_l = floor(s->phase_l + 0.5);
>> -const int phase_r = floor(s->phase_r + 0.5);
>> +const int mute_l = lrint(s->mute_l);
>> +const int mute_r = lrint(s->mute_r);
>> +const int phase_l = lrint(s->phase_l);
>> +const int phase_r = lrint(s->phase_r);
>>  double *buffer = s->buffer;
>>  AVFrame *out;
>>  double *dst;
>> --
>> 2.6.2
>>
>> ___
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
> ok
>

Actually lrint is not needed. Fixed differently.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/9] avfilter/af_stereotools: use lrint instead of floor hack

[Ganesh Ajjanagadde]

On Wed, Dec 2, 2015 at 6:03 AM, Paul B Mahol  wrote:
> On 12/2/15, Paul B Mahol  wrote:
>> On 12/2/15, Ganesh Ajjanagadde  wrote:
>>> Signed-off-by: Ganesh Ajjanagadde 
>>> ---
>>>  libavfilter/af_stereotools.c | 8 
>>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/libavfilter/af_stereotools.c b/libavfilter/af_stereotools.c
>>> index a22efb0..e878824 100644
>>> --- a/libavfilter/af_stereotools.c
>>> +++ b/libavfilter/af_stereotools.c
>>> @@ -139,10 +139,10 @@ static int filter_frame(AVFilterLink *inlink,
>>> AVFrame
>>> *in)
>>>  const double sc_level = s->sc_level;
>>>  const double delay = s->delay;
>>>  const int length = s->length;
>>> -const int mute_l = floor(s->mute_l + 0.5);
>>> -const int mute_r = floor(s->mute_r + 0.5);
>>> -const int phase_l = floor(s->phase_l + 0.5);
>>> -const int phase_r = floor(s->phase_r + 0.5);
>>> +const int mute_l = lrint(s->mute_l);
>>> +const int mute_r = lrint(s->mute_r);
>>> +const int phase_l = lrint(s->phase_l);
>>> +const int phase_r = lrint(s->phase_r);
>>>  double *buffer = s->buffer;
>>>  AVFrame *out;
>>>  double *dst;
>>> --
>>> 2.6.2
>>>
>>> ___
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel@ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>>
>>
>> ok
>>
>
> Actually lrint is not needed. Fixed differently.

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

[FFmpeg-devel] [PATCH 3/9] avfilter/af_stereotools: use lrint instead of floor hack

[Ganesh Ajjanagadde]

Signed-off-by: Ganesh Ajjanagadde 
---
 libavfilter/af_stereotools.c | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libavfilter/af_stereotools.c b/libavfilter/af_stereotools.c
index a22efb0..e878824 100644
--- a/libavfilter/af_stereotools.c
+++ b/libavfilter/af_stereotools.c
@@ -139,10 +139,10 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
 const double sc_level = s->sc_level;
 const double delay = s->delay;
 const int length = s->length;
-const int mute_l = floor(s->mute_l + 0.5);
-const int mute_r = floor(s->mute_r + 0.5);
-const int phase_l = floor(s->phase_l + 0.5);
-const int phase_r = floor(s->phase_r + 0.5);
+const int mute_l = lrint(s->mute_l);
+const int mute_r = lrint(s->mute_r);
+const int phase_l = lrint(s->phase_l);
+const int phase_r = lrint(s->phase_r);
 double *buffer = s->buffer;
 AVFrame *out;
 double *dst;
-- 
2.6.2

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

Re: [FFmpeg-devel] [PATCH 3/9] avfilter/af_stereotools: use lrint instead of floor hack

[Paul B Mahol]

On 12/2/15, Ganesh Ajjanagadde  wrote:
> Signed-off-by: Ganesh Ajjanagadde 
> ---
>  libavfilter/af_stereotools.c | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/libavfilter/af_stereotools.c b/libavfilter/af_stereotools.c
> index a22efb0..e878824 100644
> --- a/libavfilter/af_stereotools.c
> +++ b/libavfilter/af_stereotools.c
> @@ -139,10 +139,10 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> *in)
>  const double sc_level = s->sc_level;
>  const double delay = s->delay;
>  const int length = s->length;
> -const int mute_l = floor(s->mute_l + 0.5);
> -const int mute_r = floor(s->mute_r + 0.5);
> -const int phase_l = floor(s->phase_l + 0.5);
> -const int phase_r = floor(s->phase_r + 0.5);
> +const int mute_l = lrint(s->mute_l);
> +const int mute_r = lrint(s->mute_r);
> +const int phase_l = lrint(s->phase_l);
> +const int phase_r = lrint(s->phase_r);
>  double *buffer = s->buffer;
>  AVFrame *out;
>  double *dst;
> --
> 2.6.2
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

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