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

[Michael Niedermayer]

On Tue, Jun 16, 2009 at 02:58:19PM -0700, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > 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?
> 
> Well, if we do not want s->frame_offset to be 0 for 2nd then we have to
> change the logic setting s->frame_offset.

right, i was confused, patch ok

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

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- 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/20090618/35e88bc0/attachment.pgp>