[FFmpeg-devel] [PATCH] checkasm: Use av_strlcatf for appending SME info after SVE (PR #21214)

[FFmpeg-devel] [PATCH] checkasm: Use av_strlcatf for appending SME info after SVE (PR #21214)

[Martin Storsjö via ffmpeg-devel]

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?= <martin@martin.st>
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 -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org