On 28/07/14 15:34, Anton Khirnov wrote: > It is basically a wrapper around av_get_audio_frame_duration(), with a > fallback to AVCodecContext.frame_size. However, that field is set only > when the stream codec context is actually used for encoding or decoding, > which is discouraged. > > For muxing, it is generally the responsibility of the caller to set the > packet duration. > For demuxing, if the duration is not stored at the container level, it > should be set by the parser. > > Therefore, removing the frame_size fallback should not break any > important case. > --- > libavformat/internal.h | 2 -- > libavformat/utils.c | 23 +---------------------- > 2 files changed, 1 insertion(+), 24 deletions(-)
Doesn't sound that dangerous. Fine for me. fix "respo*n*sibility" while at it. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
