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

Evgeniy Stepanov eugeni.stepanov at gmail.com
Mon Sep 25 23:18:01 CEST 2006


On Monday 25 September 2006 01:02, Evgeniy Stepanov wrote:
> 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.

Done.



More information about the MPlayer-dev-eng mailing list