[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