Michael Niedermayer <[EMAIL PROTECTED]> added the comment:

AVPacket.duration specifies the packet duration. If after reordering the pts +
duration doesnt match the next pts, that is a discontinuity and should be
treated as such. If the duration isnt known then the discontinuity detection
should likely be disabled. Subtitle streams are NOT a special case. Your patch
intruduces bugs (that is ignore discontinuities in subtitle streams) but does
not fix any.
Also the patch would have to be split in 3 patches+issues at least, there are
changes to a few fprintf() these are seperate, and the 2 cases of special
handling of subtitles also are seperate and independant.

----------
status: new -> closed
substatus: new -> reject

______________________________________________________
FFmpeg issue tracker <[EMAIL PROTECTED]>
<https://roundup.mplayerhq.hu/roundup/ffmpeg/issue356>
______________________________________________________

Reply via email to