diff mbox series

[FFmpeg-devel] swscale/x86/rgb_2_rgb: Add opaque pointer to missed definitions of ff_nv12ToUV

Message ID 20231114132638.22948-1-parona@protonmail.com
State Accepted
Commit e5ce473040ae3e78da80b8a7c6ee35778b21d53c
Headers show
Series [FFmpeg-devel] swscale/x86/rgb_2_rgb: Add opaque pointer to missed definitions of ff_nv12ToUV | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Alfred Wingate Nov. 14, 2023, 1:26 p.m. UTC
Opaque parameters were previously added to the original definition of
ff_nv12ToUV, leading to gcc noticing a type mismatch with -Wlto-type-mismatch.

https://git.ffmpeg.org/gitweb/ffmpeg.git/commit/f2de911818fbd7e73343803626b697fd0c968121
https://bugs.gentoo.org/907484

Signed-off-by: Alfred Wingate <parona@protonmail.com>
---
 libswscale/x86/rgb2rgb_template.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Anton Khirnov Nov. 30, 2023, 12:01 p.m. UTC | #1
Quoting Alfred Wingate via ffmpeg-devel (2023-11-14 14:26:47)
> Opaque parameters were previously added to the original definition of
> ff_nv12ToUV, leading to gcc noticing a type mismatch with -Wlto-type-mismatch.
> 
> https://git.ffmpeg.org/gitweb/ffmpeg.git/commit/f2de911818fbd7e73343803626b697fd0c968121
> https://bugs.gentoo.org/907484
> 
> Signed-off-by: Alfred Wingate <parona@protonmail.com>
> ---
>  libswscale/x86/rgb2rgb_template.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

Looks ok, will push soonish if noone objects.
diff mbox series

Patch

diff --git a/libswscale/x86/rgb2rgb_template.c b/libswscale/x86/rgb2rgb_template.c
index 4aba25dd51..edbacea784 100644
--- a/libswscale/x86/rgb2rgb_template.c
+++ b/libswscale/x86/rgb2rgb_template.c
@@ -1823,7 +1823,8 @@  void RENAME(ff_nv12ToUV)(uint8_t *dstU, uint8_t *dstV,
                          const uint8_t *src1,
                          const uint8_t *src2,
                          int w,
-                         uint32_t *unused2);
+                         uint32_t *unused2,
+                         void *opq);
 static void RENAME(deinterleaveBytes)(const uint8_t *src, uint8_t *dst1, uint8_t *dst2,
                                       int width, int height, int srcStride,
                                       int dst1Stride, int dst2Stride)
@@ -1831,7 +1832,7 @@  static void RENAME(deinterleaveBytes)(const uint8_t *src, uint8_t *dst1, uint8_t
     int h;
 
     for (h = 0; h < height; h++) {
-        RENAME(ff_nv12ToUV)(dst1, dst2, NULL, src, NULL, width, NULL);
+        RENAME(ff_nv12ToUV)(dst1, dst2, NULL, src, NULL, width, NULL, NULL);
         src  += srcStride;
         dst1 += dst1Stride;
         dst2 += dst2Stride;