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

Baptiste Coudurier baptiste.coudurier
Sun Jun 21 05:50:53 CEST 2009


Michael Niedermayer wrote:
> 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

Applied.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org



More information about the ffmpeg-devel mailing list