[MPlayer-dev-eng] [PATCH] Add matroska v2 support

[Evgeniy Stepanov]

On Friday 08 September 2006 16:24, Federico Menarini wrote:
> Hi,
> this patch adds matroska v2 (i.e. simpleblock) support to mplayer, a
> smaller overhead structure of matroska files. To create a mkv v2
> compliant file you can use mkvmerge with --engage use_simpleblock
> switch.

Looks good, with 2 minor remarks:
1. demux_mkv.c uses spaces for indentation, so your patch should do the same.
2. A should comment, or a #define near
if (!(flags&0x80))
would be nice. Not many people remember what 0x80 stands for in SimpleBlock 
flags.