[FFmpeg-devel] [FFmpeg-commits] Always use av_set_pts_info to set the stream time base.

[Måns Rullgård]

"Reimar D?ffinger " <git at ffmpeg.org> writes:

> diff --git a/libavformat/oggparseskeleton.c b/libavformat/oggparseskeleton.c
> index ad0dded..f0e17f9 100644
> --- a/libavformat/oggparseskeleton.c
> +++ b/libavformat/oggparseskeleton.c
> @@ -60,8 +60,9 @@ static int skeleton_header(AVFormatContext *s, int idx)
>          start_den = AV_RL64(buf+20);
>
>          if (start_den) {
> -            av_reduce(&start_time, &st->time_base.den, start_num, start_den, INT_MAX);
> -            st->time_base.num = 1;
> +            int64_t base_den;
> +            av_reduce(&start_time, &base_den, start_num, start_den, INT_MAX);
> +            av_set_pts_info(st, 64, 1, base_den);

libavformat/oggparseskeleton.c: In function 'skeleton_header':
libavformat/oggparseskeleton.c:64:13: warning: passing argument 2 of 'av_reduce' from incompatible pointer type
libavutil/rational.h:78:5: note: expected 'int *' but argument is of type 'int64_t *'

-- 
M?ns Rullg?rd
mans at mansr.com