[FFmpeg-devel] [PATCH] Fix DVCPRO HD 720p Audio Timestamp Generation

[Baptiste Coudurier]

Hi Roman,

Roman V Shaposhnik wrote:
> On Fri, 2009-06-19 at 15:38 -0500, Brian Brice wrote:
>> Hello.
>>
>> I have found that DVCPRO HD 720p clips have PTS values generated
>> slightly incorrectly and have attached a patch that should fix it.
>> Roman, of course, can verify the logic is correct.
>>
>> The thing with these 720p clips there's 2 video frames within a buffer
>> that is the size of one 1080i DV frame, but 1 audio frame for 8
>> channels (4 streams of stereo).  First 4 channels are with the first
>> 720p frame, and the last 4 channels are with the second 720p frame.
>> All 8 channels (or 4 streams) should have the same PTS throughout the
>> duration of 2 720p frames.
>>
>> For example, you would get a PTS of 0 for audio streams 1 and 2, then
>> PTS 1602 for audio streams 3 and 4.  This patch moves the byte count
>> increment until after the last 2 audio streams have been read.
> 
> This looks correct. Can somebody, please, apply?

Applied.

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