From d363000eaaa239c43cf1400199aa72b845aa73e1 Mon Sep 17 00:00:00 2001 From: =?utf8?q?V=C3=ADctor=20Manuel=20J=C3=A1quez=20Leal?= Date: Fri, 3 Apr 2015 20:27:24 +0200 Subject: [PATCH] encoder: mpeg2: use fabsf() instead of abs() The member value in frame_rate_tab is float, the result of the abs() function should be float too. But abs() only manages integers. This patch replaces abs() with fabsf() to handle correctly the possible floats values. https://bugzilla.gnome.org/show_bug.cgi?id=747312 --- gst-libs/gst/vaapi/gstvaapiencoder_mpeg2.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gst-libs/gst/vaapi/gstvaapiencoder_mpeg2.c b/gst-libs/gst/vaapi/gstvaapiencoder_mpeg2.c index 4536365..9747ae1 100644 --- a/gst-libs/gst/vaapi/gstvaapiencoder_mpeg2.c +++ b/gst-libs/gst/vaapi/gstvaapiencoder_mpeg2.c @@ -22,6 +22,7 @@ */ #include "sysdeps.h" +#include #include #include #include @@ -887,9 +888,9 @@ find_frame_rate_code (const VAEncSequenceParameterBufferMPEG2 * seq_param) for (i = 0; i < sizeof (frame_rate_tab) / sizeof (frame_rate_tab[0]); i++) { - if (abs (1000 * frame_rate_tab[i].value - 1000 * frame_rate_value) < delta) { + if (fabsf (1000 * frame_rate_tab[i].value - 1000 * frame_rate_value) < delta) { code = frame_rate_tab[i].code; - delta = abs (1000 * frame_rate_tab[i].value - 1000 * frame_rate_value); + delta = fabsf (1000 * frame_rate_tab[i].value - 1000 * frame_rate_value); } } return code; -- 2.7.4