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

[Michael Niedermayer]

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 ...


> 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 ...

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

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070314/c3d9a0b4/attachment.pgp>