[FFmpeg-devel] [PATCH 2/2] avformat/rawdec: Fix avg_framerate for h264

[Ronald S. Bultje]

Hi,

On Fri, Jun 3, 2016 at 6:12 PM, Michael Niedermayer <michael at niedermayer.cc>
wrote:

> The framerate is 25 which is a fixed default and is wrong undo the 1 line
> change which caused this regression
> Fixes Ticket 5444
>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/rawdec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/rawdec.c b/libavformat/rawdec.c
> index 8c734db..2bcb672 100644
> --- a/libavformat/rawdec.c
> +++ b/libavformat/rawdec.c
> @@ -84,8 +84,8 @@ int ff_raw_video_read_header(AVFormatContext *s)
>      st->codecpar->codec_id = s->iformat->raw_codec_id;
>      st->need_parsing = AVSTREAM_PARSE_FULL_RAW;
>
> -    st->avg_frame_rate = s1->framerate;
>      st->internal->avctx->framerate = s1->framerate;
> +    st->internal->avctx->time_base = av_inv_q(s1->framerate);
>      avpriv_set_pts_info(st, 64, 1, 1200000);


This seems odd, I thought that we should not touch internal in lavf. Also,
are both parts required (- as well as +), or is the + only enough?

Ronald