On Sat, Jun 04, 2016 at 02:36:53AM +0200, Michael Niedermayer wrote: > 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 <mich...@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 <mich...@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
applied the needed part [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Everything should be made as simple as possible, but not simpler. -- Albert Einstein
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
