[FFmpeg-devel] [PATCH] matroska: Set CodecDelay and SeekPreroll in the AVStream

[Michael Niedermayer]

On Wed, Sep 18, 2013 at 02:12:58PM -0700, Vignesh Venkatasubramanian wrote:
> This patch exports the values of Codec Delay and Seek Preroll
> container elements as in the AVStream structure. The seek_preroll
> field has been added to the AVStream struct and the minor version
> of libavformat has been bumped.
> 
> Signed-off-by: Vignesh Venkatasubramanian <vigneshv at google.com>
> ---
>  libavformat/avformat.h    |  3 +++
>  libavformat/matroskadec.c | 14 ++++++++++++++
>  libavformat/version.h     |  2 +-
>  3 files changed, 18 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> index b18eb3f..94acd86 100644
> --- a/libavformat/avformat.h
> +++ b/libavformat/avformat.h
> @@ -888,6 +888,9 @@ typedef struct AVStream {
>       */
>      int pts_wrap_behavior;
>  
> +    /* Number of samples to skip after a discontinuity */
> +    int seek_preroll;
> +
>  } AVStream;

I assume this is intended to be accessed from outside avformat, in
which case it needs a wraper functions to access similar to
av_stream_get_r_frame_rate/av_stream_set_r_frame_rate
or an entry in the AVOption table
it should also be mentioned in the comment that acesses from outside
have to use these wrapers
and the comment should be doxygen compatible like the others above


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130924/fd191712/attachment.asc>