[FFmpeg-devel] [PATCH] Fix timestamp retrieval for MPEG-2 non-frame picture encoding

[Michael Niedermayer]

On Sat, Aug 15, 2009 at 01:02:48AM -0500, Brian Brice wrote:
> Hello.
> 
> I have found a bug with timestamp information being lost with MPEG-2
> files that were encoded not as a frame picture.  You'll essentially
> have 2 fields encoded one after another, with 2 PICTURE_START_CODE's
> occurring in a single frame.  The current implementation of the MPEG
> video parser overwrites the PTS/DTS that was stored at the start of
> the frame with AV_NOPTS_VALUE when the second PICTURE_START_CODE comes
> around.
> 
> I've seen this with I-frame only MPEG-2 files along with select
> DVD/VOBs.  I have uploaded a sample I-frame only clip for anyone who
> would like to test also:
> 
> http://heapify.org/dynamic/intra-only-hd.mpg -- Only about 20 frames
> (0.667 sec).
> 
> The patch is attached.  Thanks!

hehe, i just debuged the same issue on a different file and fixed it
the same way :(
argh, when i saw your patch ages ago i didnt understand the issue fully
so i left it for later ...
and now i didnt remember and just wanted to go over all other
issues in the code i was just working on ...
failure on my side ...
equivalent change in svn now though :)

[...]
-- 
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: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100222/4fdef9a6/attachment.pgp>