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

[Aurelien Jacobs]

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.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpc.diff
Type: text/x-diff
Size: 1140 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20061228/00c6a4d5/attachment.diff>