Ronald S. Bultje wrote:
Hi,

On Tue, Aug 16, 2011 at 11:06 PM, Kostya<kostya.shish...@gmail.com>  wrote:
 On Tue, Aug 16, 2011 at 04:13:35PM -0700, Ronald S. Bultje wrote:
 On Mon, Aug 15, 2011 at 11:43 PM, Kostya<kostya.shish...@gmail.com>  wrote:
 >  On Mon, Aug 15, 2011 at 09:42:50PM +0100, Måns Rullgård wrote:
 >>  Kostya<kostya.shish...@gmail.com>  writes:
 >  +    } else {
 >  +        if (type != 3)

/* I/P frames have a pts that is after the previous I or P frame,
   so the difference is positive module 13bit */

 >  +            s->pts = pc->key_dts + ((pts - pc->key_pts)&  0x1FFF);
 >  +        else

/* B frames have a pts that is before the previous I or P frame,
   so the difference is negative module 13bit */

 >  +            s->pts = pc->key_dts - ((pc->key_pts - pts)&  0x1FFF);
 >  +    }

 I suppose this could use some doxy. What happens if we have a matroska
 file with RV40 video? Does it need the PTS correction? Is the parser
 invoked?

 It's not invoked unless demuxer says so by setting stream->need_parsing.

Some comments would still be nice so I understand the above code
without crackpot smoking.

see above

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to