[MPlayer-dev-eng] Re: Add matroska v2 support

Evgeniy Stepanov eugeni.stepanov at gmail.com
Sun Sep 24 23:02:58 CEST 2006


On Friday 15 September 2006 19:10, Federico Menarini wrote:
> 2006/9/15, Evgeniy Stepanov <eugeni.stepanov at gmail.com>:
> > 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.
>
> You're right, here is the corrected patch.

I'll commit in 24h if no one objects.



More information about the MPlayer-dev-eng mailing list