Re: [FFmpeg-devel] [PATCH] lavfi/atempo: fix range check if tempo is set by command

2018-10-03 Thread Nicolas George
Wang Bin (2018-10-03): > Less mistakes. For example the author forgot to change this line. This > function is not frequently called. The consistency must be ensured, but it must be static, not dynamic by looking up the option. Just use two constant macros. > av_opt_set or avfilter_process_comman

Re: [FFmpeg-devel] [PATCH] lavfi/atempo: fix range check if tempo is set by command

2018-10-03 Thread Wang Bin
Nicolas George 于2018年10月3日周三 下午11:05写道: > wbse...@gmail.com (2018-10-03): > > From: wang-bin > > > > --- > > libavfilter/af_atempo.c | 11 ++- > > 1 file changed, 6 insertions(+), 5 deletions(-) > > > > diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c > > index 52f15f2769.

Re: [FFmpeg-devel] [PATCH] lavfi/atempo: fix range check if tempo is set by command

2018-10-03 Thread Nicolas George
wbse...@gmail.com (2018-10-03): > From: wang-bin > > --- > libavfilter/af_atempo.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c > index 52f15f2769..1a004212a7 100644 > --- a/libavfilter/af_atempo.c > +++ b

[FFmpeg-devel] [PATCH] lavfi/atempo: fix range check if tempo is set by command

2018-10-03 Thread wbsecg1
From: wang-bin --- libavfilter/af_atempo.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c index 52f15f2769..1a004212a7 100644 --- a/libavfilter/af_atempo.c +++ b/libavfilter/af_atempo.c @@ -331,9 +331,10 @@ static