PR #23529 opened by Ramiro Polla (ramiro)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/23529
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/23529.patch

It doesn't get modified, so there's no need to create a copy.



>From 957ac3c4c0eaaa3230743b30c1393bbaaffc41aa Mon Sep 17 00:00:00 2001
From: Ramiro Polla <[email protected]>
Date: Fri, 12 Jun 2026 01:04:49 +0200
Subject: [PATCH] swscale/tests/sws_ops_aarch64: remove on-stack copy of const
 data

It doesn't get modified, so there's no need to create a copy.
---
 libswscale/tests/sws_ops_aarch64.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/libswscale/tests/sws_ops_aarch64.c 
b/libswscale/tests/sws_ops_aarch64.c
index 3319af0536..2089a0f1b7 100644
--- a/libswscale/tests/sws_ops_aarch64.c
+++ b/libswscale/tests/sws_ops_aarch64.c
@@ -93,14 +93,12 @@ static int register_op(SwsContext *ctx, void *opaque, 
SwsOpList *ops)
         }
     }
 
-    /* Make on-stack copy of `ops` to iterate over */
-    SwsOpList rest = *ops;
     /* Use at most two full vregs during the widest precision section */
     int block_size = (ff_sws_op_list_max_size(ops) == 4) ? 8 : 16;
 
-    for (int i = 0; i < rest.num_ops; i++) {
+    for (int i = 0; i < ops->num_ops; i++) {
         SwsAArch64OpImplParams params = { 0 };
-        ret = convert_to_aarch64_impl(ctx, &rest, i, block_size, &params);
+        ret = convert_to_aarch64_impl(ctx, ops, i, block_size, &params);
         if (ret == AVERROR(ENOTSUP))
             continue;
         if (ret < 0)
-- 
2.52.0

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

Reply via email to