[FFmpeg-devel] [RFC/PATCH] matroska timestamps not always pts

[elupus]

Hi,

Apperently the timestamps stored in matroska container can mean different 
things. For native tracks they normally mean pts, while for others this 
isn't always the case.

The case i'm trying to nail here is the ms compatibility case. In this case, 
timestamps should correspond to the frame numbers in avi. This patch makes 
demuxer only set the correct timestamp (dts/pts) in the packet.

Sadly ffmpeg still fills in the pts of the packet with the same value as dts 
somewhere later in utils.c i'd expect. Atleast this is the case for the 
mpeg4 video's i've tested. I think this could be due to mpeg4 parser not 
setting has_b_frames, but i'm not entirely sure.

Oh, and the patch probably doesn't apply cleanly, had to manually remove 
some hunks that was unrelated. Just want to know if something along the 
lines of this would be accepted

Regards
/Joakim 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: timestamps.patch
Type: application/octet-stream
Size: 2041 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071118/8b337760/attachment.obj>