Reimar Döffinger <[email protected]> added the comment: On Mon, Dec 06, 2010 at 11:27:26AM +0000, Baptiste Coudurier wrote: > + // Handles VFR by flushing page because this frame needs to have a > timestamp > + if (st->codec->codec_id == CODEC_ID_THEORA && > + ogg_granule_to_timestamp(oggstream, granule) > > + ogg_granule_to_timestamp(oggstream, oggstream->last_granule) + 1) {
Checking the codec_id feels wrong and contradicts the comment... Also should the < case be handled in some special case (no idea it can even happen). Either way I am very glad you figured it out, less exposure to Ogg helps me preserving my sanity :-) ________________________________________________ FFmpeg issue tracker <[email protected]> <https://roundup.ffmpeg.org/issue2398> ________________________________________________
