diff mbox series

[FFmpeg-devel] checkasm: hevc_sao: Fix a regression in hevc_sao_edge

Message ID 3317084.44csPzL39Z@naoise
State Accepted
Commit e41bd6e65e1485fd1b44540638d06638406be2f4
Headers show
Series [FFmpeg-devel] checkasm: hevc_sao: Fix a regression in hevc_sao_edge | 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

Matthias Dressel Aug. 24, 2023, 2:58 p.m. UTC
From: Matthias Dressel <code@deadcode.eu>

check_func() might return NULL, in which case the function is not to be
benched. Introduced in cc679054c715acda9438e566b8de3a9eba421ac3.

Signed-off-by: Matthias Dressel <code@deadcode.eu>
---
 tests/checkasm/hevc_sao.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Martin Storsjö Aug. 24, 2023, 7:11 p.m. UTC | #1
On Thu, 24 Aug 2023, Matthias Dressel wrote:

> From: Matthias Dressel <code@deadcode.eu>
>
> check_func() might return NULL, in which case the function is not to be
> benched. Introduced in cc679054c715acda9438e566b8de3a9eba421ac3.
>
> Signed-off-by: Matthias Dressel <code@deadcode.eu>
> ---
> tests/checkasm/hevc_sao.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/checkasm/hevc_sao.c b/tests/checkasm/hevc_sao.c
> index cfee2ddf50..94aa1942e1 100644
> --- a/tests/checkasm/hevc_sao.c
> +++ b/tests/checkasm/hevc_sao.c
> @@ -134,8 +134,8 @@ static void check_sao_edge(HEVCDSPContext *h, int bit_depth)
>                     if (memcmp(dst0 + j*stride, dst1 + j*stride, w*SIZEOF_PIXEL))
>                         fail();
>                 }
> +                bench_new(dst1, src1 + offset, stride, offset_val, eo, block_size, block_size);
>             }
> -            bench_new(dst1, src1 + offset, stride, offset_val, eo, block_size, block_size);
>         }
>     }
> }
> -- 
> 2.41.0

LGTM, pushed.

// Martin
diff mbox series

Patch

diff --git a/tests/checkasm/hevc_sao.c b/tests/checkasm/hevc_sao.c
index cfee2ddf50..94aa1942e1 100644
--- a/tests/checkasm/hevc_sao.c
+++ b/tests/checkasm/hevc_sao.c
@@ -134,8 +134,8 @@  static void check_sao_edge(HEVCDSPContext *h, int bit_depth)
                     if (memcmp(dst0 + j*stride, dst1 + j*stride, w*SIZEOF_PIXEL))
                         fail();
                 }
+                bench_new(dst1, src1 + offset, stride, offset_val, eo, block_size, block_size);
             }
-            bench_new(dst1, src1 + offset, stride, offset_val, eo, block_size, block_size);
         }
     }
 }