[Ffmpeg-cvslog] r8334 - trunk/libavformat/matroska.c

[Baptiste Coudurier]

Hi

Michael Niedermayer wrote:
> Hi
> 
> On Wed, Mar 14, 2007 at 11:27:16AM +0100, Baptiste Coudurier wrote:
> [...]
>>> Wait... That's not so simple. With the attached patch, planet.earth.mkv
>>> don't play at all ! It gives only sound, no video. Probably another
>>> small stupid mistake, but I'm too tired right now. I will look at
>>> this tomorrow, and hopefully, we can get ride of my ugly pts reordering
>>> code.
>>>
>> It seems h264 decoder fails somewhere:
>> Input #0, matroska, from 'planet.earth.mkv':
>>   Duration: 00:01:02.3, bitrate: N/A
>>   Stream #0.0, 1/1000: Audio: ac3, 48000 Hz, 5:1
>>   Stream #0.1, 1/1000: Video: h264, 1280x720, 1/25, 25.00 fps(r)
>>
>> There is no pix_fmt info (yuv420p), I suspect decoder to fail because of
>> parsing does something wrong.
>>
>> Attached a quick patch to h264 to support need_parsing 2, but still it
> 
> patch ok,also it seems this code is missing for other parsers as well, feel
> free to add it to every parser ...

Applied, will look at other parsers tonight.

>> seems under H264 dts computation special case, going further would
>> require h264 parser to compute pict type, and deduce dts accordingly.
> 
> picture type is insufficient to find DTS for h.264 ...

ok.... sadly.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312