On Fri, Dec 18, 2015 at 9:45 PM, Ganesh Ajjanagadde
<gajjanaga...@gmail.com> wrote:
> lrint is more accurate, and is not slower on non-broken libm's. Thus this
> represents a Pareto improvement.
>
> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com>
> ---
> libavfilter/vf_colorlevels.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/libavfilter/vf_colorlevels.c b/libavfilter/vf_colorlevels.c
> index cb3314b..dedbe30 100644
> --- a/libavfilter/vf_colorlevels.c
> +++ b/libavfilter/vf_colorlevels.c
> @@ -132,10 +132,10 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> *in)
> const uint8_t offset = s->rgba_map[i];
> const uint8_t *srcrow = in->data[0];
> uint8_t *dstrow = out->data[0];
> - int imin = round(r->in_min * UINT8_MAX);
> - int imax = round(r->in_max * UINT8_MAX);
> - int omin = round(r->out_min * UINT8_MAX);
> - int omax = round(r->out_max * UINT8_MAX);
> + int imin = lrint(r->in_min * UINT8_MAX);
> + int imax = lrint(r->in_max * UINT8_MAX);
> + int omin = lrint(r->out_min * UINT8_MAX);
> + int omax = lrint(r->out_max * UINT8_MAX);
> double coeff;
>
> if (imin < 0) {
> @@ -179,10 +179,10 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> *in)
> const uint8_t offset = s->rgba_map[i];
> const uint8_t *srcrow = in->data[0];
> uint8_t *dstrow = out->data[0];
> - int imin = round(r->in_min * UINT16_MAX);
> - int imax = round(r->in_max * UINT16_MAX);
> - int omin = round(r->out_min * UINT16_MAX);
> - int omax = round(r->out_max * UINT16_MAX);
> + int imin = lrint(r->in_min * UINT16_MAX);
> + int imax = lrint(r->in_max * UINT16_MAX);
> + int omin = lrint(r->out_min * UINT16_MAX);
> + int omax = lrint(r->out_max * UINT16_MAX);
> double coeff;
>
> if (imin < 0) {
> --
> 2.6.4
>
pushed
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
