Re: [FFmpeg-devel] [PATCH] avutil/imgutils: optimize image copy efficiency

2023-05-05 Thread ??????
There is an if condition "FFALIGN(linesize[i], align) == linesize[i] && src_linesize[i] == linesize[i]". When src's linesize equals to dst, it has no padding in this case I think. If has padding, it will execute previous process. As a result, we could ignore the situation of padding in this case

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: optimize image copy efficiency

2023-05-04 Thread ??????
Thank you for your review, Martin. I try add some data, and submit again. -- Original -- From: "FFmpeg development discussions and patches"

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: optimize image copy efficiency

2023-05-04 Thread Martin Storsjö
On Thu, 4 May 2023, xufuji456 wrote: It makes sense when copying 4K/8K video, if linesize equals to aligned linesize. Signed-off-by: xufuji456 <839789...@qq.com> --- libavutil/imgutils.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libavutil/imgutils.c b/l

[FFmpeg-devel] [PATCH] avutil/imgutils: optimize image copy efficiency

2023-05-04 Thread xufuji456
It makes sense when copying 4K/8K video, if linesize equals to aligned linesize. Signed-off-by: xufuji456 <839789...@qq.com> --- libavutil/imgutils.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libavutil/imgutils.c b/libavutil/imgutils.c index 9ab5757cf6..