[FFmpeg-devel] [PATCH] correctly set h264 field timestamp

Michael Niedermayer michaelni
Tue Jun 16 23:36:15 CEST 2009


On Sun, Jun 14, 2009 at 11:18:20PM -0700, Baptiste Coudurier wrote:
> Hi,
> 
> $subject.
> 
> Add check for next_frame_offset because the condition !s->frame_offset
> is valid for both first fields in a stream, however the timestamp refers
> to the first field.
> 
> This is only needed for the second field.

is it correct (aka do we want it) that s->frame_offset is 0 for the 2nd field?
if yes then my gut feeling says this fix is not enough to associate
timestamps for field based h264 correctly, but maybe iam missing something

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- 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/20090616/5cd67597/attachment.pgp>



More information about the ffmpeg-devel mailing list