[FFmpeg-devel] [PATCH 3/3] lavfi: fix doxygen doc for AVFilterLink.time_base

Stefano Sabatini stefano.sabatini-lala at poste.it
Thu Aug 4 18:54:07 CEST 2011


On date Thursday 2011-08-04 15:47:02 +0300, Mina Nagy Zaki encoded:
> time_base is only applicable to video as the timebase for audio
> is defined as 1/sample_rate
> ---
>  libavfilter/avfilter.h |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/libavfilter/avfilter.h b/libavfilter/avfilter.h
> index 9e7a4cd..bf2d2b5 100644
> --- a/libavfilter/avfilter.h
> +++ b/libavfilter/avfilter.h
> @@ -646,11 +646,12 @@ struct AVFilterLink {
>      AVFilterBufferRef *out_buf;
>  
>      /**
> -     * Define the time base used by the PTS of the frames/samples
> -     * which will pass through this link.
> +     * Define the time base used by the PTS of the frames which
> +     * will pass through this link.
>       * During the configuration stage, each filter is supposed to
>       * change only the output timebase, while the timebase of the
>       * input link is assumed to be an unchangeable property.
> +     * This is applicable only to video links.
>       */
>      AVRational time_base;

Looks sane to me. I discussed this with Mina, and we came to the
conclusion that the timebase information for audio frames is carried
out by the samplerate set in the link, so you have:

time = pts * 1/samplerate

and thus time_base looks just redundant.

Please comment on this (Michael?), I'll apply in a few days if I see
no comments.
-- 
FFmpeg = Faithful Fundamentalist Mortal Purposeless Ecstatic Genius


More information about the ffmpeg-devel mailing list