diff mbox

[FFmpeg-devel,07/11] avformat/mxf: use FFABS to instead of abs

Message ID 20171115074439.25451-1-lq@chinaffmpeg.org
State Accepted
Commit cb259467d1ea0320b4150180f0d94b01c5b7e0ed
Headers show

Commit Message

Liu Steven Nov. 15, 2017, 7:44 a.m. UTC 
Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
---
 libavformat/mxf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Niedermayer Nov. 16, 2017, 2:59 p.m. UTC | #1 
On Wed, Nov 15, 2017 at 03:44:39PM +0800, Steven Liu wrote:
> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
> ---
>  libavformat/mxf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM

thx

[...]
Steven Liu Nov. 20, 2017, 3:08 a.m. UTC | #2 
2017-11-16 22:59 GMT+08:00 Michael Niedermayer <michael@niedermayer.cc>:
> On Wed, Nov 15, 2017 at 03:44:39PM +0800, Steven Liu wrote:
>> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
>> ---
>>  libavformat/mxf.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> LGTM
>
Pushed

Thx
> thx
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> In a rich man's house there is no place to spit but his face.
> -- Diogenes of Sinope
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
diff mbox

Patch

diff --git a/libavformat/mxf.c b/libavformat/mxf.c
index bfc3218b81..2e23b5d739 100644
--- a/libavformat/mxf.c
+++ b/libavformat/mxf.c
@@ -155,7 +155,7 @@  const MXFSamplesPerFrame *ff_mxf_get_samples_per_frame(AVFormatContext *s,
     int idx = av_find_nearest_q_idx(time_base, mxf_time_base);
     AVRational diff = av_sub_q(time_base, mxf_time_base[idx]);
 
-    diff.num = abs(diff.num);
+    diff.num = FFABS(diff.num);
 
     if (av_cmp_q(diff, (AVRational){1, 1000}) >= 0)
         return NULL;