[FFmpeg-cvslog] checkasm/sw_scale: add assertion for hscale assumption

Niklas Haas git at videolan.org
Mon Dec 23 12:26:24 EET 2024


ffmpeg | branch: master | Niklas Haas <git at haasn.dev> | Wed Dec 11 10:13:46 2024 +0100| [fe9bf7cd5235bbdbe95eb610540e6e8851e58b7d] | committer: Niklas Haas

checkasm/sw_scale: add assertion for hscale assumption

This code only checks hcScale. In practice this is not an issue because
the function pointers should always be identical to hyScale for the same
filter size.

Add an assertion just to make sure this assumption never regresses.

Signed-off-by: Niklas Haas <git at haasn.dev>
Sponsored-by: Sovereign Tech Fund

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=fe9bf7cd5235bbdbe95eb610540e6e8851e58b7d
---

 tests/checkasm/sw_scale.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/checkasm/sw_scale.c b/tests/checkasm/sw_scale.c
index 350f2b73d4..3d229775d7 100644
--- a/tests/checkasm/sw_scale.c
+++ b/tests/checkasm/sw_scale.c
@@ -346,6 +346,7 @@ static void check_hscale(void)
                 memcpy(filterAvx2, filter, sizeof(uint16_t) * (SRC_PIXELS * MAX_FILTER_WIDTH + MAX_FILTER_WIDTH));
                 ff_shuffle_filter_coefficients(c, filterPosAvx, width, filterAvx2, sws->dst_w);
 
+                av_assert0(c->hyScale == c->hcScale);
                 if (check_func(c->hcScale, "hscale_%d_to_%d__fs_%d_dstW_%d", c->srcBpc, c->dstBpc + 1, width, sws->dst_w)) {
                     memset(dst0, 0, SRC_PIXELS * sizeof(dst0[0]));
                     memset(dst1, 0, SRC_PIXELS * sizeof(dst1[0]));



More information about the ffmpeg-cvslog mailing list