[MPlayer-dev-eng] [PATCH] musepack native decoder

Aurelien Jacobs aurel at gnuage.org
Fri Dec 29 00:50:43 CET 2006


On Thu, 28 Dec 2006 18:48:47 +0100
Aurelien Jacobs <aurel at gnuage.org> wrote:

> On Thu, 28 Dec 2006 12:02:39 +0100
> "Guillaume POIRIER" <poirierg at gmail.com> wrote:
> 
> > Hi,
> > 
> > On 12/28/06, Aurelien Jacobs <aurel at gnuage.org> wrote:
> > > On Sun, 24 Dec 2006 17:52:33 +0100 (MET)
> > > <uoti.urpala at pp1.inet.fi> wrote:
> > >
> > > > Aurelien Jacobs wrote:
> > > > > It currently require -demuxer lavf to work. I don't know how to
> > > > > fix this nicely ? My idea would be to totaly remove libmpcdec
> > > >
> > > > Why does it require -remuxer lavf? How does the output differ from
> > > > demux_mpc?
> > >
> > > I haven't investigated that further. I guess they use different
> > > extradata format or something like this.
> > >
> > > Anyway, patch applied.
> > >
> > > Anyone against removing libmpcdec support ??
> > 
> > Purely from debugging purposes, I think that MPlayer's MPC decoder
> > wrapper should say in for a while so that if ever there's any sample
> > that doesn't decode correctly, it's possible to test with both
> > MPlayer's and lavc MPC decoder/wrapper.
> 
> Ok. Then a simple fix is to not compile demux_mpc.c unless we compile
> with libmpcdec.
> I will commit attached patch soon.

Done.

Aurel



More information about the MPlayer-dev-eng mailing list