[MPlayer-dev-eng] [PATCH] Matroska/Wavpack-support

Aurelien Jacobs aurel at gnuage.org
Mon Oct 22 00:58:00 CEST 2007


Diego Biurrun wrote:

> On Fri, Oct 19, 2007 at 10:23:05PM +0200, Aurelien Jacobs wrote:
> > On Fri, 19 Oct 2007 02:21:43 +0200
> > Jan Christoph Terasa <kohlrabi82 at gmx.de> wrote:
> > 
> > > This patch enables playback support of wavpack streams inside matroska 
> > > containers. Since Wavpack-support is already provided by ffdshow the changes 
> > > are just minor/trivial. On my test playback works fine, so I ignored the body 
> > > of the last if-statement.
> > 
> > So I guess the whole if-statement should be removed !
> > Except that, the patch looks OK.
> 
> IMO you should take over maintainership of the Matroska demuxer.  Moritz
> does not follow the mailing lists any longer and you are the original
> author of the code ...

I'm not that much interested in mplayer demuxers. I prefer improving lavf
demuxer so that it get at least as good as mplayer one, so that we can
use lavf by default.
But, well, OK. I will try to maintain this one... Wait... It seems I'm
already marked as maintainer, along with Moritz ! I didn't even remembered
about that. So everything is fine.

Patch tested, and applied. (the last if-statement was in fact required)

Aurel



More information about the MPlayer-dev-eng mailing list