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

Michael Niedermayer michael at niedermayer.cc
Sat Jun 4 02:36:53 CEST 2016


On Fri, Jun 03, 2016 at 07:39:08PM -0400, Ronald S. Bultje wrote:
> 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?

the - part is what is needed
i added the 2nd part as that was how it was originally before the
regression


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

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160604/cd4bcef5/attachment.sig>


More information about the ffmpeg-devel mailing list