Commit 03123e40 authored by Pavel Koshevoy's avatar Pavel Koshevoy

lavfi/atempo: fix tempo range limit inconsistency

yae_set_tempo was overlooked when max tempo limit was raised to 100.

tested with:
./ffmpeg_g -i Delerium/SemanticSpaces/Gateway.mp3 \
-af asendcmd=f=asendcmd.cfg,atempo=1.0 -y /tmp/asendcmd-atempo.wav

where asendcmd.cfg was:
15.0-45.0 [enter] atempo tempo 2.0,
          [leave] atempo tempo 0.5;

60.0-300.0 [enter] atempo tempo 4.0,
           [leave] atempo tempo 1.0;
parent d39fae08
......@@ -149,11 +149,16 @@ typedef struct ATempoContext {
uint64_t nsamples_out;
} ATempoContext;
#define YAE_ATEMPO_MIN 0.5
#define YAE_ATEMPO_MAX 100.0
#define OFFSET(x) offsetof(ATempoContext, x)
static const AVOption atempo_options[] = {
{ "tempo", "set tempo scale factor",
OFFSET(tempo), AV_OPT_TYPE_DOUBLE, { .dbl = 1.0 }, 0.5, 100.0,
OFFSET(tempo), AV_OPT_TYPE_DOUBLE, { .dbl = 1.0 },
YAE_ATEMPO_MIN,
YAE_ATEMPO_MAX,
AV_OPT_FLAG_AUDIO_PARAM | AV_OPT_FLAG_FILTERING_PARAM },
{ NULL }
};
......@@ -331,9 +336,9 @@ static int yae_set_tempo(AVFilterContext *ctx, const char *arg_tempo)
return AVERROR(EINVAL);
}
if (tempo < 0.5 || tempo > 2.0) {
av_log(ctx, AV_LOG_ERROR, "Tempo value %f exceeds [0.5, 2.0] range\n",
tempo);
if (tempo < YAE_ATEMPO_MIN || tempo > YAE_ATEMPO_MAX) {
av_log(ctx, AV_LOG_ERROR, "Tempo value %f exceeds [%f, %f] range\n",
tempo, YAE_ATEMPO_MIN, YAE_ATEMPO_MAX);
return AVERROR(EINVAL);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment