Le decadi 30 messidor, an CCXXII, Benoit Fouet a écrit :
> +        if (frame && av_frame_get_pkt_timebase(frame).num)
> +            avpkt->duration = av_rescale_q(av_frame_get_pkt_duration(frame),
> +                                           av_frame_get_pkt_timebase(frame),
> +                                           avctx->time_base);

I suspect this is not a good idea at all: in a lot of cases,
av_frame_get_pkt_duration(frame) contains a random value not correlated to
the actual duration of the packet.

Making use of a field that was previously unused is an API change, and it
must be considered very carefully.

I suspect you can observe the inconsistent behaviour just by using ffmpeg
with a frame rate change, but I am not certain that is as simple as that.
Other applications may break more easily.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to