[MPlayer-dev-eng] Re: [PATCH] new C only matroska demuxer

[Aurelien Jacobs]

On Fri, 23 Jan 2004 21:57:01 +0100
ChristianHJW <christian at matroska.org> wrote:

> Aurelien, we have MPEG2 working in MKV now, if you want a sample file 
> pls. come to the #matroska channel, it would be nice if you could add 
> MPEG2 playback from MKV in your mplayer patch. VLC people have it 
> working since today, Xine people got test files already ...

Here is a very simple patch to do this. The only problem I had is that
the following code in video_read_frame (libmpdemux/video.c) caused
the MPEG2 stream to play at a bad frame rate.

      case DEMUXER_TYPE_MATROSKA:
	if(d_video->pts>0 && pts1>0 && d_video->pts>pts1)
	  frame_time=d_video->pts-pts1;
        break;

I simply removed it but I guess it can cause trouble with some files
(even if I've not found such a file). This should probably be done
only for some types of matroska contents. I someone tell me with
which content it can be useful exactly (I guess it's REAL video),
I will improve this patch.

Aurélien
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: mkv-mpeg2.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040127/c1b17b96/attachment.txt>