On 04/13/2014 05:39 AM, Anton Khirnov wrote: > The parser has no way of knowing video duration, so this value must > always be outright made up.
Please change this to something like: The parser has no way of knowing video duration, and therefore no video parsers set it. > --- > libavformat/utils.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index 164cdd7..e35c1e4 100644 > --- a/libavformat/utils.c > +++ b/libavformat/utils.c > @@ -956,12 +956,6 @@ static int parse_packet(AVFormatContext *s, AVPacket > *pkt, int stream_index) > st->time_base, > AV_ROUND_DOWN); > } > - } else if (st->codec->time_base.num != 0 && > - st->codec->time_base.den != 0) { > - out_pkt.duration = av_rescale_q_rnd(st->parser->duration, > - st->codec->time_base, > - st->time_base, > - AV_ROUND_DOWN); > } > > out_pkt.stream_index = st->index; Patch looks ok -Justin _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel