[FFmpeg-devel] [PATCH 1/2] tests/checkasm/vvc_mc: fix indentation

[FFmpeg-devel] [PATCH 1/2] tests/checkasm/vvc_mc: fix indentation

[James Almer]

Signed-off-by: James Almer <jamrial@gmail.com>
---
 tests/checkasm/vvc_mc.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/tests/checkasm/vvc_mc.c b/tests/checkasm/vvc_mc.c
index 09cac82edb..40be837d68 100644
--- a/tests/checkasm/vvc_mc.c
+++ b/tests/checkasm/vvc_mc.c
@@ -341,8 +341,10 @@ static void check_vvc_sad(void)
         for (int w = 8; w <= 16; w *= 2) {
             for(int offy = 0; offy <= 4; offy++) {
                 for(int offx = 0; offx <= 4; offx++) {
-                    if(w * h >= 128) {
-                        if(check_func(c.inter.sad, "sad_%dx%d", w, h)) {
+                    if (w * h < 128)
+                        continue;
+
+                    if (check_func(c.inter.sad, "sad_%dx%d", w, h)) {
                         int result0;
                         int result1;
 
@@ -353,7 +355,6 @@ static void check_vvc_sad(void)
                             fail();
                         if(offx == 0 && offy == 0)
                             bench_new(src0 + PIXEL_STRIDE * 2 + 2, src1 + PIXEL_STRIDE * 2 + 2, offx, offy, w, h);
-                        }
                     }
                 }
             }
-- 
2.45.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] [PATCH 2/2] tests/checkasm/vvc_mc: don't zero the SAD buffers

[James Almer]

They will be filled immediately after.

Signed-off-by: James Almer <jamrial@gmail.com>
---
 tests/checkasm/vvc_mc.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/tests/checkasm/vvc_mc.c b/tests/checkasm/vvc_mc.c
index 40be837d68..bc6b580f42 100644
--- a/tests/checkasm/vvc_mc.c
+++ b/tests/checkasm/vvc_mc.c
@@ -333,9 +333,6 @@ static void check_vvc_sad(void)
     declare_func(int, const int16_t *src0, const int16_t *src1, int dx, int dy, int block_w, int block_h);
 
     ff_vvc_dsp_init(&c, bit_depth);
-    memset(src0, 0, MAX_CTU_SIZE * MAX_CTU_SIZE * 4 * sizeof(uint16_t));
-    memset(src1, 0, MAX_CTU_SIZE * MAX_CTU_SIZE * 4 * sizeof(uint16_t));
-
     randomize_pixels(src0, src1, MAX_CTU_SIZE * MAX_CTU_SIZE * 4);
     for (int h = 8; h <= 16; h *= 2) {
         for (int w = 8; w <= 16; w *= 2) {
-- 
2.45.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".

Re: [FFmpeg-devel] [PATCH 2/2] tests/checkasm/vvc_mc: don't zero the SAD buffers

[Nuo Mi]

On Wed, May 29, 2024 at 10:03 PM James Almer <jamrial@gmail.com> wrote:

> They will be filled immediately after.
>
Pushed.
thank you, James

>
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
>  tests/checkasm/vvc_mc.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/tests/checkasm/vvc_mc.c b/tests/checkasm/vvc_mc.c
> index 40be837d68..bc6b580f42 100644
> --- a/tests/checkasm/vvc_mc.c
> +++ b/tests/checkasm/vvc_mc.c
> @@ -333,9 +333,6 @@ static void check_vvc_sad(void)
>      declare_func(int, const int16_t *src0, const int16_t *src1, int dx,
> int dy, int block_w, int block_h);
>
>      ff_vvc_dsp_init(&c, bit_depth);
> -    memset(src0, 0, MAX_CTU_SIZE * MAX_CTU_SIZE * 4 * sizeof(uint16_t));
> -    memset(src1, 0, MAX_CTU_SIZE * MAX_CTU_SIZE * 4 * sizeof(uint16_t));
> -
>      randomize_pixels(src0, src1, MAX_CTU_SIZE * MAX_CTU_SIZE * 4);
>      for (int h = 8; h <= 16; h *= 2) {
>          for (int w = 8; w <= 16; w *= 2) {
> --
> 2.45.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".