PR #21214 opened by Martin Storsjö (mstorsjo)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21214
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21214.patch

If we had SVE enabled and formatted info about its vector lengths, it would be 
overwritten by the SME info.

This is only an issue for SME, which can be printed after SVE. Or should we 
consistently use `av_strlcatf` for all of these printouts for consistency?


From 009fff560c4c44b4adceec0ee3db21e264f77169 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?= <[email protected]>
Date: Tue, 16 Dec 2025 14:43:11 +0200
Subject: [PATCH] checkasm: Use av_strlcatf for appending SME info after SVE

If we had SVE enabled and formatted info about its vector lengths,
it would be overwritten by the SME info.
---
 tests/checkasm/checkasm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c
index 62fa7fef12..461f1e7e5a 100644
--- a/tests/checkasm/checkasm.c
+++ b/tests/checkasm/checkasm.c
@@ -1049,8 +1049,8 @@ int main(int argc, char *argv[])
 #endif
 #if ARCH_AARCH64 && HAVE_SME
     if (have_sme(av_get_cpu_flags()))
-        snprintf(arch_info_buf, sizeof(arch_info_buf),
-                 "SME %d bits, ", 8 * ff_aarch64_sme_length());
+        av_strlcatf(arch_info_buf, sizeof(arch_info_buf),
+                    "SME %d bits, ", 8 * ff_aarch64_sme_length());
 #endif
 #if ARCH_RISCV && HAVE_RVV
     if (av_get_cpu_flags() & AV_CPU_FLAG_RVV_I32)
-- 
2.49.1

_______________________________________________
ffmpeg-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to