[FFmpeg-devel] libavcodec/mpeg12.c Problem showed up in SVN-r18959

[Baptiste Coudurier]

Hi,

On 07/22/2009 07:00 PM, Greg McCrory wrote:
> On 07/22/2009 08:08 PM, Limin Wang wrote:
>> Hi,
>>
>>
>> * Greg McCrory<greg01 at coolbrew.com> [2009-07-22 16:29:39 -0500]:
>>
>>> Index: libavcodec/mpeg12.c
>>> ===================================================================
>>> --- libavcodec/mpeg12.c (revision 18958)
>>> +++ libavcodec/mpeg12.c (revision 18959)
>>> @@ -2226,7 +2226,7 @@
>>> }
>>> }
>>> if(s&& state == PICTURE_START_CODE){
>>> - ff_fetch_timestamp(s, i-4, 1);
>>> + ff_fetch_timestamp(s, i-3, 1);
>>> }
>>> }
>>> }
>>
>> I have got one mpeg2 samples has problem before and I use below
>> patch to fix it. Maybe you can try it whether it's same problem.
>>
>>
>> --- libavcodec/mpeg12.c (?? 19500)
>> +++ libavcodec/mpeg12.c (????)
>> @@ -2224,8 +2224,10 @@
>> return i-3;
>> }
>> }
>> - if(s&& state == PICTURE_START_CODE){
>> - ff_fetch_timestamp(s, i-3, 1);
>> +
>> + if(s&& s->fetch_timestamp&& state == PICTURE_START_CODE){
>> + ff_fetch_timestamp(s, i-4, 1);
>> + s->fetch_timestamp=0;
>> }
>> }
>> }
>>
>>
>> Thanks,
>> Limin
>
> Sorry, no. Still exhibits same jitter problem.
>
> This, however works beautifully with ntsc-dvd mpeg2 video:
>
> if(s && state == PICTURE_START_CODE){
> ff_fetch_timestamp(s, i-4, 1);
> }
>
> Like I said, I don't know enough about what going on to understand why,
> I just know that it works.

Can you upload a small sample according to
http://www.ffmpeg.org/bugreports.html

Thanks.

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