PR #23501 opened by Zhao Zhili (quink)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/23501
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/23501.patch

# Summary of changes

av_clipd() already bounds the result to [1, 65536]. The surrounding
av_clip() can never alter it.

<!--
If this PR requires new FATE test samples, attach them to the PR and
list their target paths below (relative to the fate-suite root).

Attached filenames must match the sample's filename:

```fate-samples
# e.g. vorbis/new-sample.ogg
```
-->



>From 9787a9a37976dde5921dd3b020b6904d2d396dcf Mon Sep 17 00:00:00 2001
From: Zhao Zhili <[email protected]>
Date: Tue, 16 Jun 2026 18:35:45 +0800
Subject: [PATCH] avfilter/avf_showcwt: drop redundant outer av_clip

av_clipd() already bounds the result to [1, 65536]. The surrounding
av_clip() can never alter it.
---
 libavfilter/avf_showcwt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavfilter/avf_showcwt.c b/libavfilter/avf_showcwt.c
index 4187aa3bb6..dbb3c8386b 100644
--- a/libavfilter/avf_showcwt.c
+++ b/libavfilter/avf_showcwt.c
@@ -882,7 +882,7 @@ static int config_output(AVFilterLink *outlink)
                  frequency_band(s->frequency_band,
                                 s->frequency_band_count, maximum_frequency - 
minimum_frequency,
                                 minimum_frequency, s->frequency_scale, 
s->deviation);
-    s->nb_consumed_samples = av_clip(av_clipd(nb_samples, 1, 65536), 1, 65536);
+    s->nb_consumed_samples = av_clipd(nb_samples, 1, 65536);
 
     s->nb_threads = FFMIN(s->frequency_band_count, 
ff_filter_get_nb_threads(ctx));
     s->nb_channels = inlink->ch_layout.nb_channels;
-- 
2.52.0

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

Reply via email to