Message ID | 20210207000423.8948-2-cus@passwd.hu |
---|---|
State | Accepted |
Commit | 529bf5daf2935fb87787d7f7a5263e1b254d59d2 |
Headers | show |
Series | [FFmpeg-devel,1/3] avfilter/avf_showcqt: use av_gettime_relative() instead of av_gettime() | expand |
Context | Check | Description |
---|---|---|
andriy/x86_make | success | Make finished |
andriy/x86_make_fate | success | Make fate finished |
andriy/PPC64_make | success | Make finished |
andriy/PPC64_make_fate | success | Make fate finished |
Marton Balint (12021-02-07): > Also rename the parameters of the function to match with the implementation. > > Signed-off-by: Marton Balint <cus@passwd.hu> > --- > libavdevice/timefilter.h | 15 +-------------- > 1 file changed, 1 insertion(+), 14 deletions(-) Probably ok, I do not remember correcly enough. Regards,
On Sun, 7 Feb 2021, Nicolas George wrote: > Marton Balint (12021-02-07): >> Also rename the parameters of the function to match with the implementation. >> >> Signed-off-by: Marton Balint <cus@passwd.hu> >> --- >> libavdevice/timefilter.h | 15 +-------------- >> 1 file changed, 1 insertion(+), 14 deletions(-) > > Probably ok, I do not remember correcly enough. Thanks, applied. Regards, Marton
diff --git a/libavdevice/timefilter.h b/libavdevice/timefilter.h index cb3d0a788f..de70945062 100644 --- a/libavdevice/timefilter.h +++ b/libavdevice/timefilter.h @@ -39,16 +39,6 @@ typedef struct TimeFilter TimeFilter; /** * Create a new Delay Locked Loop time filter * - * feedback2_factor and feedback3_factor are the factors used for the - * multiplications that are respectively performed in the second and third - * feedback paths of the loop. - * - * Unless you know what you are doing, you should set these as follow: - * - * o = 2 * M_PI * bandwidth * period_in_seconds - * feedback2_factor = sqrt(2) * o - * feedback3_factor = o * o - * * Where bandwidth is up to you to choose. Smaller values will filter out more * of the jitter, but also take a longer time for the loop to settle. A good * starting point is something between 0.3 and 3 Hz. @@ -59,11 +49,8 @@ typedef struct TimeFilter TimeFilter; * @param brandwidth filtering bandwidth, in Hz * * @return a pointer to a TimeFilter struct, or NULL on error - * - * For more details about these parameters and background concepts please see: - * http://www.kokkinizita.net/papers/usingdll.pdf */ -TimeFilter * ff_timefilter_new(double clock_period, double feedback2_factor, double feedback3_factor); +TimeFilter * ff_timefilter_new(double time_base, double period, double bandwidth); /** * Update the filter
Also rename the parameters of the function to match with the implementation. Signed-off-by: Marton Balint <cus@passwd.hu> --- libavdevice/timefilter.h | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-)