2014/1/17 Derek Buitenhuis <[email protected]> > From: Justin Ruggles <[email protected]> > > The track duration is often not reliable or is not the duration > represented by the number of frames. In those cases, avg_frame_rate > was reported incorrectly. Removing this code falls back to the > default calculation in avformat_find_stream_info(). > > This is a partial revert of commit c3aeaa540. > > Signed-off-by: Derek Buitenhuis <[email protected]> > --- > libavformat/mov.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index b5019f5..dc5b42b 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -2138,10 +2138,6 @@ static int mov_read_trak(MOVContext *c, AVIOContext > *pb, MOVAtom atom) > st->sample_aspect_ratio = av_d2q(((double)st->codec->height * > sc->width) / > ((double)st->codec->width * > sc->height), INT_MAX); > } > - > - if (st->duration != AV_NOPTS_VALUE && st->duration > 0) > - av_reduce(&st->avg_frame_rate.num, &st->avg_frame_rate.den, > - sc->time_scale*st->nb_frames, st->duration, > INT_MAX); > } > > // done for ai5q, ai52, ai55, ai1q, ai12 and ai15. > -- > 1.8.5.3 > > _______________________________________________ > libav-devel mailing list > [email protected] > https://lists.libav.org/mailman/listinfo/libav-devel >
st->duration is not track duration. It's media duration since it comes from mdhd.duration. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
