[MPlayer-dev-eng] [PATCH] demux_mkv wrong keyframe tagging?

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Fri Apr 14 12:50:05 CEST 2006


Hi,
currently mkv demuxer seems to tag keyframes with a "flags" value of 1,
whereas all other code seems to use 0x10.
So, would the attached patch be correct?
I think the problems should show when using mencoder -ovc copy, but I do
not have any really useful samples...

Greetings,
Reimar Döffinger
-------------- next part --------------
Index: libmpdemux/demux_mkv.c
===================================================================
RCS file: /cvsroot/mplayer/main/libmpdemux/demux_mkv.c,v
retrieving revision 1.57
diff -u -r1.57 demux_mkv.c
--- libmpdemux/demux_mkv.c	3 Apr 2006 19:41:17 -0000	1.57
+++ libmpdemux/demux_mkv.c	14 Apr 2006 10:44:21 -0000
@@ -3142,7 +3142,7 @@
                   memcpy (dp->buffer, buffer, size);
                   if (modified)
                     free (buffer);
-                  dp->flags = (block_bref == 0 && block_fref == 0) ? 1 : 0;
+                  dp->flags = (block_bref == 0 && block_fref == 0) ? 0x10 : 0;
                   dp->pts = mkv_d->last_pts + i * track->default_duration;
                   ds_add_packet (ds, dp);
                 }


More information about the MPlayer-dev-eng mailing list