Hi!

Attached patch fixes a useless warning when compiling with clang on arm.

Please comment, Carl Eugen
From b2773d1ca9473380b99400c2bbd5a6729622f51a Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <ceffm...@gmail.com>
Date: Mon, 17 Dec 2018 02:12:13 +0100
Subject: [PATCH] lsws/utils: Split "emms_c();" in two lines.

Silences a clang warning when not compiling for x86:
libswscale/utils.c:345:13: warning: while loop has empty body
---
 libswscale/utils.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libswscale/utils.c b/libswscale/utils.c
index df68bcc..de64694 100644
--- a/libswscale/utils.c
+++ b/libswscale/utils.c
@@ -342,7 +342,8 @@ static av_cold int initFilter(int16_t **outFilter, int32_t **filterPos,
     const int64_t fone = 1LL << (54 - FFMIN(av_log2(srcW/dstW), 8));
     int ret            = -1;
 
-    emms_c(); // FIXME should not be required but IS (even for non-MMX versions)
+    emms_c() // FIXME should not be required but IS (even for non-MMX versions)
+        ;
 
     // NOTE: the +3 is for the MMX(+1) / SSE(+3) scaler which reads over the end
     FF_ALLOC_ARRAY_OR_GOTO(NULL, *filterPos, (dstW + 3), sizeof(**filterPos), fail);
-- 
1.7.10.4

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

Reply via email to