Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types
@ 2024-05-13  9:28 Andreas Rheinhardt
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 02/10] tests/checkasm/huffyuvdsp: Use correct function pointer type Andreas Rheinhardt
                   ` (9 more replies)
  0 siblings, 10 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:28 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Forgotten in b3bbbb14d0685c8c1fbcf8455e59c7f444290c7c.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/hevc_add_res.c |  2 +-
 tests/checkasm/hevc_deblock.c |  6 ++++--
 tests/checkasm/hevc_pel.c     | 28 ++++++++++++++--------------
 tests/checkasm/hevc_sao.c     |  8 ++++----
 4 files changed, 23 insertions(+), 21 deletions(-)

diff --git a/tests/checkasm/hevc_add_res.c b/tests/checkasm/hevc_add_res.c
index f35e9fccd9..9dec3705c1 100644
--- a/tests/checkasm/hevc_add_res.c
+++ b/tests/checkasm/hevc_add_res.c
@@ -50,7 +50,7 @@ static void compare_add_res(int size, ptrdiff_t stride, int overflow_test, int m
     LOCAL_ALIGNED_32(uint8_t, dst0, [32 * 32 * 2]);
     LOCAL_ALIGNED_32(uint8_t, dst1, [32 * 32 * 2]);
 
-    declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, int16_t *res, ptrdiff_t stride);
+    declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, const int16_t *res, ptrdiff_t stride);
 
     randomize_buffers(res0, size);
     randomize_buffers2(dst0, size, mask);
diff --git a/tests/checkasm/hevc_deblock.c b/tests/checkasm/hevc_deblock.c
index 04cf9d87ac..c7f4f7e9ab 100644
--- a/tests/checkasm/hevc_deblock.c
+++ b/tests/checkasm/hevc_deblock.c
@@ -57,7 +57,8 @@ static void check_deblock_chroma(HEVCDSPContext *h, int bit_depth, int c)
     LOCAL_ALIGNED_32(uint8_t, buf0, [BUF_SIZE]);
     LOCAL_ALIGNED_32(uint8_t, buf1, [BUF_SIZE]);
 
-    declare_func(void, uint8_t *pix, ptrdiff_t stride, int32_t *tc, uint8_t *no_p, uint8_t *no_q);
+    declare_func(void, uint8_t *pix, ptrdiff_t stride,
+                 const int32_t *tc, const uint8_t *no_p, const uint8_t *no_q);
 
     if (check_func(c ? h->hevc_h_loop_filter_chroma_c : h->hevc_h_loop_filter_chroma,
                          "hevc_h_loop_filter_chroma%d%s", bit_depth, c ? "_full" : ""))
@@ -226,7 +227,8 @@ static void check_deblock_luma(HEVCDSPContext *h, int bit_depth, int c)
     uint8_t *ptr0 = buf0 + BUF_OFFSET,
             *ptr1 = buf1 + BUF_OFFSET;
 
-    declare_func(void, uint8_t *pix, ptrdiff_t stride, int beta, int32_t *tc, uint8_t *no_p, uint8_t *no_q);
+    declare_func(void, uint8_t *pix, ptrdiff_t stride, int beta,
+                 const int32_t *tc, const uint8_t *no_p, const uint8_t *no_q);
     memset(buf0, 0, BUF_SIZE);
 
     for (int j = 0; j < 3; j++) {
diff --git a/tests/checkasm/hevc_pel.c b/tests/checkasm/hevc_pel.c
index ed22ec4f9d..aebdf104e6 100644
--- a/tests/checkasm/hevc_pel.c
+++ b/tests/checkasm/hevc_pel.c
@@ -88,7 +88,7 @@ static void checkasm_check_hevc_qpel(void)
 
     HEVCDSPContext h;
     int size, bit_depth, i, j;
-    declare_func(void, int16_t *dst, uint8_t *src, ptrdiff_t srcstride,
+    declare_func(void, int16_t *dst, const uint8_t *src, ptrdiff_t srcstride,
                  int height, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -132,7 +132,7 @@ static void checkasm_check_hevc_qpel_uni(void)
 
     HEVCDSPContext h;
     int size, bit_depth, i, j;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
                  int height, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -182,7 +182,7 @@ static void checkasm_check_hevc_qpel_uni_w(void)
     HEVCDSPContext h;
     int size, bit_depth, i, j;
     const int *denom, *wx, *ox;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
                  int height, int denom, int wx, int ox, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -239,8 +239,8 @@ static void checkasm_check_hevc_qpel_bi(void)
 
     HEVCDSPContext h;
     int size, bit_depth, i, j;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
-                 int16_t *src2,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
+                 const int16_t *src2,
                  int height, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -292,8 +292,8 @@ static void checkasm_check_hevc_qpel_bi_w(void)
     HEVCDSPContext h;
     int size, bit_depth, i, j;
     const int *denom, *wx, *ox;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
-                 int16_t *src2,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
+                 const int16_t *src2,
                  int height, int denom, int wx0, int wx1,
                  int ox0, int ox1, intptr_t mx, intptr_t my, int width);
 
@@ -352,7 +352,7 @@ static void checkasm_check_hevc_epel(void)
 
     HEVCDSPContext h;
     int size, bit_depth, i, j;
-    declare_func(void, int16_t *dst, uint8_t *src, ptrdiff_t srcstride,
+    declare_func(void, int16_t *dst, const uint8_t *src, ptrdiff_t srcstride,
                  int height, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -396,7 +396,7 @@ static void checkasm_check_hevc_epel_uni(void)
 
     HEVCDSPContext h;
     int size, bit_depth, i, j;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
                  int height, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -446,7 +446,7 @@ static void checkasm_check_hevc_epel_uni_w(void)
     HEVCDSPContext h;
     int size, bit_depth, i, j;
     const int *denom, *wx, *ox;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
                  int height, int denom, int wx, int ox, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -503,8 +503,8 @@ static void checkasm_check_hevc_epel_bi(void)
 
     HEVCDSPContext h;
     int size, bit_depth, i, j;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
-                 int16_t *src2,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
+                 const int16_t *src2,
                  int height, intptr_t mx, intptr_t my, int width);
 
     for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
@@ -556,8 +556,8 @@ static void checkasm_check_hevc_epel_bi_w(void)
     HEVCDSPContext h;
     int size, bit_depth, i, j;
     const int *denom, *wx, *ox;
-    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
-                 int16_t *src2,
+    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
+                 const int16_t *src2,
                  int height, int denom, int wx0, int wx1,
                  int ox0, int ox1, intptr_t mx, intptr_t my, int width);
 
diff --git a/tests/checkasm/hevc_sao.c b/tests/checkasm/hevc_sao.c
index d05af9ac72..21c22b9749 100644
--- a/tests/checkasm/hevc_sao.c
+++ b/tests/checkasm/hevc_sao.c
@@ -78,8 +78,8 @@ static void check_sao_band(HEVCDSPContext *h, int bit_depth)
         int block_size = sao_size[i];
         int prev_size = i > 0 ? sao_size[i - 1] : 0;
         ptrdiff_t stride = PIXEL_STRIDE*SIZEOF_PIXEL;
-        declare_func(void, uint8_t *dst, uint8_t *src, ptrdiff_t dst_stride, ptrdiff_t src_stride,
-                     int16_t *sao_offset_val, int sao_left_class, int width, int height);
+        declare_func(void, uint8_t *dst, const uint8_t *src, ptrdiff_t dst_stride, ptrdiff_t src_stride,
+                     const int16_t *sao_offset_val, int sao_left_class, int width, int height);
 
         if (check_func(h->sao_band_filter[i], "hevc_sao_band_%d_%d", block_size, bit_depth)) {
 
@@ -116,8 +116,8 @@ static void check_sao_edge(HEVCDSPContext *h, int bit_depth)
         int prev_size = i > 0 ? sao_size[i - 1] : 0;
         ptrdiff_t stride = PIXEL_STRIDE*SIZEOF_PIXEL;
         int offset = (AV_INPUT_BUFFER_PADDING_SIZE + PIXEL_STRIDE)*SIZEOF_PIXEL;
-        declare_func(void, uint8_t *dst, uint8_t *src, ptrdiff_t stride_dst,
-                     int16_t *sao_offset_val, int eo, int width, int height);
+        declare_func(void, uint8_t *dst, const uint8_t *src, ptrdiff_t stride_dst,
+                     const int16_t *sao_offset_val, int eo, int width, int height);
 
         for (int w = prev_size + 4; w <= block_size; w += 4) {
             randomize_buffers(src0, src1, BUF_SIZE);
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 02/10] tests/checkasm/huffyuvdsp: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
@ 2024-05-13  9:29 ` Andreas Rheinhardt
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 03/10] tests/checkasm/llviddsp: " Andreas Rheinhardt
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:29 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/huffyuvdsp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/checkasm/huffyuvdsp.c b/tests/checkasm/huffyuvdsp.c
index a08f5a8391..7491a8f14c 100644
--- a/tests/checkasm/huffyuvdsp.c
+++ b/tests/checkasm/huffyuvdsp.c
@@ -42,7 +42,7 @@ static void check_add_int16(HuffYUVDSPContext *c, unsigned mask, int width, cons
     uint16_t *dst0 = av_mallocz(width * sizeof(uint16_t));
     uint16_t *dst1 = av_mallocz(width * sizeof(uint16_t));
 
-    declare_func(void, uint16_t *dst, uint16_t *src, unsigned mask, int w);
+    declare_func(void, uint16_t *dst, const uint16_t *src, unsigned mask, int w);
 
     if (!src0 || !src1 || !dst0 || !dst1)
         fail();
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 03/10] tests/checkasm/llviddsp: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 02/10] tests/checkasm/huffyuvdsp: Use correct function pointer type Andreas Rheinhardt
@ 2024-05-13  9:29 ` Andreas Rheinhardt
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 04/10] tests/checkasm/motion: " Andreas Rheinhardt
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:29 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/llviddsp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/checkasm/llviddsp.c b/tests/checkasm/llviddsp.c
index 00ad21a7cc..b75c0ea099 100644
--- a/tests/checkasm/llviddsp.c
+++ b/tests/checkasm/llviddsp.c
@@ -114,7 +114,7 @@ static void check_add_left_pred(LLVidDSPContext *c, int width, int acc, const ch
     uint8_t *dst1 = av_mallocz(width);
     uint8_t *src0 = av_calloc(width, sizeof(*src0));
     uint8_t *src1 = av_calloc(width, sizeof(*src1));
-    declare_func(int, uint8_t *dst, uint8_t *src, ptrdiff_t w, int acc);
+    declare_func(int, uint8_t *dst, const uint8_t *src, ptrdiff_t w, int acc);
 
     init_buffer(src0, src1, uint8_t, width);
 
@@ -143,7 +143,7 @@ static void check_add_left_pred_16(LLVidDSPContext *c, unsigned mask, int width,
     uint16_t *dst1 = av_calloc(width, sizeof(*dst1));
     uint16_t *src0 = av_calloc(width, sizeof(*src0));
     uint16_t *src1 = av_calloc(width, sizeof(*src1));
-    declare_func(int, uint16_t *dst, uint16_t *src, unsigned mask, ptrdiff_t w, unsigned acc);
+    declare_func(int, uint16_t *dst, const uint16_t *src, unsigned mask, ptrdiff_t w, unsigned acc);
 
     init_buffer(src0, src1, uint16_t, width);
 
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 04/10] tests/checkasm/motion: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 02/10] tests/checkasm/huffyuvdsp: Use correct function pointer type Andreas Rheinhardt
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 03/10] tests/checkasm/llviddsp: " Andreas Rheinhardt
@ 2024-05-13  9:29 ` Andreas Rheinhardt
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 05/10] tests/checkasm/vp8dsp: " Andreas Rheinhardt
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:29 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Forgotten in abb85429f3424375f21bdd135656c2d88357b3d5.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/motion.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/checkasm/motion.c b/tests/checkasm/motion.c
index 71a0ee3fc0..e7a36bbbda 100644
--- a/tests/checkasm/motion.c
+++ b/tests/checkasm/motion.c
@@ -52,8 +52,8 @@ static void test_motion(const char *name, me_cmp_func test_func)
     LOCAL_ALIGNED_16(uint8_t, img2, [WIDTH * HEIGHT]);
 
     declare_func_emms(AV_CPU_FLAG_MMX, int, struct MpegEncContext *c,
-                      uint8_t *blk1 /* align width (8 or 16) */,
-                      uint8_t *blk2 /* align 1 */, ptrdiff_t stride,
+                      const uint8_t *blk1 /* align width (8 or 16) */,
+                      const uint8_t *blk2 /* align 1 */, ptrdiff_t stride,
                       int h);
 
     if (test_func == NULL) {
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 05/10] tests/checkasm/vp8dsp: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (2 preceding siblings ...)
  2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 04/10] tests/checkasm/motion: " Andreas Rheinhardt
@ 2024-05-13  9:30 ` Andreas Rheinhardt
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 06/10] tests/checkasm/vvc_mc: " Andreas Rheinhardt
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:30 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Forgotten in a54e53a1c428299b19c7b4e2b66d01c0482c41dd.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/vp8dsp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/checkasm/vp8dsp.c b/tests/checkasm/vp8dsp.c
index 4cd0f8ac4f..6e989819ef 100644
--- a/tests/checkasm/vp8dsp.c
+++ b/tests/checkasm/vp8dsp.c
@@ -269,7 +269,8 @@ static void check_mc(void)
     LOCAL_ALIGNED_16(uint8_t, dst1, [16 * 16]);
     VP8DSPContext d;
     int type, k, dx, dy;
-    declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *, ptrdiff_t, uint8_t *, ptrdiff_t, int, int, int);
+    declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *, ptrdiff_t,
+                      const uint8_t *, ptrdiff_t, int, int, int);
 
     ff_vp78dsp_init(&d);
 
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 06/10] tests/checkasm/vvc_mc: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (3 preceding siblings ...)
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 05/10] tests/checkasm/vp8dsp: " Andreas Rheinhardt
@ 2024-05-13  9:30 ` Andreas Rheinhardt
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 07/10] tests/checkasm/vf_colorspace: " Andreas Rheinhardt
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:30 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/vvc_mc.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tests/checkasm/vvc_mc.c b/tests/checkasm/vvc_mc.c
index 97f57cb401..71087dc3dd 100644
--- a/tests/checkasm/vvc_mc.c
+++ b/tests/checkasm/vvc_mc.c
@@ -121,7 +121,8 @@ static void check_put_vvc_luma_uni(void)
 
     VVCDSPContext c;
     declare_func(void, uint8_t *dst, ptrdiff_t dststride,
-        uint8_t *src, ptrdiff_t srcstride,  int height, const int8_t *hf, const int8_t *vf, int width);
+        const uint8_t *src, ptrdiff_t srcstride,  int height,
+        const int8_t *hf, const int8_t *vf, int width);
 
     for (int bit_depth = 8; bit_depth <= 12; bit_depth += 2) {
         ff_vvc_dsp_init(&c, bit_depth);
@@ -219,7 +220,8 @@ static void check_put_vvc_chroma_uni(void)
 
     VVCDSPContext c;
     declare_func(void, uint8_t *dst, ptrdiff_t dststride,
-        uint8_t *src, ptrdiff_t srcstride,  int height, const int8_t *hf, const int8_t *vf, int width);
+        const uint8_t *src, ptrdiff_t srcstride, int height,
+        const int8_t *hf, const int8_t *vf, int width);
 
     for (int bit_depth = 8; bit_depth <= 12; bit_depth += 2) {
         ff_vvc_dsp_init(&c, bit_depth);
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 07/10] tests/checkasm/vf_colorspace: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (4 preceding siblings ...)
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 06/10] tests/checkasm/vvc_mc: " Andreas Rheinhardt
@ 2024-05-13  9:30 ` Andreas Rheinhardt
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 08/10] tests/checkasm/vf_bwdif: " Andreas Rheinhardt
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:30 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Forgotten in 9b26a8077f1c0139fdcc236d3de08cd2bdc4ec0f.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/vf_colorspace.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/checkasm/vf_colorspace.c b/tests/checkasm/vf_colorspace.c
index 31efa671f9..1b0d202ce6 100644
--- a/tests/checkasm/vf_colorspace.c
+++ b/tests/checkasm/vf_colorspace.c
@@ -53,8 +53,8 @@ static const unsigned bpp_mask[] = { 0xffffffff, 0x03ff03ff, 0x0fff0fff };
 
 static void check_yuv2yuv(void)
 {
-    declare_func(void, uint8_t *dst[3], ptrdiff_t dst_stride[3],
-                 uint8_t *src[3], ptrdiff_t src_stride[3],
+    declare_func(void, uint8_t *dst[3], const ptrdiff_t dst_stride[3],
+                 uint8_t *src[3], const ptrdiff_t src_stride[3],
                  int w, int h, const int16_t coeff[3][3][8],
                  const int16_t off[2][8]);
     ColorSpaceDSPContext dsp;
@@ -122,7 +122,7 @@ static void check_yuv2yuv(void)
 static void check_yuv2rgb(void)
 {
     declare_func(void, int16_t *dst[3], ptrdiff_t dst_stride,
-                 uint8_t *src[3], ptrdiff_t src_stride[3],
+                 uint8_t *src[3], const ptrdiff_t src_stride[3],
                  int w, int h, const int16_t coeff[3][3][8],
                  const int16_t off[8]);
     ColorSpaceDSPContext dsp;
@@ -198,7 +198,7 @@ static void check_yuv2rgb(void)
 
 static void check_rgb2yuv(void)
 {
-    declare_func(void, uint8_t *dst[3], ptrdiff_t dst_stride[3],
+    declare_func(void, uint8_t *dst[3], const ptrdiff_t dst_stride[3],
                  int16_t *src[3], ptrdiff_t src_stride,
                  int w, int h, const int16_t coeff[3][3][8],
                  const int16_t off[8]);
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 08/10] tests/checkasm/vf_bwdif: Use correct function pointer type
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (5 preceding siblings ...)
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 07/10] tests/checkasm/vf_colorspace: " Andreas Rheinhardt
@ 2024-05-13  9:30 ` Andreas Rheinhardt
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 09/10] tests/checkasm/vf_blend: Add missing function parameter Andreas Rheinhardt
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:30 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Forgotten in fa06f48371ba2b73c139810db5bb893dc27eda7a.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/vf_bwdif.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/checkasm/vf_bwdif.c b/tests/checkasm/vf_bwdif.c
index fae61b62e4..222e6aa4fa 100644
--- a/tests/checkasm/vf_bwdif.c
+++ b/tests/checkasm/vf_bwdif.c
@@ -40,7 +40,7 @@
         const int stride = WIDTH;                                              \
         const int mask = (1<<depth)-1;                                         \
                                                                                \
-        declare_func(void, void *dst, void *prev, void *cur, void *next,       \
+        declare_func(void, void *dst, const void *prev, const void *cur, const void *next, \
                         int w, int prefs, int mrefs, int prefs2, int mrefs2,   \
                         int prefs3, int mrefs3, int prefs4, int mrefs4,        \
                         int parity, int clip_max);                             \
@@ -181,7 +181,7 @@ void checkasm_check_vf_bwdif(void)
             for (parity = 0; parity != 2; ++parity) {
                 if (check_func(ctx_8.filter_edge, "bwdif8.edge.s%d.p%d", spat, parity)) {
 
-                    declare_func(void, void *dst1, void *prev1, void *cur1, void *next1,
+                    declare_func(void, void *dst1, const void *prev1, const void *cur1, const void *next1,
                                             int w, int prefs, int mrefs, int prefs2, int mrefs2,
                                             int parity, int clip_max, int spat);
 
@@ -225,7 +225,7 @@ void checkasm_check_vf_bwdif(void)
         const int stride = WIDTH;
         const int mask = (1<<8)-1;
 
-        declare_func(void, void *dst1, void *cur1, int w, int prefs, int mrefs,
+        declare_func(void, void *dst1, const void *cur1, int w, int prefs, int mrefs,
                      int prefs3, int mrefs3, int parity, int clip_max);
 
         randomize_buffers( cur0,  cur1, mask, 11*WIDTH);
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 09/10] tests/checkasm/vf_blend: Add missing function parameter
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (6 preceding siblings ...)
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 08/10] tests/checkasm/vf_bwdif: " Andreas Rheinhardt
@ 2024-05-13  9:30 ` Andreas Rheinhardt
  2024-05-14 19:48   ` Marton Balint
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 10/10] tests/checkasm/sw_gbrp: Use correct function types for calls Andreas Rheinhardt
  2024-05-15 10:37 ` [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
  9 siblings, 1 reply; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:30 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Forgotten in 5b8faaad6c71bbb90951ca1642391e11cf6f5f91.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/vf_blend.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/checkasm/vf_blend.c b/tests/checkasm/vf_blend.c
index b5a96ee4bc..5ebfc11fed 100644
--- a/tests/checkasm/vf_blend.c
+++ b/tests/checkasm/vf_blend.c
@@ -68,7 +68,7 @@
                      const uint8_t *bottom, ptrdiff_t bottom_linesize,                     \
                      uint8_t *dst, ptrdiff_t dst_linesize,                                 \
                      ptrdiff_t width, ptrdiff_t height,                                    \
-                     struct FilterParams *param, double *values);                          \
+                     struct FilterParams *param, double *values, int starty);              \
         w = WIDTH / depth;                                                                 \
                                                                                            \
         for (i = 0; i < BUF_UNITS - 1; i++) {                                              \
@@ -76,14 +76,14 @@
             int dst_offset = i * SIZE_PER_UNIT; /* dst must be aligned */                  \
             randomize_buffers();                                                           \
             call_ref(top1 + src_offset, w, bot1 + src_offset, w,                           \
-                     dst1 + dst_offset, w, w, HEIGHT, &param, NULL);                       \
+                     dst1 + dst_offset, w, w, HEIGHT, &param, NULL, 0);                    \
             call_new(top2 + src_offset, w, bot2 + src_offset, w,                           \
-                     dst2 + dst_offset, w, w, HEIGHT, &param, NULL);                       \
+                     dst2 + dst_offset, w, w, HEIGHT, &param, NULL, 0);                    \
             if (memcmp(top1, top2, BUF_SIZE) || memcmp(bot1, bot2, BUF_SIZE) || memcmp(dst1, dst2, BUF_SIZE)) \
                 fail();                                                                    \
         }                                                                                  \
         bench_new(top2, w / 4, bot2, w / 4, dst2, w / 4,                                   \
-                  w / 4, HEIGHT / 4, &param, NULL);                                        \
+                  w / 4, HEIGHT / 4, &param, NULL, 0);                                     \
     } while (0)
 
 void checkasm_check_blend(void)
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 10/10] tests/checkasm/sw_gbrp: Use correct function types for calls
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (7 preceding siblings ...)
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 09/10] tests/checkasm/vf_blend: Add missing function parameter Andreas Rheinhardt
@ 2024-05-13  9:30 ` Andreas Rheinhardt
  2024-05-15 10:37 ` [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-13  9:30 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

E.g. f2de911818fbd7e73343803626b697fd0c968121 forgot to
add the unused void* here.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 tests/checkasm/sw_gbrp.c | 34 ++++++++++++++++++----------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/tests/checkasm/sw_gbrp.c b/tests/checkasm/sw_gbrp.c
index 848e5366ad..b845da32a6 100644
--- a/tests/checkasm/sw_gbrp.c
+++ b/tests/checkasm/sw_gbrp.c
@@ -76,7 +76,7 @@ static void check_output_yuv2gbrp(void)
     uint8_t *dst0[4];
     uint8_t *dst1[4];
 
-    declare_func(void, void *c, const int16_t *lumFilter,
+    declare_func(void, struct SwsContext *c, const int16_t *lumFilter,
                        const int16_t **lumSrcx, int lumFilterSize,
                        const int16_t *chrFilter, const int16_t **chrUSrcx,
                        const int16_t **chrVSrcx, int chrFilterSize,
@@ -202,10 +202,11 @@ static void check_input_planar_rgb_to_y(void)
 #define LARGEST_INPUT_SIZE 512
 #define INPUT_SIZES 6
     static const int input_sizes[] = {8, 24, 128, 144, 256, 512};
-    uint8_t *src[4];
+    const uint8_t *src[4];
     int32_t rgb2yuv[9] = {0};
 
-    declare_func(void, uint8_t *dst, uint8_t *src[4], int w, int32_t *rgb2yuv);
+    declare_func(void, uint8_t *dst, const uint8_t *src[4],
+                       int w, int32_t *rgb2yuv, void *opaque);
 
     LOCAL_ALIGNED_8(int32_t, src_r, [LARGEST_INPUT_SIZE]);
     LOCAL_ALIGNED_8(int32_t, src_g, [LARGEST_INPUT_SIZE]);
@@ -243,13 +244,13 @@ static void check_input_planar_rgb_to_y(void)
                 memset(dst0_y, 0xFF, LARGEST_INPUT_SIZE * sizeof(int32_t));
                 memset(dst1_y, 0xFF, LARGEST_INPUT_SIZE * sizeof(int32_t));
 
-                call_ref(dst0_y, src, dstW, rgb2yuv);
-                call_new(dst1_y, src, dstW, rgb2yuv);
+                call_ref(dst0_y, src, dstW, rgb2yuv, NULL);
+                call_new(dst1_y, src, dstW, rgb2yuv, NULL);
 
                 if (memcmp(dst0_y, dst1_y, dstW * byte_size))
                     fail();
 
-                bench_new(dst1_y, src, dstW, rgb2yuv);
+                bench_new(dst1_y, src, dstW, rgb2yuv, NULL);
 
             }
         }
@@ -269,11 +270,11 @@ static void check_input_planar_rgb_to_uv(void)
 #define LARGEST_INPUT_SIZE 512
 #define INPUT_SIZES 6
     static const int input_sizes[] = {8, 24, 128, 144, 256, 512};
-    uint8_t *src[4];
+    const uint8_t *src[4];
     int32_t rgb2yuv[9] = {0};
 
     declare_func(void, uint8_t *dstU, uint8_t *dstV,
-                       uint8_t *src[4], int w, int32_t *rgb2yuv);
+                       const uint8_t *src[4], int w, int32_t *rgb2yuv, void *opaque);
 
     LOCAL_ALIGNED_8(int32_t, src_r, [LARGEST_INPUT_SIZE]);
     LOCAL_ALIGNED_8(int32_t, src_g, [LARGEST_INPUT_SIZE]);
@@ -316,14 +317,14 @@ static void check_input_planar_rgb_to_uv(void)
                 memset(dst1_u, 0xFF, LARGEST_INPUT_SIZE * sizeof(int32_t));
                 memset(dst1_v, 0xFF, LARGEST_INPUT_SIZE * sizeof(int32_t));
 
-                call_ref(dst0_u, dst0_v, src, dstW, rgb2yuv);
-                call_new(dst1_u, dst1_v, src, dstW, rgb2yuv);
+                call_ref(dst0_u, dst0_v, src, dstW, rgb2yuv, NULL);
+                call_new(dst1_u, dst1_v, src, dstW, rgb2yuv, NULL);
 
                 if (memcmp(dst0_u, dst1_u, dstW * byte_size) ||
                     memcmp(dst0_v, dst1_v, dstW * byte_size))
                     fail();
 
-                bench_new(dst1_u, dst1_v, src, dstW, rgb2yuv);
+                bench_new(dst1_u, dst1_v, src, dstW, rgb2yuv, NULL);
             }
         }
     }
@@ -342,10 +343,11 @@ static void check_input_planar_rgb_to_a(void)
 #define LARGEST_INPUT_SIZE 512
 #define INPUT_SIZES 6
     static const int input_sizes[] = {8, 24, 128, 144, 256, 512};
-    uint8_t *src[4];
+    const uint8_t *src[4];
     int32_t rgb2yuv[9] = {0};
 
-    declare_func(void, uint8_t *dst, uint8_t *src[4], int w, int32_t *rgb2yuv);
+    declare_func(void, uint8_t *dst, const uint8_t *src[4],
+                       int w, int32_t *rgb2yuv, void *opaque);
 
     LOCAL_ALIGNED_8(int32_t, src_r, [LARGEST_INPUT_SIZE]);
     LOCAL_ALIGNED_8(int32_t, src_g, [LARGEST_INPUT_SIZE]);
@@ -386,12 +388,12 @@ static void check_input_planar_rgb_to_a(void)
                 memset(dst0_a, 0x00, LARGEST_INPUT_SIZE * sizeof(int32_t));
                 memset(dst1_a, 0x00, LARGEST_INPUT_SIZE * sizeof(int32_t));
 
-                call_ref(dst0_a, src, dstW, rgb2yuv);
-                call_new(dst1_a, src, dstW, rgb2yuv);
+                call_ref(dst0_a, src, dstW, rgb2yuv, NULL);
+                call_new(dst1_a, src, dstW, rgb2yuv, NULL);
 
                 if (memcmp(dst0_a, dst1_a, dstW * byte_size))
                     fail();
-                bench_new(dst1_a, src, dstW, rgb2yuv);
+                bench_new(dst1_a, src, dstW, rgb2yuv, NULL);
             }
         }
     }
-- 
2.40.1

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 09/10] tests/checkasm/vf_blend: Add missing function parameter
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 09/10] tests/checkasm/vf_blend: Add missing function parameter Andreas Rheinhardt
@ 2024-05-14 19:48   ` Marton Balint
  0 siblings, 0 replies; 12+ messages in thread
From: Marton Balint @ 2024-05-14 19:48 UTC (permalink / raw)
  To: FFmpeg development discussions and patches



On Mon, 13 May 2024, Andreas Rheinhardt wrote:

> Forgotten in 5b8faaad6c71bbb90951ca1642391e11cf6f5f91.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> tests/checkasm/vf_blend.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/tests/checkasm/vf_blend.c b/tests/checkasm/vf_blend.c
> index b5a96ee4bc..5ebfc11fed 100644
> --- a/tests/checkasm/vf_blend.c
> +++ b/tests/checkasm/vf_blend.c
> @@ -68,7 +68,7 @@
>                      const uint8_t *bottom, ptrdiff_t bottom_linesize,                     \
>                      uint8_t *dst, ptrdiff_t dst_linesize,                                 \
>                      ptrdiff_t width, ptrdiff_t height,                                    \
> -                     struct FilterParams *param, double *values);                          \
> +                     struct FilterParams *param, double *values, int starty);              \

You are going to need to adjust the parameters after the vf_blend patch I 
just pushed.

Thanks,
Marton


>         w = WIDTH / depth;                                                                 \
>                                                                                            \
>         for (i = 0; i < BUF_UNITS - 1; i++) {                                              \
> @@ -76,14 +76,14 @@
>             int dst_offset = i * SIZE_PER_UNIT; /* dst must be aligned */                  \
>             randomize_buffers();                                                           \
>             call_ref(top1 + src_offset, w, bot1 + src_offset, w,                           \
> -                     dst1 + dst_offset, w, w, HEIGHT, &param, NULL);                       \
> +                     dst1 + dst_offset, w, w, HEIGHT, &param, NULL, 0);                    \
>             call_new(top2 + src_offset, w, bot2 + src_offset, w,                           \
> -                     dst2 + dst_offset, w, w, HEIGHT, &param, NULL);                       \
> +                     dst2 + dst_offset, w, w, HEIGHT, &param, NULL, 0);                    \
>             if (memcmp(top1, top2, BUF_SIZE) || memcmp(bot1, bot2, BUF_SIZE) || memcmp(dst1, dst2, BUF_SIZE)) \
>                 fail();                                                                    \
>         }                                                                                  \
>         bench_new(top2, w / 4, bot2, w / 4, dst2, w / 4,                                   \
> -                  w / 4, HEIGHT / 4, &param, NULL);                                        \
> +                  w / 4, HEIGHT / 4, &param, NULL, 0);                                     \
>     } while (0)
>
> void checkasm_check_blend(void)
> -- 
> 2.40.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types
  2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
                   ` (8 preceding siblings ...)
  2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 10/10] tests/checkasm/sw_gbrp: Use correct function types for calls Andreas Rheinhardt
@ 2024-05-15 10:37 ` Andreas Rheinhardt
  9 siblings, 0 replies; 12+ messages in thread
From: Andreas Rheinhardt @ 2024-05-15 10:37 UTC (permalink / raw)
  To: ffmpeg-devel

Andreas Rheinhardt:
> Forgotten in b3bbbb14d0685c8c1fbcf8455e59c7f444290c7c.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  tests/checkasm/hevc_add_res.c |  2 +-
>  tests/checkasm/hevc_deblock.c |  6 ++++--
>  tests/checkasm/hevc_pel.c     | 28 ++++++++++++++--------------
>  tests/checkasm/hevc_sao.c     |  8 ++++----
>  4 files changed, 23 insertions(+), 21 deletions(-)
> 
> diff --git a/tests/checkasm/hevc_add_res.c b/tests/checkasm/hevc_add_res.c
> index f35e9fccd9..9dec3705c1 100644
> --- a/tests/checkasm/hevc_add_res.c
> +++ b/tests/checkasm/hevc_add_res.c
> @@ -50,7 +50,7 @@ static void compare_add_res(int size, ptrdiff_t stride, int overflow_test, int m
>      LOCAL_ALIGNED_32(uint8_t, dst0, [32 * 32 * 2]);
>      LOCAL_ALIGNED_32(uint8_t, dst1, [32 * 32 * 2]);
>  
> -    declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, int16_t *res, ptrdiff_t stride);
> +    declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, const int16_t *res, ptrdiff_t stride);
>  
>      randomize_buffers(res0, size);
>      randomize_buffers2(dst0, size, mask);
> diff --git a/tests/checkasm/hevc_deblock.c b/tests/checkasm/hevc_deblock.c
> index 04cf9d87ac..c7f4f7e9ab 100644
> --- a/tests/checkasm/hevc_deblock.c
> +++ b/tests/checkasm/hevc_deblock.c
> @@ -57,7 +57,8 @@ static void check_deblock_chroma(HEVCDSPContext *h, int bit_depth, int c)
>      LOCAL_ALIGNED_32(uint8_t, buf0, [BUF_SIZE]);
>      LOCAL_ALIGNED_32(uint8_t, buf1, [BUF_SIZE]);
>  
> -    declare_func(void, uint8_t *pix, ptrdiff_t stride, int32_t *tc, uint8_t *no_p, uint8_t *no_q);
> +    declare_func(void, uint8_t *pix, ptrdiff_t stride,
> +                 const int32_t *tc, const uint8_t *no_p, const uint8_t *no_q);
>  
>      if (check_func(c ? h->hevc_h_loop_filter_chroma_c : h->hevc_h_loop_filter_chroma,
>                           "hevc_h_loop_filter_chroma%d%s", bit_depth, c ? "_full" : ""))
> @@ -226,7 +227,8 @@ static void check_deblock_luma(HEVCDSPContext *h, int bit_depth, int c)
>      uint8_t *ptr0 = buf0 + BUF_OFFSET,
>              *ptr1 = buf1 + BUF_OFFSET;
>  
> -    declare_func(void, uint8_t *pix, ptrdiff_t stride, int beta, int32_t *tc, uint8_t *no_p, uint8_t *no_q);
> +    declare_func(void, uint8_t *pix, ptrdiff_t stride, int beta,
> +                 const int32_t *tc, const uint8_t *no_p, const uint8_t *no_q);
>      memset(buf0, 0, BUF_SIZE);
>  
>      for (int j = 0; j < 3; j++) {
> diff --git a/tests/checkasm/hevc_pel.c b/tests/checkasm/hevc_pel.c
> index ed22ec4f9d..aebdf104e6 100644
> --- a/tests/checkasm/hevc_pel.c
> +++ b/tests/checkasm/hevc_pel.c
> @@ -88,7 +88,7 @@ static void checkasm_check_hevc_qpel(void)
>  
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
> -    declare_func(void, int16_t *dst, uint8_t *src, ptrdiff_t srcstride,
> +    declare_func(void, int16_t *dst, const uint8_t *src, ptrdiff_t srcstride,
>                   int height, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -132,7 +132,7 @@ static void checkasm_check_hevc_qpel_uni(void)
>  
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
>                   int height, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -182,7 +182,7 @@ static void checkasm_check_hevc_qpel_uni_w(void)
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
>      const int *denom, *wx, *ox;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
>                   int height, int denom, int wx, int ox, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -239,8 +239,8 @@ static void checkasm_check_hevc_qpel_bi(void)
>  
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> -                 int16_t *src2,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
> +                 const int16_t *src2,
>                   int height, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -292,8 +292,8 @@ static void checkasm_check_hevc_qpel_bi_w(void)
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
>      const int *denom, *wx, *ox;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> -                 int16_t *src2,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
> +                 const int16_t *src2,
>                   int height, int denom, int wx0, int wx1,
>                   int ox0, int ox1, intptr_t mx, intptr_t my, int width);
>  
> @@ -352,7 +352,7 @@ static void checkasm_check_hevc_epel(void)
>  
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
> -    declare_func(void, int16_t *dst, uint8_t *src, ptrdiff_t srcstride,
> +    declare_func(void, int16_t *dst, const uint8_t *src, ptrdiff_t srcstride,
>                   int height, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -396,7 +396,7 @@ static void checkasm_check_hevc_epel_uni(void)
>  
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
>                   int height, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -446,7 +446,7 @@ static void checkasm_check_hevc_epel_uni_w(void)
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
>      const int *denom, *wx, *ox;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
>                   int height, int denom, int wx, int ox, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -503,8 +503,8 @@ static void checkasm_check_hevc_epel_bi(void)
>  
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> -                 int16_t *src2,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
> +                 const int16_t *src2,
>                   int height, intptr_t mx, intptr_t my, int width);
>  
>      for (bit_depth = 8; bit_depth <= 12; bit_depth++) {
> @@ -556,8 +556,8 @@ static void checkasm_check_hevc_epel_bi_w(void)
>      HEVCDSPContext h;
>      int size, bit_depth, i, j;
>      const int *denom, *wx, *ox;
> -    declare_func(void, uint8_t *dst, ptrdiff_t dststride, uint8_t *src, ptrdiff_t srcstride,
> -                 int16_t *src2,
> +    declare_func(void, uint8_t *dst, ptrdiff_t dststride, const uint8_t *src, ptrdiff_t srcstride,
> +                 const int16_t *src2,
>                   int height, int denom, int wx0, int wx1,
>                   int ox0, int ox1, intptr_t mx, intptr_t my, int width);
>  
> diff --git a/tests/checkasm/hevc_sao.c b/tests/checkasm/hevc_sao.c
> index d05af9ac72..21c22b9749 100644
> --- a/tests/checkasm/hevc_sao.c
> +++ b/tests/checkasm/hevc_sao.c
> @@ -78,8 +78,8 @@ static void check_sao_band(HEVCDSPContext *h, int bit_depth)
>          int block_size = sao_size[i];
>          int prev_size = i > 0 ? sao_size[i - 1] : 0;
>          ptrdiff_t stride = PIXEL_STRIDE*SIZEOF_PIXEL;
> -        declare_func(void, uint8_t *dst, uint8_t *src, ptrdiff_t dst_stride, ptrdiff_t src_stride,
> -                     int16_t *sao_offset_val, int sao_left_class, int width, int height);
> +        declare_func(void, uint8_t *dst, const uint8_t *src, ptrdiff_t dst_stride, ptrdiff_t src_stride,
> +                     const int16_t *sao_offset_val, int sao_left_class, int width, int height);
>  
>          if (check_func(h->sao_band_filter[i], "hevc_sao_band_%d_%d", block_size, bit_depth)) {
>  
> @@ -116,8 +116,8 @@ static void check_sao_edge(HEVCDSPContext *h, int bit_depth)
>          int prev_size = i > 0 ? sao_size[i - 1] : 0;
>          ptrdiff_t stride = PIXEL_STRIDE*SIZEOF_PIXEL;
>          int offset = (AV_INPUT_BUFFER_PADDING_SIZE + PIXEL_STRIDE)*SIZEOF_PIXEL;
> -        declare_func(void, uint8_t *dst, uint8_t *src, ptrdiff_t stride_dst,
> -                     int16_t *sao_offset_val, int eo, int width, int height);
> +        declare_func(void, uint8_t *dst, const uint8_t *src, ptrdiff_t stride_dst,
> +                     const int16_t *sao_offset_val, int eo, int width, int height);
>  
>          for (int w = prev_size + 4; w <= block_size; w += 4) {
>              randomize_buffers(src0, src1, BUF_SIZE);

Will apply this patchset tonight with the modifications necessitated by
recent changes (thanks for the heads-up, Marton).

- Andreas

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

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2024-05-15 10:37 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-13  9:28 [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt
2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 02/10] tests/checkasm/huffyuvdsp: Use correct function pointer type Andreas Rheinhardt
2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 03/10] tests/checkasm/llviddsp: " Andreas Rheinhardt
2024-05-13  9:29 ` [FFmpeg-devel] [PATCH 04/10] tests/checkasm/motion: " Andreas Rheinhardt
2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 05/10] tests/checkasm/vp8dsp: " Andreas Rheinhardt
2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 06/10] tests/checkasm/vvc_mc: " Andreas Rheinhardt
2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 07/10] tests/checkasm/vf_colorspace: " Andreas Rheinhardt
2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 08/10] tests/checkasm/vf_bwdif: " Andreas Rheinhardt
2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 09/10] tests/checkasm/vf_blend: Add missing function parameter Andreas Rheinhardt
2024-05-14 19:48   ` Marton Balint
2024-05-13  9:30 ` [FFmpeg-devel] [PATCH 10/10] tests/checkasm/sw_gbrp: Use correct function types for calls Andreas Rheinhardt
2024-05-15 10:37 ` [FFmpeg-devel] [PATCH 01/10] tests/checkasm/hevc_*: Fix funtion pointer types Andreas Rheinhardt

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git