[FFmpeg-devel] [PATCH] H.264 timestamps in h264_parser - complete set

Michael Niedermayer michaelni
Fri Feb 20 14:43:43 CET 2009

On Fri, Feb 20, 2009 at 12:00:35PM +0100, Ivan Schreter wrote:
> Michael Niedermayer wrote:
>> the parser also can set timetamps, they as well are in a container based
>> timebase
> Ok, then here the next patch to handle frame duration properly. Namely, for 
> interlaced videos, it uses frame duration of 3600 (for 50i) instead of 
> 1800, since it computes it from framerate. Since the parser can compute 
> timestamps, it can also compute duration...
> Actual duration computation will follow in H.264 parser patch to compute 
> timestamps (it's actually already there in old version of the patch, but 
> not propagated to lavf).
> I'll clean up the patches for the parser and post them later today 
> (dependent on this patch, though).

this patch looks wrong, we already have a repeat_pict specifying duration

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090220/2d7fef26/attachment.pgp>

More information about the ffmpeg-devel mailing list