On 03/01/2012 03:31 PM, Anton Khirnov wrote:
> Demuxers are not supposed to set it.
> Set stream timebase and framerates instead (this is a cfr container with
> no timestamps).
> ---
> libavformat/electronicarts.c | 4 +-
> tests/ref/fate/ea-dct | 270
> +++++++++++++++++++++---------------------
> tests/ref/fate/ea-vp60 | 268
> +++++++++++++++++++++---------------------
> tests/ref/fate/ea-vp61 | 242 +++++++++++++++++++-------------------
> 4 files changed, 393 insertions(+), 391 deletions(-)
>
> diff --git a/libavformat/electronicarts.c b/libavformat/electronicarts.c
> index a10e645..c7fbded 100644
> --- a/libavformat/electronicarts.c
> +++ b/libavformat/electronicarts.c
> @@ -417,9 +417,11 @@ static int ea_read_header(AVFormatContext *s)
> st->codec->codec_type = AVMEDIA_TYPE_VIDEO;
> st->codec->codec_id = ea->video_codec;
> st->codec->codec_tag = 0; /* no fourcc */
> - st->codec->time_base = ea->time_base;
> st->codec->width = ea->width;
> st->codec->height = ea->height;
> + avpriv_set_pts_info(st, 33, ea->time_base.num, ea->time_base.den);
> + st->r_frame_rate = st->avg_frame_rate =
> (AVRational){ea->time_base.den,
> +
> ea->time_base.num};
> }
LGTM.
-Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
