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

Reply via email to